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

js/weak-cryptographic-algorithm/BrokenCryptoAlgorithm got 25-30x slower #18604

Open
maschwenk opened this issue Jan 28, 2025 · 2 comments
Open
Labels
question Further information is requested

Comments

@maschwenk
Copy link

Description of the issue

Similar to

this rule appears to run 20-30x slower on the same exact repo with the same hardware

[106/106 eval 30m34s] Evaluation done; writing results to codeql/javascript-queries/Security/CWE-327/BrokenCryptoAlgorithm.bqrs.

unsure root cause is the same as the other issue, but disabling rule rule like

query-filters:
  - exclude:
      id: js/weak-cryptographic-algorithm

is a workaround

@asgerf
Copy link
Contributor

asgerf commented Jan 28, 2025

Hi @maschwenk, thanks for the report.

If you've seen this on a public repository could you share a link to the repository?

@maschwenk
Copy link
Author

maschwenk commented Jan 28, 2025

Sadly not public, and unsure of how to give a repro. Failing on a very large 20k+ LoC typescript app.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants