arianvp / arianvp/servis

Move QueryParam into Handler

Open
#5 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
TeX
Stars
28
Forks
2
PR merge metrics
No merged PRs in 30d

Description

Query parameters are annyoing to prove disjointness because they can appear in any order. We shouldn't consider them part of the route. But part of the Handler. This is similar to how its done in https://github.com/tel/serv but also flask

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by locating QueryParam, Handler, and the route-matching or disjointness entry points; the issue names no files or tests. Compare the intended behavior with tel/serv and Flask, then verify that query-parameter order no longer affects route disjointness and that query parameters are handled by Handler.

Written by the indexing model from the issue text.

Assessment

Tech stack
flask
Domain
backend
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.