[jira] [Reopened] (OFBIZ-6769) The renderContentAsText method should configure text sanitizer by "sanitizer.permissive.policy" in owasp.properties

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

[jira] [Reopened] (OFBIZ-6769) The renderContentAsText method should configure text sanitizer by "sanitizer.permissive.policy" in owasp.properties

Nicolas Malin (Jira)

     [ https://issues.apache.org/jira/browse/OFBIZ-6769?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Jacques Le Roux reopened OFBIZ-6769:
------------------------------------

Mmm wait, It seems I was not waken enough this morning, I re-thought about that...

This is not how sanitizer.permissive.policy is supposed to work. You are supposed to use sanitizer.permissive.policy=true in owasp.properties and provide your own PERMISSIVE_POLICY in HtmlEncoder.sanitize() (in UtilCodec.java). Not to use sanitizer.permissive.policy=true to bypass HtmlEncoder.sanitize()

See https://owasp-java-html-sanitizer.googlecode.com/svn/trunk/distrib/javadoc/org/owasp/html/PolicyFactory.html and OFBIZ-6669 for details

So I will revert your change and close as invalid.

> The renderContentAsText method should configure text sanitizer by "sanitizer.permissive.policy" in owasp.properties
> --------------------------------------------------------------------------------------------------------------------
>
>                 Key: OFBIZ-6769
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-6769
>             Project: OFBiz
>          Issue Type: Sub-task
>            Reporter: Supachai Chaima-ngua
>            Assignee: Jacques Le Roux
>            Priority: Minor
>              Labels: content
>             Fix For: Upcoming Branch
>
>         Attachments: ofbiz-renderContentAsText.diff
>
>
> The renderContentAsText method should configure text sanitizer by "sanitizer.permissive.policy" in owasp.properties. If electronic text contains javascript, the renderContentAsText method will remove some content.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)