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
[Bug] Prettify not working if JSON body contains comments #1830
Comments
asonkeri
added a commit
to asonkeri/bruno
that referenced
this issue
Mar 16, 2024
Added Microsoft's node-jsonc-parser library that is able to format json with comments. Seems to handle bigints properly too. It may be good to consider replacing existing `decomment` library with `jsonc-parser` if its only use case is to decomment json. Fixes usebruno#1830
5 tasks
Formally, comments are not part of JSON and many tools will tell you about that, too. |
As documented in #396, Bruno supports JSON with comments in the request body whether they are part of JSON spec or not. |
helloanoop
added a commit
that referenced
this issue
Apr 9, 2024
Added Microsoft's node-jsonc-parser library that is able to format json with comments. Seems to handle bigints properly too. It may be good to consider replacing existing `decomment` library with `jsonc-parser` if its only use case is to decomment json. Fixes #1830 Co-authored-by: Anoop M D <anoop.md1421@gmail.com>
This will go out in the |
slowjoe007
pushed a commit
to slowjoe007/bruno
that referenced
this issue
Apr 10, 2024
Added Microsoft's node-jsonc-parser library that is able to format json with comments. Seems to handle bigints properly too. It may be good to consider replacing existing `decomment` library with `jsonc-parser` if its only use case is to decomment json. Fixes usebruno#1830 Co-authored-by: Anoop M D <anoop.md1421@gmail.com>
Its-treason
pushed a commit
to Its-treason/bruno
that referenced
this issue
Apr 13, 2024
Added Microsoft's node-jsonc-parser library that is able to format json with comments. Seems to handle bigints properly too. It may be good to consider replacing existing `decomment` library with `jsonc-parser` if its only use case is to decomment json. Fixes usebruno#1830 Co-authored-by: Anoop M D <anoop.md1421@gmail.com>
lizziemac
pushed a commit
to lizziemac/bruno
that referenced
this issue
May 4, 2024
Added Microsoft's node-jsonc-parser library that is able to format json with comments. Seems to handle bigints properly too. It may be good to consider replacing existing `decomment` library with `jsonc-parser` if its only use case is to decomment json. Fixes usebruno#1830 Co-authored-by: Anoop M D <anoop.md1421@gmail.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Similarly to #1553 prettifying the JSON body fails if the body contains comments:
Tested on v1.11.0 on Windows 11
The text was updated successfully, but these errors were encountered: