Skip to content

BUG: functions:secrets:prune was not affecting secrets created after apphosting launch#10840

Open
Berlioz wants to merge 2 commits into
mainfrom
vsfan_secret_prune
Open

BUG: functions:secrets:prune was not affecting secrets created after apphosting launch#10840
Berlioz wants to merge 2 commits into
mainfrom
vsfan_secret_prune

Conversation

@Berlioz

@Berlioz Berlioz commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

see:

* This used to be firebase-managed: true, but was later changed to firebase-managed: functions to

@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 pruneSecrets function in src/functions/secrets.ts to retrieve secrets labeled with either true or functions when collecting Firebase-managed secrets. There are no review comments, and I have no feedback to provide.

@shettyvarun268 shettyvarun268 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.

Thanks for fixing this!
LGTM

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.

3 participants