Skip to content

Security Audit & Remediation: GitHub Actions#3192

Merged
lahirumaramba merged 3 commits into
firebase:mainfrom
inlined:security-audit/github-actions
Jun 25, 2026
Merged

Security Audit & Remediation: GitHub Actions#3192
lahirumaramba merged 3 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.

@lahirumaramba lahirumaramba left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thank you! LGTM

@lahirumaramba lahirumaramba merged commit c550a22 into firebase:main Jun 25, 2026
17 checks passed
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.

2 participants