Skip to content

[AI] Enable Firebase App Check with limited-use tokens - #2849

Merged
rlazo merged 5 commits into
masterfrom
rl.appcheck.limitedtoken
Oct 8, 2026
Merged

rlazo merged 5 commits into
masterfrom
rl.appcheck.limitedtoken

Conversation

@rlazo

@rlazo rlazo commented Oct 8, 2026

Copy link
Copy Markdown
Contributor
  • Add firebase-appcheck-debug dependency to build config and version catalog.
  • Initialize DebugAppCheckProviderFactory in MainActivity.
  • Configure Firebase.ai calls across ViewModels to use limited-use App Check tokens.

- Add `firebase-appcheck-debug` dependency to build config and version
catalog.
- Initialize `DebugAppCheckProviderFactory` in `MainActivity`.
- Configure `Firebase.ai` calls across ViewModels to use limited-use
App Check tokens.
@rlazo
rlazo requested review from VinayGuthal and thatfiredev and removed request for VinayGuthal October 8, 2026 17:19

@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 integrates Firebase App Check into the quickstart application, adding the debug App Check dependency and configuring various ViewModels to use limited-use App Check tokens. Feedback on the changes highlights a critical issue in MainActivity.kt, where DebugAppCheckProviderFactory is installed unconditionally. This would break Firebase AI calls in production builds, so it is recommended to wrap this initialization in a BuildConfig.DEBUG check.

Comment thread firebase-ai/app/src/main/java/com/google/firebase/quickstart/ai/MainActivity.kt Outdated
rlazo and others added 3 commits October 8, 2026 13:21
…i/MainActivity.kt

Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
@rlazo
rlazo merged commit f8032fa into master Oct 8, 2026
18 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