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

Unify the way to process errors for xPolicy #2894

Open
shawnh2 opened this issue Mar 12, 2024 · 3 comments · May be fixed by #3302
Open

Unify the way to process errors for xPolicy #2894

shawnh2 opened this issue Mar 12, 2024 · 3 comments · May be fixed by #3302

Comments

@shawnh2
Copy link
Contributor

shawnh2 commented Mar 12, 2024

Description:

nowadays, the BTP and SP use different strategy dealing with errors, the BTP directly return the error from translate methods, but SP joins the errors then return. maybe it's better to unify the way to process all the translated errors.

[optional Relevant Links:]

originally post in #2848 (comment)

@shawnh2
Copy link
Contributor Author

shawnh2 commented Mar 12, 2024

recommended way:

@shawnh2
Copy link
Contributor Author

shawnh2 commented Mar 12, 2024

assign myself, since it's related to #2631

Copy link

This issue has been automatically marked as stale because it has not had activity in the last 30 days.

@github-actions github-actions bot added the stale label Apr 11, 2024
@shawnh2 shawnh2 linked a pull request Apr 29, 2024 that will close this issue
@arkodg arkodg added this to the v1.1.0-rc1 milestone May 23, 2024
@arkodg arkodg removed the stale label May 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment