-
Notifications
You must be signed in to change notification settings - Fork 34
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
Retry on http status unauthorized #152
Conversation
@@ -1,18 +1,36 @@ | |||
module github.com/bitrise-steplib/steps-google-play-deploy | |||
|
|||
go 1.16 | |||
go 1.20 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think I have asked this question in someone else's PR already but I do not remember the answer and could not find which PR it was.
The default go on our stacks is 1.18 and here we are defining a more recent version. The tests pass and everything looks fine so it does not cause a problem right but technically this value should point the minimum version the app is compatible I think. So just want to double check that this is fine here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It does not cause issues in this case, it will still work with 1.18. However in and after go 1.21, go runtime takes extra steps to use the defined go version, so will have to be more careful.
Checklist
step.yml
andREADME.md
is updated with the changes (if needed)Version
Requires a PATCH version update
Context
Similar workaround to fastlane/fastlane#21507 (comment).
Retry HTTP calls when 401 is (randomly) returned by Google, that causes customer impact.
Retry working: https://app.bitrise.io/build/065421cb-2e6f-4f0d-acd8-44035136eb07
Changes
Investigation details
Decisions