-
Notifications
You must be signed in to change notification settings - Fork 2.1k
[Bug] URL escaped UTF-8 code units get de-encoded #975
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
Comments
👋 Thanks for opening your first issue! If you're reporting a 🐞 bug, please make sure To help make this a smooth process, please be sure you have first read the |
I also ran into an issue where the |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
This is still an issue in Insomnia 6.0.2. When I set a query parameter with a value such as It seems like Insomnia's logic is backwards and it's doing the opposite of what it should do (it's removing percent-encoding from query parameter values instead of adding it). |
So, how do we bump this up? |
Also an issue on MacOS v6.0.2 It breaks if the first two characters after the % are 0-9 or A-F Is the issue somewhere around here? https://github.com/getinsomnia/insomnia/blob/develop/packages/insomnia-url/src/querystring.js#L242 |
Hmm, ya, already-encoded characters should probably stay encoded. @ganey already found the existing logic to do this but it currently only performs this for a specific set of whitelisted characters. |
I thought I used to be able to query with a leading %, but it seems to be broken in 6.0.2.
|
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
Any update on this? |
Still running into this issue. Even affects double-encoding: This happens for both of the following URLs
|
Sorry folks! This was incorrectly tagged as a feature request along with other bugs. I'm going back through and cleaning up, sorry for the inconvenience. |
I ran into this problem specifically with commas. Using |
Details
I'm using query params to send a request towards an API but the URL isn't rendered how I want it to.
This is what I'm trying to achive using the query tab in in the request:
But gets transformed into:
I'm using a service that requires me to make the request in the first format and so I can't test this service using Insomnia.
The text was updated successfully, but these errors were encountered: