Skip to content

Draft: Error out unauthorized

David Smith Admin requested to merge error_out_unauthorized into master

Overview

Found that bad http requests were not logging error info about http failures, so added some changes to log the failure specifics

NOTE: Based of changes from !500 (merged)

  • Updated Api methods to raise exeptions (HTTPError) for status
  • Updated stable_retry decorator to end retries if HTTP error is found
  • Updated unit tests to fake successful response codes

Tasks

  • Update docs if needed (workflow, runbooks, README, migration metrics, etc.)

Acceptance Criteria

  • Existing UTs pass locally
Edited by Petar Prokić

Merge request reports