Hi Betrand,

Thanks for tackling this and the other issue regarding HtmlResponse.

I am not sure about the new public static escapeHtmlText method, though:

  * Do we really need to add new methods ?
  * Is HtmlResponse the correct location ?
  * Isn't such a method already available elsewhere in Sling ?
  * IMHO we should also deprecate the Sling API HtmlResponse since
    we have a better public mechanism now in the Sling POST Servlet.

Regards
Felix

Am Dienstag, den 24.05.2011, 07:46 +0100 schrieb Bertrand Delacretaz
(JIRA): 
> [ 
> https://issues.apache.org/jira/browse/SLING-2085?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
>  ]
> 
> Bertrand Delacretaz resolved SLING-2085.
> ----------------------------------------
> 
>        Resolution: Fixed
>     Fix Version/s: Engine 2.2.4
> 
> Implemented in revision 1126891
> 
> > RequestHistoryConsolePlugin should escape HTML text
> > ---------------------------------------------------
> >
> >                 Key: SLING-2085
> >                 URL: https://issues.apache.org/jira/browse/SLING-2085
> >             Project: Sling
> >          Issue Type: Bug
> >          Components: Engine
> >    Affects Versions: Engine 2.2.2
> >            Reporter: Bertrand Delacretaz
> >            Priority: Minor
> >             Fix For: Engine 2.2.4
> >
> >
> > The RequestHistoryConsolePlugin should escape the HTML text that it outputs
> 
> --
> This message is automatically generated by JIRA.
> For more information on JIRA, see: http://www.atlassian.com/software/jira


Reply via email to