Repository navigation
update auth deployment to use any existing app - #11273
Conversation
There was a problem hiding this comment.
Code Review
This pull request updates the Auth deployment process to search for and use existing Firebase apps across any platform (including iOS and Android) before creating a new 'Default Web App'. Feedback on these changes suggests defensively filtering the platform passed to the provisioning API to ensure only valid deployment platforms (IOS, ANDROID, or WEB) are used, and prioritizing other existing Web apps over mobile apps when the 'Default Web App' is not found.
joehan
left a comment
There was a problem hiding this comment.
LGTM, but definitely double check this for ios to make sure it works across paltforms. I know android and web are used in AI studio with this endpoint, but idk if ios is used anywhere
| - Fixed a race condition in `SourceTokenScraper` that could cause concurrent function deployments to hang when refreshing an expired source token (#6989). | ||
| - Fixed an issue where a Function Kit and a functions codebase sharing the same source directory failed `firebase.json` validation. (#11253) | ||
| - Fixed an issue where Cloud Functions creation did not retry on HTTP 403 service account propagation errors. (#11251) | ||
| - Updated Auth deployment to use existing apps from any platform before creating a Default Web App. (#11250) |
There was a problem hiding this comment.
| - Updated Auth deployment to use existing apps from any platform before creating a Default Web App. (#11250) | |
| - Updated Auth deployment to use existing apps from any platform before creating a default web app. (#11250) |
joehan
left a comment
There was a problem hiding this comment.
LGTM, but definitely double check this for ios to make sure it works across paltforms. I know android and web are used in AI studio with this endpoint, but idk if ios is used anywhere
|
Thanks for the review! Confirmed that deployments with only an ios app available on the project works |
Description
Fixes #11250
Scenarios Tested
Test on a project with only an Android app - ok
Test on a project with only an iOS app - ok
Test on a project with only a Web app - ok
Test on a project with no app - ok
Sample Commands
firebase deploy --only auth