fix: remove the replacement of escape characters in json response formatting

[musale]

Overview

Removes replacement of escaped characters in the stringified json. closes #3158

Testing Instructions

• run the query https://graph.microsoft.com/v1.0/appCatalogs/teamsApps?$expand=*
• Check the response preview tab to have well formatted json

[github-actions[bot]]

Azure Static Web Apps: Your stage site is ready! Visit it here: https://jolly-sand-0ac78c710-3163.centralus.azurestaticapps.net

[thewahome]

@musale have you confirmed this to work with Safari? Though done a long time back, the replacement code was supposed to fix this issue #895 .

If this change makes the replacement code unnecessary, then we might not need to stringify then parse. We can just pass the Json as is and get rid of this function

[musale]

Pinging @Onokaev to validate this on Safari.

[SilasKenneth]

Tested in Safari on iOS 17.5.1

———

[musale]

Thank you @SilasKenneth! Also tested the paths in the #895 and they work as expected.

[thewahome]

@musale since the test works, the stringify -> parse is an unnecessary set of legacy steps. Is this something that we can confidently remove?

[musale]

@musale since the test works, the stringify -> parse is an unnecessary set of legacy steps. Is this something that we can confidently remove?

Sure! Unless that telemetry is important, I don't see the need for that function too.

[github-actions[bot]]

Azure Static Web Apps: Your stage site is ready! Visit it here: https://jolly-sand-0ac78c710-3163.centralus.azurestaticapps.net

[github-actions[bot]]

Azure Static Web Apps: Your stage site is ready! Visit it here: https://jolly-sand-0ac78c710-3163.centralus.azurestaticapps.net

[sonarcloud[bot]]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud