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.

Sign up for GitHub

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

Jump to bottom

GET/POST/PATCH requests containing '%' as param or within the body will raise Rack::Utils::InvalidParameterError: invalid %-encoding #9

Open
foxxx0 opened this issue Jan 16, 2017 · 2 comments
Open

GET/POST/PATCH requests containing '%' as param or within the body will raise Rack::Utils::InvalidParameterError: invalid %-encoding #9

foxxx0 opened this issue Jan 16, 2017 · 2 comments
Assignees
@foxxx0
Labels
bug

Comments

@foxxx0
Copy link
Contributor

foxxx0 commented Jan 16, 2017

Stumbled upon this problem while trying to set a MailAccount password containing a '%' (percent sign).

It seems that there is a somewhat undesirable behavior of Rack in this case when Content-Type is set to application/x-www-form-urlencoded or even empty.

This needs to be fixed inside the application to simply return 400 - invalid request.
@foxxx0 foxxx0 added the bug label Jan 16, 2017
@foxxx0 foxxx0 self-assigned this Jan 16, 2017
@foxxx0
Copy link
Contributor Author

foxxx0 commented Jan 16, 2017

rack/rack#337

@foxxx0
Copy link
Contributor Author

foxxx0 commented Jan 16, 2017

freerange/webhooks#21 (comment)

@foxxx0 foxxx0 mentioned this issue Jan 16, 2017
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@foxxx0 foxxx0

Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@foxxx0