-
-
Notifications
You must be signed in to change notification settings - Fork 375
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
Fix inefficient regular expressions in code #1147
Comments
Think they can also be seen https://github.com/htmlhint/HTMLHint/security/code-scanning |
Those aren't visible publicly though - only to admins. That's why I linked to LGTM (even though that service is closing down later this year...) |
Are there any active links in 2024 to take a look at them? |
Replacing the capturing group quantifier ( ... )* with a non-capturing group quantifier (?: ... )* to reduce the risk of exponential backtracking. htmlhint#1147
The positive lookahead assertion `(?= ...)` is used to ensure that the check occurs after the first lowercase letter and all subsequent characters have been verified to resolve the issue of reference backtracking htmlhint#1147
REF: https://lgtm.com/projects/g/htmlhint/HTMLHint/?mode=list
There are a number of Inefficient regular expressions and other related issues in the codebase.
PRs are welcome!
The text was updated successfully, but these errors were encountered: