Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix: Fix redaction of credentials in Firestore settings #1989

Merged
merged 4 commits into from
Jan 25, 2024

Conversation

MarkDuckworth
Copy link
Contributor

@MarkDuckworth MarkDuckworth commented Jan 23, 2024

fix: Fix redaction of credentials in Firestore settings

Rollback removal of toJSON for Firestore settings and corrected the implementation. Added tests.

@MarkDuckworth MarkDuckworth requested review from a team as code owners January 23, 2024 16:11
@product-auto-label product-auto-label bot added size: s Pull request size is small. api: firestore Issues related to the googleapis/nodejs-firestore API. labels Jan 23, 2024
Copy link
Contributor

@milaGGL milaGGL left a comment

Choose a reason for hiding this comment

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

LGTM. Thank you Mark for looking into this.

@MarkDuckworth MarkDuckworth changed the title Rollback removal of toJSON for Firestore settings and corrected the implementation fix: Fix redaction of credentials in Firestore settings Jan 25, 2024
@MarkDuckworth MarkDuckworth merged commit 98e668b into main Jan 25, 2024
@MarkDuckworth MarkDuckworth deleted the markduckworth/fix-firestore-settings-tojson branch January 25, 2024 16:46
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
api: firestore Issues related to the googleapis/nodejs-firestore API. size: s Pull request size is small.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants