Skip to content

Conversation

@alukach
Copy link
Contributor

@alukach alukach commented May 15, 2025

Related Issue(s): #

Proposed Changes:

PR Checklist:

  • This PR is made against the dev branch (all proposed changes except releases should be against dev, not main).
  • This PR has no breaking changes.
  • This PR contains only one commit.
  • I have updated or added new tests to cover the changes in this PR.
  • All tests are passing.
  • This PR affects the Source Cooperative Frontend & API,
    and I have opened issue/PR #XXX to track the change.

venkat-ecov-dev and others added 10 commits March 7, 2025 16:33
Co-authored-by: source-coop-release[bot] <187876225+source-coop-release[bot]@users.noreply.github.com>
* Working first pass  of custom error handling

* Add logging, more refinement

* Rework to make use of From<Box<dyn APIError>>; log error
* Reorg errors

* Finalize flattening & error handling

* Continue to flatten

* Make use of api handler util
* Add more clarity to cloud provider errors

* More specific type
* Simplified error handling for missing primary mirror.
* Replaced fetch_repository method with direct API call in is_authorized.
* Improved clarity in API key processing and response handling.
@venkat-ecov-dev venkat-ecov-dev merged commit 73858d6 into main May 15, 2025
1 check passed
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.

3 participants