[jira] [Comment Edited] (SLING-11538) Add display context for JSON string

["Konrad Windszus (Jira)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/SLING-11538?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17578978#comment-17578978 ] 

Konrad Windszus edited comment on SLING-11538 at 8/13/22 2:44 PM:
------------------------------------------------------------------

{-}As implementation one could either write such an escape method from scratch or use the JSON-P Standard ({-}[-https://javaee.github.io/jsonp/).-|https://javaee.github.io/jsonp/)]

The simplest approach is just leveraging {{StringEscapeUtils.escapeJson(...)}} from Commons Lang3 ([https://commons.apache.org/proper/commons-lang/apidocs/org/apache/commons/lang3/StringEscapeUtils.html#escapeJson-java.lang.String-)] which is already a dependency.


was (Author: kwin):
As implementation one could either write such an escape method from scratch or use the JSON-P Standard ([https://javaee.github.io/jsonp/).|https://javaee.github.io/jsonp/)]

> Add display context for JSON string
> -----------------------------------
>
>                 Key: SLING-11538
>                 URL: https://issues.apache.org/jira/browse/SLING-11538
>             Project: Sling
>          Issue Type: Improvement
>          Components: HTL
>            Reporter: Konrad Windszus
>            Assignee: Konrad Windszus
>            Priority: Major
>             Fix For: Scripting HTL Engine 1.4.22-1.4.0
>
>          Time Spent: 20m
>  Remaining Estimate: 0h
>
> It would be useful to add an output context to HTL to be used inside JSON. As JSON is very complex, the most essential one which currently cannot be achieved with any other existing contexts is escaping for a JSON String value (compare with https://github.com/adobe/htl-spec/issues/5).
> I propose to introduce a new context {{jsonString}} next to {{scriptString}} in https://github.com/apache/sling-org-apache-sling-scripting-sightly/blob/192d953514e6e579428cda157a7e83fc2a05cc01/src/main/java/org/apache/sling/scripting/sightly/impl/engine/extension/XSSRuntimeExtension.java#L93.
> As it is not part of the official HTL spec at https://github.com/adobe/htl-spec/blob/master/SPECIFICATION.md#121-display-context it needs to be listed as Sling-specific addition in https://sling.apache.org/documentation/bundles/scripting/scripting-htl.html#extensions-of-the-htl-specification.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)