Marshalling the value in set command
Issue link - https://github.com/DiceDB/dice/issues/750
Changes Made:
- Handling of values to ensure proper serialization and deserialization of nested map structures in SET command.
- Added UTs
@pratikpandey21 Just want to confirm if the escape characters in API response is fine?
@pratikpandey21 Testcases are running fine, Please help in next steps to allow approval for the CI workflows
@pratikpandey21 Just want to confirm if the escape characters in API response is fine?
![]()
I don't think escape chars should be there in response. It should be a proper json. @lovish2525 please check once. cc: @pratikpandey21
@pratikpandey21 Just want to confirm if the escape characters in API response is fine?
I don't think escape chars should be there in response. It should be a proper json. @lovish2525 please check once. cc: @pratikpandey21
In redis JSON.GET i have noticed escape characters there should not be problem with that as long as client is able to parse it without having errors
@vinitparekh17 @pratikpandey21 Fixed http UTs
@vinitparekh17 So are we good to go with escape chars?
@lovish2525 looks like redis does have escape chars in response. I believe we can keep the same. Meanwhile, please fix the build errors. cc: @pratikpandey21
@lucifercr07 Kindly approve the workflow
@lovish2525 lgtm @lucifercr07 hold on this marge for a while, i am about to create pr for http response in proper json format