[GitHub] [sling-org-apache-sling-xss] rombert commented on pull request #28: SLING-7231 Move to owasp sanitizer library

2022-09-21 Thread GitBox
rombert commented on PR #28: URL: https://github.com/apache/sling-org-apache-sling-xss/pull/28#issuecomment-1253640668 I think that we should merge it and make any follow-up fixes later, it's quite large as it is and a lot of work has been put in the initial contributions and in the follow

[GitHub] [sling-org-apache-sling-xss] rombert commented on pull request #28: SLING-7231 Move to owasp sanitizer library

2022-09-02 Thread GitBox
rombert commented on PR #28: URL: https://github.com/apache/sling-org-apache-sling-xss/pull/28#issuecomment-1235417417 @kwin - I think this is ready for a re-review, except for https://github.com/apache/sling-org-apache-sling-xss/pull/28#discussion_r915033485 , which I will try to complete

[GitHub] [sling-org-apache-sling-xss] rombert commented on pull request #28: SLING-7231 Move to owasp sanitizer library

2022-07-06 Thread GitBox
rombert commented on PR #28: URL: https://github.com/apache/sling-org-apache-sling-xss/pull/28#issuecomment-1176471732 I worked with @nonanalou of preparing this PR, so I'll collaborate on getting this ready to be merged and providing some context where necessary. -- This is an automated