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

Wrap remote endpoints with error handling #44248

Closed
ilyasfoo opened this issue Feb 1, 2024 · 0 comments · Fixed by #44641 or #44775
Closed

Wrap remote endpoints with error handling #44248

ilyasfoo opened this issue Feb 1, 2024 · 0 comments · Fixed by #44641 or #44775
Assignees
Labels
team: Ghidorah type: task The issue is an internally driven task (e.g. from another A8c team).

Comments

@ilyasfoo
Copy link
Contributor

ilyasfoo commented Feb 1, 2024

Issue description

Project thread: pdibGW-2Mv-p2

Follow-up from #44037

Wrap remote endpoint rule evaluators with appropriate error handlers with logging for development environments.

Acceptance criteria

  1. All processing data from remote API endpoints needs to be wrapped in broad error-catching check.
  2. In production environment, do nothing with the error.
  3. In development environment, either re-throw the error or add logging.
@ilyasfoo ilyasfoo added the type: task The issue is an internally driven task (e.g. from another A8c team). label Feb 1, 2024
@ilyasfoo ilyasfoo self-assigned this Feb 1, 2024
@ilyasfoo ilyasfoo reopened this Feb 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
team: Ghidorah type: task The issue is an internally driven task (e.g. from another A8c team).
Projects
None yet
2 participants