-
Notifications
You must be signed in to change notification settings - Fork 19
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
onlyAllowTags doesn't support case insensitive tags #190
Comments
bbcode like an a html is ignoring case of tags, so i think the parser and |
Thanks! Pretty excited for this fix. |
JiLiZART
added a commit
that referenced
this issue
Sep 28, 2023
Fixed in 3.0.2 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The allowed tags settings doesn't have an option for checking for case sensitivity.
One option to fix this is to add a .toLowerCase() to the isAllowedTag() function in the parser
toTagNode() already contains a .toLowerCase() statement
so all tags get converted to lowercase anyway.
Could we either add .toLowerCase() to the isAllowedTag function, or, if we want block incorrectly cased tags, can we add an option that when flagged allows for case insensitive tags?
The text was updated successfully, but these errors were encountered: