-
Notifications
You must be signed in to change notification settings - Fork 899
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
Messaging server prompts should not accept empty password #23031
Closed
miyamotoh opened this issue
May 9, 2024
· 2 comments
· Fixed by ManageIQ/manageiq-appliance_console#251
Closed
Messaging server prompts should not accept empty password #23031
miyamotoh opened this issue
May 9, 2024
· 2 comments
· Fixed by ManageIQ/manageiq-appliance_console#251
Labels
Comments
@agrare there is a regex (and description) for prompts. That can probably get this. |
miyamotoh
changed the title
Messaging server prompts should not allow empty password
Messaging server prompts should not accept empty password
May 10, 2024
1 task
@kbrock we do this double-prompt for a new password and check that the password isn't empty in the internal_database_configuration class so I moved that into a common place, would love your feedback if you get a chance |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
ManageIQ started requiring Messaging/Kafka in recent releases. In
appliance_console
>4. Configure Application
, you can configure it as part of initial setup by answering a series of questions/prompts. For Messaging part, it'd ask;It is easy to just hit ENTER 3 times for the Q2-4, though "empty password" would result in below error. It should not move onto Q5, if password entered was empty.
Similarly, I've heard Kafka doesn't like
localhost
as hostname, and thus for Q2, if something likelocalhost
or maybe what'd resolve to127.0.0.1
is entered, it should perhaps be rejected as well.The text was updated successfully, but these errors were encountered: