Skip to content

Consecutive exception hides original error #1017

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

Closed
bart-degreed opened this issue Jun 17, 2021 · 0 comments · Fixed by #1021
Closed

Consecutive exception hides original error #1017

bart-degreed opened this issue Jun 17, 2021 · 0 comments · Fixed by #1021
Labels

Comments

@bart-degreed
Copy link
Contributor

For example, see here. The initial DataStoreUpdateException disappears because a new exception is thrown when disposing PlaceholderResourceCollector before the first exception can be caught. This results in a 500 error response plus a log entry at error level. Instead the follow-up exception should be swallowed so that the original DataStoreUpdateException can propagate.

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

Successfully merging a pull request may close this issue.

1 participant