Skip to content

chore: update Functions backend dependencies and upgrade to V2 API - #1499

Open
inlined wants to merge 3 commits into
mainfrom
security-audit/functions-backend
Open

chore: update Functions backend dependencies and upgrade to V2 API#1499
inlined wants to merge 3 commits into
mainfrom
security-audit/functions-backend

Conversation

@inlined

@inlined inlined commented Jul 30, 2026

Copy link
Copy Markdown
Member

Security Audit & Remediation: functions-backend

A. Previous CVEs

B. Changes Made

  • Updated firebase-admin to ^14.2.0 and firebase-functions to ^7.3.2.
  • Upgraded Cloud Functions definitions in the testapp backend and sample backend to use the Firebase Functions V2 API.
  • Added overrides in package.json for uuid (^11.1.1) and brace-expansion (^5.0.8) to resolve transitive CVEs.
  • Generated and added missing package-lock.json files.
  • Added node_modules/ to root .gitignore.
  • Removes a production dependency and is safe because it's running the full software suite that would run in docker in Cloud Functions.

C. Remaining CVEs

  • None

D. Introduced CVEs

  • None

E. Testing Strategy

  • Ran local syntax check (node -c index.js) successfully on both files.
  • Ran npm audit in both package directories showing 0 vulnerabilities.

Release Notes

relnote: "chore: update functions backend dependencies to resolve security vulnerabilities and upgrade callable functions to V2 API"

@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 the Firebase Cloud Functions to the v2 API, upgrades dependencies, and targets Node.js 22. Feedback is provided regarding potential runtime TypeErrors when accessing properties on request.data if the client sends no payload; using optional chaining is recommended to handle these cases safely.

Comment thread functions/testapp/Assets/Firebase/Sample/Functions/.functions/functions/index.js Outdated
Comment thread functions/testapp/functions/functions/index.js Outdated
@inlined
inlined force-pushed the security-audit/functions-backend branch from 3136954 to f7bf8da Compare July 30, 2026 00:38
@inlined
inlined requested a review from AustinBenoit July 30, 2026 01:11
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