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

ParamConverter should throw UserFriendlyException #27

Closed
slominskir opened this issue Nov 13, 2024 · 0 comments · Fixed by #28
Closed

ParamConverter should throw UserFriendlyException #27

slominskir opened this issue Nov 13, 2024 · 0 comments · Fixed by #28

Comments

@slominskir
Copy link
Member

In order to satisfy CodeQL Exception messages cannot be shared with users unless explicitly overridden, which the UserFriendlyException class does. Therefore the ParamConverter class should throw UserFriendlyExceptions instead of built-in Exceptions such as IllegalArgumentException in order to better support delivering user friendly messages to the user without tripping security scanners.

Related: #23

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

Successfully merging a pull request may close this issue.

1 participant