Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Raise an error if query string has multiple definitions #1338

Merged
merged 1 commit into from Jul 29, 2018

Conversation

@pgjones
Copy link
Member

@pgjones pgjones commented Jul 29, 2018

Previously to this commit if when creating an environ (i.e. for
testing) a query string is defined in the path and as an argument the
path-query-string would be ignored. This results in the path not
matching any routes and hence an unexpected result. This commit fixes
this issue by raising a ValueError if someone should do this.

Raising ValueError is considered the correct approach as it is unclear
which individual query string is the user's intention or if they
should be combined.

Previously to this commit if when creating an environ (i.e. for
testing) a query string is defined in the path and as an argument the
path-query-string would be ignored. This results in the path not
matching any routes and hence an unexpected result. This commit fixes
this issue by raising a ValueError if someone should do this.

Raising ValueError is considered the correct approach as it is unclear
which individual query string is the user's intention or if they
should be combined.
@davidism davidism merged commit 816a1e9 into pallets:master Jul 29, 2018
1 check passed
@davidism
Copy link
Member

@davidism davidism commented Jul 29, 2018

Thanks, I added a changelog entry in master. Remember to use separate branches for PRs. :-)

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 13, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants