Skip to content
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.

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

Validate credential account #345

Merged
merged 2 commits into from
Dec 22, 2023
Merged

Validate credential account #345

merged 2 commits into from
Dec 22, 2023

Conversation

adrianha
Copy link
Contributor

@adrianha adrianha commented Dec 22, 2023

Copy link

google-cla bot commented Dec 22, 2023

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@adrianha
Copy link
Contributor Author

@tonybaroneee probably can help to review this? 🙏

@kaibolay kaibolay merged commit ca2ef36 into firebase:master Dec 22, 2023
3 checks passed
@adrianha
Copy link
Contributor Author

hi @kaibolay, could you help to publish new version that includes this diff?

@kaibolay
Copy link
Collaborator

@adrianha I have some additional changes queued up for a new release. Could you please test #347 and let me know if it works for you?

@adrianha
Copy link
Contributor Author

Understood, will try it next year @kaibolay 🎉

@adrianha
Copy link
Contributor Author

adrianha commented Jan 1, 2024

hi @kaibolay,

looks fine on my end 👍

[13:18:14]: 🔐 Authenticating with Application Default Credentials
[13:18:14]: ⌛ Uploading the APK.
[13:18:18]: 🕵️ Validating upload.
[13:18:26]: ✅ Uploaded APK successfully and created release 1.0.0 (1).
[13:18:26]: 📜 Setting release notes.
[13:18:28]: 📦 Distributing release.
[13:18:29]: 🎉 App Distribution upload finished successfully. Setting Actions.lane_context[SharedValues::FIREBASE_APP_DISTRO_RELEASE] to the uploaded release.

@kaibolay
Copy link
Collaborator

kaibolay commented Jan 2, 2024

@adrianha Thank you. It's released now.

@adrianha
Copy link
Contributor Author

adrianha commented Jan 3, 2024

thank you @kaibolay!

@adrianha
Copy link
Contributor Author

adrianha commented Jan 3, 2024

hi @kaibolay, just realized that fastlane-plugin-firebase_app_distribution 0.8.0 depends on google-apis-firebaseappdistribution_v1 and google-apis-firebaseappdistribution_v1alpha, is it expected?

I got Service account impersonation failed with status 403 error after updating to 0.8.0 then it's resolved after updating other dependencies:

  • aws-partitions: 1.872.0 –> 1.876.0
  • aws-sdk-s3: 1.141.0 –> 1.142.0
  • fastlane: 2.217.0 –> 2.218.0

@kaibolay
Copy link
Collaborator

kaibolay commented Jan 3, 2024

hi @kaibolay, just realized that fastlane-plugin-firebase_app_distribution 0.8.0 depends on google-apis-firebaseappdistribution_v1 and google-apis-firebaseappdistribution_v1alpha, is it expected?

Yes. get_udids uses a v1alpha API.

I got Service account impersonation failed with status 403 error after updating to 0.8.0 then it's resolved after updating other dependencies:

  • aws-partitions: 1.872.0 –> 1.876.0
  • aws-sdk-s3: 1.141.0 –> 1.142.0
  • fastlane: 2.217.0 –> 2.218.0

That's really odd. None of these are direct (or indirect) dependencies of fastlane-plugin-firebase_app_distribution.

@adrianha
Copy link
Contributor Author

adrianha commented Jan 4, 2024

I see, maybe it's intermittent then 🤔

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