Render HTML-safe JSON strings, especially for use in <script> tags
#483
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
[Presently there's no issue for this, though #181 might be closest. I use this with Maud in a private codebase of my own, and I thought it could be more widely useful. I also wanted to expose my approach to the light to check if it does actually make sense.]
Reproduced from
maud/src/json.rs:TryRendertrait, say, that would allow for fallible renderers. Alternatively, I think theexpect(...)call could remain in theimpl Render for serde_json::ValuebecauseValueshould be almost/entirely guaranteed to be valid JSON. TheJsonwrapper could be removed; just ask callers to useserde_json::to_valuethemselves.There are also a few things I fixed in passing, but I can extract those to a separate PR if necessary.