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
influxdb_user: Fixed unhandled exception on invalid login credentials #50134
Conversation
f64233b
to
498fbc8
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for the fix, looks good
There are no negative tests at all for this module. I would love to see some negative tests - and volunteer to do this during christmas holidays although the error message is pretty descriptive already. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
shipit
I would be okay, to merge now and would appreciate your PR for extended test use cases. @gundalow ? |
SUMMARY
Fixes #50131
Catches invalid authentication errors when searching for users and exits properly.
ISSUE TYPE
COMPONENT NAME
influxdb_user