Skip to content

Test : Add test when raw string has rel="noopener noreferrer" #293

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.

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

Merged
merged 1 commit into from
Jan 15, 2024

Conversation

subbudvk
Copy link
Contributor

@subbudvk subbudvk commented Nov 22, 2023

When the string to be sanitized already has rel="noopener noreferrer" the library appends it again causing it to be added twice. This PR gives a failing test. If it's fine, I'll create a PR for the fix in dev.

@subbudvk subbudvk changed the title Test : Add test for duplicate noopener noreferrer when Original strin… Test : Add test when raw string has rel="noopener noreferrer" Nov 22, 2023
@mikesamuel mikesamuel merged commit 91c5fdc into OWASP:main Jan 15, 2024
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