-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Bruno converts decimal numbers in JSON body to strings #2438
Comments
The attached file is a .bru file, but those are not allowed to be uploaded, so I changed the file extension. |
Seems to be an issue with "long" decimals as not all of the decimals in your example got converted to strings. |
The problem is this line here for someone looking to fix this. It is using a library to parse the JSON but that library is converting these long decimals into "BigNumber" type. @sanjai0py is the one who added this BigNumber parsing in so might be able to speak to the reasoning or if this side-effect was not known. |
@sanjai0py Can you comment on this? |
This has been fixed as a part of PR #2773, and will go out in the v1.27.0 release. |
I have checked the following:
Describe the bug
Bruno converts decimal numbers in JSON to strings as visible in the screenshot.
.bru file to reproduce the bug
example.json
Screenshots/Live demo link
The text was updated successfully, but these errors were encountered: