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

GMC: the jsonstring escape need to be improved #245

Open
KfreeZ opened this issue Jul 31, 2024 · 0 comments
Open

GMC: the jsonstring escape need to be improved #245

KfreeZ opened this issue Jul 31, 2024 · 0 comments
Assignees
Labels

Comments

@KfreeZ
Copy link
Collaborator

KfreeZ commented Jul 31, 2024

For security concern, #243 introduced the escapes of jsonstring to avoid code scan alert.
However, it's a quick fix, the more detailed improvement need to be done and a full UT is required.
Thanks @mkbhanda point this out and this issue is to track the following up.
the json spec:
https://www.json.org/json-en.html

@KfreeZ KfreeZ self-assigned this Jul 31, 2024
@KfreeZ KfreeZ added the gmc label Jul 31, 2024
@KfreeZ KfreeZ added the good first issue Good for newcomers label Aug 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants