[Bug]: 502 error on manual approval for larger files
What happened?
This was intermittent and checked it for -stage before Fastly. But now I can reproduce it at all attempts, it's the manual approval of a larger file (>40MB)
What did you expect to happen?
Version is approved , but the error is displayed instead of the reviewer page.
Is there an existing issue for this?
- [X] I have searched the existing issues
┆Issue is synchronized with this Jira Task
Shouldn't be tied to Fastly, since reviewers domain is not behind a CDN. That's a synchronous request which can take a while for large files (need to wait for autograph, grabbing the signed content, move it on storage), it certainly shouldn't fail though...
@ioanarusiczki could you try with more files on dev & stage and report back ? With the URLs that failed please.
what throws the error -> in all cases add-on is approved but the 502 error is present
LastPass dev https://reviewers.addons-dev.allizom.org/en-US/reviewers/review/633559 stage https://reviewers.addons-dev.allizom.org/en-US/reviewers/review/633559
Yomikiri https://reviewers.addons-dev.allizom.org/en-US/reviewers/review/633560 https://reviewers.addons.allizom.org/en-US/reviewers/review/1031720
Emoji dev https://reviewers.addons-dev.allizom.org/en-US/reviewers/review/633563
stage dmitlichess addon https://reviewers.addons.allizom.org/en-US/reviewers/review/1031722
70mb extension (large image files) https://reviewers.addons-dev.allizom.org/en-US/reviewers/review/633564
what did not throw the error
https://reviewers.addons-dev.allizom.org/en-US/reviewers/review-listed/633561 15mb, Dashlane https://reviewers.addons-dev.allizom.org/en-US/reviewers/review-listed/633562 phantom 11 mb (lots of js files) https://reviewers.addons.allizom.org/en-US/reviewers/review-listed/1031721 50 mb file (with large images)
@wagnerand have reviewers seen this in production ?
I forwarded the question to reviewers.
I've seen the error happen on production as well