GMC: the jsonstring escape need to be improved

opea-project / GenAIInfra

Containerization and cloud native suite for OPEA

Apache License 2.0

32 stars 60 forks source link

GMC: the jsonstring escape need to be improved #245

Open KfreeZ opened 4 months ago

KfreeZ commented 4 months ago

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

chickenrae commented 1 month ago

@KfreeZ Are you able to complete this during the month of October for the OPEA Hackathon?