common-workflow-language / common-workflow-language/schema_salad

need test for "check_exists does not follow redirects"

Offen
#451 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
Vorherrschende Sprache
Python
Sterne
86
Forks
64
Ø Merge
19 Std. 24 Min.
Gemergte PRs (30 T.)
19

Beschreibung

schema_salad.Fetcher.check_exists does a head() check on URLs. It turns out that the Python requests library does not follow redirects by default on HEAD requests. As a result, blanket redirects such as from http->https will be considered "success" even if it would be a 404 after the redirect.

The fix is to add the allow_redirects flag.

Beitragsleitfaden

Beitragsleitfaden öffnen

Bewertung

Dieses Issue wurde noch nicht bewertet.

Neue Issues direkt in Ihr Postfach

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