Regression in output of sync_state_genSyncSpec

[tomaka]

Before #8783, calling the sync_state_genSyncSpec JSON-RPC function would return something like this:

{"jsonrpc":"2.0","id":"...","result": {"name":"Westend", "chainType": "Live", ... } }

After the PR, the output looks like this:

{"jsonrpc":"2.0","id":"...","result": "{\"name\":\"Westend\", \"chainType\": \"Live\", ... }" }

In other words, the result has been stringified.

cc @niklasad1

[niklasad1]

oops, sorry let me have a look

[tomaka]

Note that I haven't actually checked what the function was returning before the PR (I have only checked the after). However we have a script that started failing after the PR has been deployed, and this script is directly expecting an object instead of a string, so what I wrote is the only possible explanation.

[niklasad1]

you are correct, it returned serde_json::Value before I think I was got annoyed that as_json already returned it as JSON but forgot that the Strings are serialized differently.

one sec, I think we need to block the current release for this right?