Skip to content

bug: bump AdminPortal auth0-js to ^9.12.0 (crypto-js CVE-2023-46233) - #524

Open
devin-ai-integration[bot] wants to merge 1 commit into
masterfrom
devin/1791812144-sfind-cfb2a2f1-crypto-js-cve-2023-46233
Open

devin-ai-integration[bot] wants to merge 1 commit into
masterfrom
devin/1791812144-sfind-cfb2a2f1-crypto-js-cve-2023-46233

Conversation

@devin-ai-integration

@devin-ai-integration devin-ai-integration Bot commented Oct 12, 2026 •

Copy link
Copy Markdown

Summary

Fixes code-scan finding sfind-cfb2a2f1 (Snyk SNYK-JS-CRYPTOJS-6028119 / CVE-2023-46233, high): crypto-js@3.3.0 PBKDF2 defaults to SHA1 with 1 iteration. It reaches AdminPortal through auth0-js@8.12.3 > idtoken-verifier@1.5.1 > crypto-js@3.3.0.

-    "auth0-js": "^8.8.0",
+    "auth0-js": "^9.12.0",

This is the smallest upgrade Snyk lists for this path: auth0-js@9.12.0 > idtoken-verifier@2.2.4 > crypto-js@4.2.0.

Verification

  • ng build (Node 12.22.12, CLI 1.1.2): passes. auth0-js isn't in the bundle graph, so the output matches before and after.
  • ng lint: 3 errors (prefer-const in login, primary-transaction and savings-transaction *.component.spec.ts). They are already on master and not in files this PR touches.
  • ng test: Karma 1.7.1 with headless Chrome fails at startup in this VM with ERROR [karma]: { inspect: [Function: inspect] }, the same on master. Unit tests did not run.
  • Not done: npm install of the new tree and the Snyk before/after re-scan. registry.npmjs.org and api.snyk.io are blocked in this session (access requested), so snyk test errors out. So I have no measured before/after issue count. The finding reports 1 issue on this path (crypto-js@3.3.0 via auth0-js@8.12.3). The fix depends on Snyk's stated upgrade path, which resolves crypto-js@4.2.0 (the fixed version). Please run cd AdminPortal && rm -rf node_modules package-lock.json && npm install && snyk test on Node 12 to confirm.

Devin-Org: engineering

Link to Devin session: https://devin-gtm.devinenterprise.com/sessions/271527ec18194ee0947717900e37c0d8
Open in Devin Desktop: https://devin-gtm.devinenterprise.com/desktop/session/271527ec18194ee0947717900e37c0d8?variant=devin


Devin Review

Status Commit
⚪ Not started —

Run Devin Review

Devin Review (Staging)

@devin-ai-integration

Copy link
Copy Markdown
Author

I'll fix CI failures and address comments from users with write access. I'll skip comments containing "(aside)".

  • Disable automatic comment, CI, and merge conflict monitoring

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔍 Devin Review: 1 flag

Not posted on this PR by your GitHub settings — view it in Devin Review. (Configure)

Devin Review

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.

0 participants