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

Bench new-site doesn't work with empty passwords, flags error: Unexpected Extra Argument. #49

Closed
ayosafacundo opened this issue Jan 8, 2025 · 2 comments

Comments

@ayosafacundo
Copy link

bench new-site $site_name --db-root-password $sqlpasswrd --admin-password $adminpasswrd

Expected functionality:
It should check if you've inputted a password or left empty and take that into account when setting up a new site.

It executes this command: bench new-site $site_name --db-root-password $sqlpasswrd --admin-password $adminpasswrd
which for empty passwords turns out to be: bench new-site $site_name --db-root-password --admin-password, returning unexpected extra argument error.

@flexcomng
Copy link
Owner

AFAIK, Frappe does not allow empty passwords and MariaDB definitely does not allow empty passwords so you have to set your root/MariaDB and administrator passwords in order to complete an install.

@flexcomng flexcomng closed this as not planned Won't fix, can't repro, duplicate, stale Jan 16, 2025
@ayosafacundo
Copy link
Author

I was able to do it setting a password, but it was weird that the bash script allowed me to not set any password and then delivered an unexpected extra argument error blaming bench. Would be nice for the bash script to actually enforce a password.

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

No branches or pull requests

2 participants