New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Uses commons-text to escape html for safer output #186

Closed

iccole wants to merge 2 commits into master from escape-output

Contributor

iccole commented Feb 4, 2025

This is what I had tested. I may have missed elsewhere in the request processing where other filter behavior occurs @hcayless


          Uses commons-text to escape html for safer output

455e163

iccole requested a review from hcayless

February 4, 2025 16:47


          This facet value is already scrubbed and used to compose the page

f1c7640

iccole closed this

iccole deleted the escape-output branch

February 4, 2025 19:49

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet