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

Add support for non-root MySQL databases #195

Closed
wants to merge 2 commits into from
Closed

Add support for non-root MySQL databases #195

wants to merge 2 commits into from

Conversation

Grendel7
Copy link

Fix for issue #194

# Default MariaDB root user has empty password and can only connect from localhost.
'MYSQL_ROOT_PASSWD': get_conf('DB_ROOT_PASSWD', ''),
'USE_EXISTING_DB': get_conf('USE_EXISTING_DB', '0'),
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should make similar checks as other boolean flags, e.g. comparing to 'true', '1', 'yes'.
So this becomes like
'USE_EXISTING_DB': get_conf('USE_EXISTING_DB', '').lower() in ('true', '1', 'yes')

@oexar
Copy link

oexar commented Feb 11, 2020

also the database port is hardcoded

It would be great to be able to change it with something like :

db_port = get_conf('DB_PORT '3306')

@Neamar
Copy link

Neamar commented May 21, 2020

Hey there! So if I understand correctly:

If so, I'm probably going to try setting it up :)

@mlazzarotto
Copy link

Bumping the thread, it's 2023 guys...

@freeplant
Copy link
Member

The latest version already support using external database: https://manual.seafile.com/docker/deploy_seafile_with_docker/#use-an-existing-mysql-server

And DB_ROOT_PASSWD is needed during installation. Later, after Seafile is installed, the user seafile will be used to connect to the mysql-server (in conf/seafile.conf). You can remove the DB_ROOT_PASSWD.

@freeplant freeplant closed this Mar 22, 2024
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 this pull request may close these issues.

6 participants