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

fix(#2376): send unmodified request json body #2382

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

lohxt1
Copy link
Collaborator

@lohxt1 lohxt1 commented May 29, 2024

fixes: #2376, #2362, #1768

added a checkbox in the collection settings' presets tab to choose if the request body should be parsed and de-commented, or be sent as it is needs to be added

Screenshot 2024-06-19 at 3 30 11 PM

(part of the bruno config presets)

raw_json_toggle.mov

currently bruno supports adding comments in the json body
and before the actual request is made the comments are removed and the body is parsed as json
ref: PR

@jwetzell
Copy link
Contributor

jwetzell commented Jun 21, 2024

Does this cover #2207, #2438 as well? I would think getting a less destructive JSON parsing library like suggested in one of those issues would be the better/more transparent solution for the end user.

The examples brought up by the users in these several issues should probably be added to the tests if there is any for this to ensure behavior as well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

trailing decimal points with a 0 are truncated.
2 participants