Skip to content

Conversation

@krowvin
Copy link
Collaborator

@krowvin krowvin commented Dec 5, 2025

When you query CDA with an invalid API key/office you will get a 401.

Also updated blob tests

If it is a GET request that does not require auth you can remove the API key,

For cwms-cli that looks like --api-key= with nothing after it to unset it

Wanted to add this so it is more obvious the API key or office is invalid.

If you call one office with a key intended for another office, you will also see the 401.

  • Made it so the HTTP errors always return the status code and status phrase.
  • Added 401 hint
  • Moved the raw error above the hint so the hint is more obvious
  • Added escape characters to help breakup the console.

@krowvin krowvin requested a review from Enovotny December 5, 2025 02:19
@krowvin krowvin changed the title Http Error Hint Additions Http Error Hint Additions + Blob Test Fixes Dec 5, 2025
@krowvin
Copy link
Collaborator Author

krowvin commented Dec 5, 2025

Integration test possibly failing due to needing #245 ?

Fixed

@sonarqubecloud
Copy link

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants