1732793 - AOM calls should retry on proxy failure

Status: NEW

(Toolkit :: Add-ons Manager, enhancement, P3)

Description

[Shane Caraveo (:mixedpuppy)]

a bypassProxy flag has been added in https://phabricator.services.mozilla.com/D126550

ProductUpdateChecker should make use of this and retry on failed proxy requests.

Comment 1

[Rob Wu [:robwu]]

By ProductUpdateChecker, did you mean ProductAddonChecker? It uses ServiceRequest already, but without the bypassProxy flag. The update check and download are both covered by this file.

There is also the AddonUpdateChecker, which also uses ServiceRequest without setting bypassProxy. After the update check, AddonInstall downloads without the beConservative or bypassProxy flags though, so adding the flag to the update check in this case would only be useful if the download logic also gets updated.

Comment 2

[Shane Caraveo (:mixedpuppy)]

ProductAddonChecker