Skip to content

Conversation

draperunner
Copy link
Contributor

Upgrade OWASP sanitizer

This upgrades the OWASP sanitizer library to the latest version. Guava
is removed from this version.
https://github.com/OWASP/java-html-sanitizer/releases/tag/release-20220608.1

The order of noreferrer, nofollow, noopener is apparently random now, so
I had to rewrite those tests to not care about order
(OWASP/java-html-sanitizer#336).

Rename test files to make them run

The files should match the expected name pattern so that are executed
through maven.

The files should match the expected name pattern so that are executed
through maven.
This upgrades the OWASP sanitizer library to the latest version. Guava
is removed from this version.
https://github.com/OWASP/java-html-sanitizer/releases/tag/release-20220608.1

The order of noreferrer, nofollow, noopener is apparently random now, so
I had to rewrite those tests to not care about order
(OWASP/java-html-sanitizer#336).
@draperunner draperunner requested a review from a team April 15, 2024 09:51
Copy link

@isollid isollid left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good :D

@draperunner draperunner merged commit 57d8d89 into main Apr 15, 2024
@draperunner draperunner deleted the upgrade-sanitizer branch April 15, 2024 12:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants