Skip to content

Security Audit & Remediation: GitHub Actions#3192

Open
inlined wants to merge 2 commits into
firebase:mainfrom
inlined:security-audit/github-actions
Open

Security Audit & Remediation: GitHub Actions#3192
inlined wants to merge 2 commits into
firebase:mainfrom
inlined:security-audit/github-actions

Conversation

@inlined

@inlined inlined commented Jun 23, 2026

Copy link
Copy Markdown
Member

Security Audit & Remediation: GitHub Actions

A. Previous CVEs

B. Changes Made

  • Lockfile & Bundle Security Updates: form-data and undici are transitive dependencies required by mailgun.js (^13.1.0). The existing version range required by mailgun.js already allows patched versions, so package.json did not require modifications. Running npm audit fix updated the locked transitive resolutions in .github/actions/send-email/package-lock.json.
  • Rebuilt Action Bundle: Ran npm run pack (ncc build) to recompile the patched transitive dependencies into the bundled runtime file .github/actions/send-email/dist/index.js executed by GitHub Actions.

C. Remaining CVEs

  • None within .github unit scope (npm audit in .github/actions/send-email reports 0 vulnerabilities).

D. Introduced CVEs

  • None

E. Testing Strategy

  • Verified zero vulnerabilities reported post-fix (npm audit).
  • Verified zero workflow syntax regressions and confirmed existing lint checks pass.

@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 updates several dependency versions in the .github/actions/send-email/package-lock.json file, including upgrading form-data to 4.0.6, hasown to 2.0.4, and undici to 6.27.0. No review comments were provided, and there is no feedback to address.

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