Skip to content

chore(firestore): update googleapis_auth dependency to 2.3.3#301

Open
demolaf wants to merge 2 commits into
mainfrom
firestore-googleapis-auth
Open

chore(firestore): update googleapis_auth dependency to 2.3.3#301
demolaf wants to merge 2 commits into
mainfrom
firestore-googleapis-auth

Conversation

@demolaf

@demolaf demolaf commented Jul 9, 2026

Copy link
Copy Markdown
Member

No description provided.

@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 googleapis_auth dependency in pubspec.yaml from ^2.3.0 to ^2.3.3 to resolve auth/insufficient-permission errors when using Application Default Credentials without a quota project set. The CHANGELOG.md has been updated accordingly. There are no review comments, so I have no feedback to provide.

@github-actions

github-actions Bot commented Jul 9, 2026

Copy link
Copy Markdown

Coverage Report

✅ Coverage 75.48% meets 40% threshold

Total Coverage: 75.48%
Lines Covered: 5208/6900

Package Breakdown

Package Coverage
google_cloud_firestore 78.68%
firebase_admin_sdk 72.66%

Minimum threshold: 40%

@demolaf demolaf requested a review from kevmoo July 9, 2026 17:57
- Fixed `Firestore.getAll()` retrying transient errors indefinitely; it now retries a bounded number of times before surfacing the error.
- Updated `Transaction.delete` and `Transaction.update` type constraints to accept `DocumentReference<Object?>`. (thanks to @Levin-Me)
- Made `Timestamp` encodable by adding `toJson` method. (thanks to @OutdatedGuy)
- Update dependency `googleapis_auth: ^2.3.3` to fix `auth/insufficient-permission` errors with Application Default Credentials that have no quota project set.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Is there an open issue on this?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

It's now closed #293, should I open a new one referencing this?

@kevmoo

kevmoo commented Jul 10, 2026 via email

Copy link
Copy Markdown
Collaborator

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