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

Shows more information for wrapped exceptions #2017

Merged

Conversation

bkositza
Copy link
Contributor

@bkositza bkositza commented Jul 2, 2024

Description

Sometimes we have to handle exceptions that are wrapped in other exceptions. Hopefully these wrapper-exceptions give us information about its wrapped exceptions in the message. Unfortunately, some of them does not. So we can try to catch the missing information from the overriden toString()-method that includes this information e.g. com.lowagie.text.ExceptionConverter

Additional Notes

Checklist

  • Code change has been tested and works locally
  • Code was formatted via IntelliJ and follows SonarLint & best practices
  • Patch Tasks: Is local execution of Patch Tasks necessary? If so, please also mark the PR with the tag.

Sometimes we have to handle exceptions that are wrapped in other exceptions. Hopefully these wrapper-exceptions give us information about its wrapped exceptions in the message. Unfortunately, some of them does not. So we can try to catch the missing information from the overriden toString()-method that includes this information e.g. com.lowagie.text.ExceptionConverter

Fixes: SIRI-985
@bkositza bkositza added 🧬 Enhancement Contains new features 🔥 Hotfix PRs that fix a major bug and can be hotfixed labels Jul 2, 2024
@bkositza bkositza self-assigned this Jul 2, 2024
@bkositza bkositza merged commit d7f6782 into develop Jul 2, 2024
5 checks passed
@bkositza bkositza deleted the feature/SIRI-985-show-more-information-wrapped-exceptions branch July 2, 2024 12:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🧬 Enhancement Contains new features 🔥 Hotfix PRs that fix a major bug and can be hotfixed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants