Skip to content
This repository was archived by the owner on Jul 31, 2022. It is now read-only.

Get request does not parse wildcard characters +& correctly. #115

Closed
Katsute opened this issue Nov 15, 2020 · 0 comments · Fixed by #116
Closed

Get request does not parse wildcard characters +& correctly. #115

Katsute opened this issue Nov 15, 2020 · 0 comments · Fixed by #116
Assignees
@Katsute
Labels
bug Something isn't working as expected priority Something important is broken. Only use if a feature essential to operations is broken.
Milestone
4.2.0

Comments

@Katsute
Copy link
Member

Katsute commented Nov 15, 2020 •
edited
Loading

The decoder method does not parse + and & as a literal. It reads them as and ​ for some reason.
@Katsute Katsute added bug Something isn't working as expected priority Something important is broken. Only use if a feature essential to operations is broken. labels Nov 15, 2020
@Katsute Katsute added this to the 4.2.0 milestone Nov 15, 2020
@Katsute Katsute self-assigned this Nov 15, 2020
Katsute added a commit that referenced this issue Nov 15, 2020
@Katsute
Fix #115
be40336 
Issue was caused by decoder parsing an already decoded query. This would break the decoder method for characters `+` and `&`.
@Katsute Katsute linked a pull request Nov 15, 2020 that will close this issue
Fixed #115, GET wildcards +& not parsed correctly #116
Merged
Katsute added a commit that referenced this issue Nov 15, 2020
@Katsute
Fix #115 (#116)
a96ae6b 
Issue was caused by decoder parsing an already decoded query. This would break the decoder method for characters `+` and `&`.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@Katsute Katsute

Labels
bug Something isn't working as expected priority Something important is broken. Only use if a feature essential to operations is broken.
Projects
None yet
Milestone
4.2.0
Development

Successfully merging a pull request may close this issue.

Fixed #115, GET wildcards +& not parsed correctly Ktt-Development/simplehttpserver
1 participant
@Katsute