Skip to content

[Firebase AI] Add App Check Limited-Use mode integration tests #15230

New issue

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

Sign up for GitHub

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

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 3 commits into from
Aug 19, 2025
Merged

[Firebase AI] Add App Check Limited-Use mode integration tests #15230

merged 3 commits into from
Aug 19, 2025
+73 −28

Conversation

andrewheard
Copy link
Contributor

  • Added integration tests to verify that calls succeed in limited-use mode
    • Verified manually that the TTL is 5 minutes
  • Disabled the staging and proxy bypass integration test configs to reduce resource utilization

#no-changelog

paulb777
paulb777 approved these changes Aug 19, 2025
View reviewed changes
Copy link
Member

@paulb777 paulb777 left a comment

Choose a reason for hiding this comment

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

lgtm on green

@andrewheard andrewheard marked this pull request as ready for review August 19, 2025 22:44
Base automatically changed from daymxn-fal-limitedusetokens-support to main August 19, 2025 22:50
@andrewheard andrewheard force-pushed the ah/ai-limitedusetokens-integration branch from 756eeb2 to 05ee97a Compare August 19, 2025 22:58
@andrewheard andrewheard added this to the 12.2.0 - M169 milestone Aug 19, 2025
@andrewheard andrewheard merged commit f33b4cb into main Aug 19, 2025
47 checks passed
@andrewheard andrewheard deleted the ah/ai-limitedusetokens-integration branch August 19, 2025 23:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@paulb777 paulb777 paulb777 approved these changes

@ncooke3 ncooke3 ncooke3 approved these changes

Assignees
No one assigned
Labels
api: firebaseai
Projects
None yet
Milestone
12.2.0 - M169
Development

Successfully merging this pull request may close these issues.
3 participants
@andrewheard @paulb777 @ncooke3