no retries on HTTP 5xx errors

Offen Anfängerfreundlich
#3,044 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Maintainer antworten meist innerhalb von 1 Tag

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit
2/5
Geschätzter Aufwand
1-3 Stunden
Anfängerfreundlichkeit
67/100
Issue-Typ
Feature
Klarheit
Größtenteils klar
Aktivitätsstatus
Aktiv
Tech-Stack
python
Bereich
networking

Rechercherichtung

Finde Urllib3Fetcher und untersuche, wie es den PoolManager und die urllib3-Wiederholungsversuche konfiguriert; prüfe die vorhandenen Tests für den Fetcher auf das Wiederholungsverhalten. Füge Wiederholungsversuche für HTTP-Antworten 502, 503 und 504 hinzu und überprüfe anschließend, ob diese Fälle durch die Tests abgedeckt sind.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

Urllib3Fetcher ends up using a PoolManager that uses the default Retry(total=3) which has status_forcelist=set(): This means 502/503/504 responses without a Retry-After header are not retried.

I think this is overly cautious: I think we can retry those.

In practice this would make sigstore-python a little more reliable as sigstores TUF repo runs on Google Cloud GCS which does not include Retry-After on 504s but often works on a retry if a 504 happens

Vorherrschende Sprache
Python
Sterne
1.7k
Forks
306
Ø Merge
4 T. 40 Min.
Gemergte PRs (30 T.)
12

Entwicklungsumgebung

Erste Schritte

  1. Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
  3. Forken Sie das Repository und arbeiten Sie in einem Branch.
  4. Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus theupdateframework/python-tuf

Alle Issues in theupdateframework/python-tuf

Ähnliche Issues

Weitere Issues zu Python

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.