Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Create meaningful error messages by encapsulating backend server errors for Ratify Users #1681

Open
yizha1 opened this issue Aug 2, 2024 · 2 comments
Assignees

Comments

@yizha1
Copy link
Collaborator

yizha1 commented Aug 2, 2024

This issue was created to track the following comment:

          Passing through errors is not a user-friendly practice. We need to catch the error from ACR and wrap it into something meaningful\ for the user of Rarify. The user of Ratify may have no knowledge of where the artifact is and even not know what artifact descriptor is. We need to provide meaningful message for the user of Ratyify and not the user of ACR.

Originally posted by @toddysm in #1662 (comment)

@Horiodino
Copy link

Horiodino commented Jan 8, 2025

Hey @yizha1 , I'd be happy to work on this if no one else is currently handling it. If there's anything else I can do, or if you have any documentation or references that could help resolve the issue, let me know.

@yizha1
Copy link
Collaborator Author

yizha1 commented Jan 9, 2025

@Horiodino Great. I will assign this issue to you. You can check the link in the description. The problem is that the error was just passed through, instead, it should be captured by Ratify and create a new meaningful error message. /cc @susanshi @binbin-li for help in case you need support on the source code.

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

No branches or pull requests

2 participants