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

Mention Invalid Login error on MINIO_SERVER_URL #1312

Merged
merged 3 commits into from
Sep 23, 2024

Conversation

ecerulm
Copy link
Contributor

@ecerulm ecerulm commented Aug 29, 2024

The MinIO Console currently does not warn the user if the MinIO server is not reachable. MinIO Console will just say "Invalid Login" when the MINIO_SERVER_URL is not reachable which is misleading in the sense that will make think most people that the username/password/access key is incorrect so I think it should be mentioned in the documentation that Invalid Login could be just due to connectivity between console and server.

Related to minio/console#3428 and minio/console#3427

@marktheunissen

@harshavardhana
Copy link
Member

It isn't done in vain, it was an ask by the customer to avoid reporting "login errors" to avoid brute force attacks on the login vector in the name of security by (obscurity)

We used to be more descriptive in the past but it was implemented to make it vague on purpose.

@ecerulm
Copy link
Contributor Author

ecerulm commented Aug 29, 2024

It isn't done in vain, it was an ask by the customer to avoid reporting "login errors" to avoid brute force attacks on the login vector in the name of security by (obscurity)

Yeah, I suspected that was on purpose, but mentioning on the documentation (this PR) I hope it's ok. I guess that the wait for 10 seconds before getting the "invalid login" already gives away (in hindsight) that is a communication problem between MinIO Console and MinIO Server, but mentioning in the documentation will make it easier to find in an internet serarch.

I wrote another issue minio/console#3427 about actually changing the "Invalid Login" for "Server unreachable" in that specific case.

@ravindk89 ravindk89 added tiny Small, bite-sized fixes that require minimal effort external-contributor labels Sep 23, 2024
@ravindk89 ravindk89 merged commit 9171ccc into minio:main Sep 23, 2024
@ravindk89
Copy link
Collaborator

Thanks for your patience @ecerulm and your contribution!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
external-contributor tiny Small, bite-sized fixes that require minimal effort
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants