Skip to content
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

False positive (javascript): "Invalid regular expression flag" for v flag #18721

Open
nicolo-ribaudo opened this issue Feb 9, 2025 · 3 comments

Comments

@nicolo-ribaudo
Copy link

Description of the false positive

v is a valid regular expression flag

Code samples or links to source code

let x = /abc/v;

https://github.com/mozilla/pdf.js/blob/d88035525c69b72ecec2f5ea21b591d2d541a2b5/web/autolinker.js#L112

URL to the alert on GitHub code scanning (optional)

@jketema
Copy link
Contributor

jketema commented Feb 10, 2025

Hi @nicolo-ribaudo

Thank you for this false positive report. Resolving this issue is not a current product priority, but we acknowledge the report and will track it internally for future consideration, or if we observe repeated instances of the same problem.

@nicolo-ribaudo
Copy link
Author

Is the JavaScript parser open source? It would be a trivial pull request for me.

@jketema
Copy link
Contributor

jketema commented Feb 10, 2025

Looks like you already found the parser.

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

Successfully merging a pull request may close this issue.

2 participants