-
-
Notifications
You must be signed in to change notification settings - Fork 4.6k
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
feat: add the ability to configure query params encoding for requests #4412
feat: add the ability to configure query params encoding for requests #4412
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Pranay-Pandey thank you for the contribution, I have made some minor changes to the PR
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It'll be good to display a warning (might lead to malformed requests, etc) via the inspector while the encoding preference is set to Disable
as an enhancement down the line.
8610a8a
to
4aa2fff
Compare
Closes #2829
Add optional setting of encoding the URL parameters in the request. A new setting option is introduced which can be toggled between auto, disable and auto.
Also have fixed the bug with preProcessRequest function where double encoding was happening.
What's changed
General
introduced and moved language, experiments section withinPreview