[bug] Quick fix for handling non 200 status codes when loading specs from URI #1695

hedge-sparrow · 2024-11-25T14:24:24Z

Go http client already handles 3xx responses for us

Description, Motivation and Context

Please include a summary of the change or what problem it solves. Please also include relevant motivation and context.

Fixes: #1694

New and existing tests pass locally with introduced changes.
Tests for the changes have been added (for bug fixes / features)
The commit message(s) are informative and highlight any breaking changes
Any documentation required has been added/updated. For changes to https://troubleshoot.sh/ create a PR here

Go http client already handles 3xx responses for us

hedge-sparrow added 2 commits November 25, 2024 14:19

Quick fix for handling non 200 status codes when loading specs from URI

e92ff2a

Go http client already handles 3xx responses for us

note

20bfc8c

hedge-sparrow added the bug::normal label Nov 25, 2024

hedge-sparrow requested a review from a team as a code owner November 25, 2024 14:24

hedge-sparrow added the type::bug Something isn't working label Nov 25, 2024

banjoh approved these changes Nov 25, 2024

View reviewed changes

hedge-sparrow merged commit ecc92b1 into main Nov 25, 2024
24 checks passed

hedge-sparrow deleted the ash-jones/sc-116285/uri-fields-with-unexpected-content-cause branch November 25, 2024 15:04