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
Handle 'Request was throttled' more gracefully in sign
#1647
Comments
That message is actually the response web-ext sign got from the server, and so this is not something we can investigate and/or change on the "web-ext" tool side. In the meantime I'm closing this as invalid (from a "web-ext cli tool" perspective). |
@tomquas sorry for the inconvenience. The rate limits are in place to combat real and ongoing attacks against our systems. I suggest sleeping your script after signing each new version or adding a wait-retry-loop each time you see a throttled response code. |
@diox This could be solved at the web-ext side by detecting the specific error type and then automatically retrying after waiting for the specified amount of time. Is it acceptable from AMO's side of view that we implement it here like that? |
Yes, but with some caveats. You can use the If the |
Thanks diox. I've reopened this issue and marked it as a feature request. The throttling is not a bug (which is why this report was initially closed), but we can do something about it. |
@Rob--W I'm sure that you may already noticed it, but I think that it still worth to point it out here: the actual signing feature is implemented by the sign-addon dependency (github repo: https://github.com/mozilla/sign-addon), and so it may make sense to evaluate if it would be more reasonable to implement the new behavior inside the dependency. |
thx everybody, this is going in the right direction ;) |
Worth noting: under current limits, a single user can submit 3 add-ons per minute and 20 add-ons in an hour (and tbh I consider that generous, we might lower that, or introduce a limit per day as well). Another set of limits is added per IP. So don't expect fast builds if you're integrating that in your CI and try to sign more than 20 add-ons. |
sign
Is this a feature request or a bug?
you choose
What is the current behavior?
we need to sign 20+ extension build variants for different brands and configurations.
during the build, more often than not we do see
and we have to catch up on where things failed. total pain in the neck.
What is the expected or desired behavior?
just sign em...
Version information (for bug reports)
The text was updated successfully, but these errors were encountered: