Skip to content

Security Audit & Remediation: eventarc#3190

Closed
inlined wants to merge 2 commits into
firebase:mainfrom
inlined:security-audit/eventarc
Closed

Security Audit & Remediation: eventarc#3190
inlined wants to merge 2 commits into
firebase:mainfrom
inlined:security-audit/eventarc

Conversation

@inlined

@inlined inlined commented Jun 23, 2026

Copy link
Copy Markdown
Member

Security Audit & Remediation: eventarc

A. Previous CVEs

B. Changes Made

  • Removed unmaintained trivial wrapper uuid and refactored call sites in src/eventarc/eventarc-utils.ts to use native Node.js runtime stdlib crypto.randomUUID()

C. Remaining CVEs

  • None within eventarc unit scope. (Transitive dev/optional CVEs in validator, tough-cookie, teeny-request, tar, vinyl-fs, z-schema affect root dev/optional dependencies with no impact on eventarc).

D. Introduced CVEs

  • None

E. Testing Strategy

  • Ran existing unit test suite (npx mocha ... --grep "eventarc-utils") - 100% passing.
  • Executed automated linter (npm run lint:src).

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review

This pull request replaces the external uuid library dependency with Node's built-in crypto module, updating the CloudEvent ID generation to use crypto.randomUUID(). There are no review comments, and I have no feedback to provide.

@inlined

inlined commented Jun 23, 2026

Copy link
Copy Markdown
Member Author

Closing because origin/main already contains this remediation (randomUUID from node:crypto).

@inlined inlined closed this Jun 23, 2026
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.

1 participant