[ 
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:45 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/)]

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.

> 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)

Reply via email to