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

Documentation : MariaDB user creation and privileges grant #75

Closed
teimahu opened this issue Apr 29, 2021 · 2 comments
Closed

Documentation : MariaDB user creation and privileges grant #75

teimahu opened this issue Apr 29, 2021 · 2 comments
Assignees
Labels
done Implemented, merged and tested enhancement Refactoring, improvement or maintenance task

Comments

@teimahu
Copy link

teimahu commented Apr 29, 2021

Hi,

I think there is a mistake with the SQL command for granting rights to the newly created user in file : docs/getting-started/advanced/databases.md

On my setup (MariaDB 10.4.18), I had to change the grant line from:
GRANT ALL PRIVILEGES ON photoprism.* to 'photoprism'@'%';

To:
GRANT ALL PRIVILEGES ON photoprism.* to 'photoprism'@'%' IDENTIFIED BY 'insecure';

And I would end with a:
FLUSH PRIVILEGES;

So that new grant is taken into account.

Thanks,
Mathieu

lastzero added a commit that referenced this issue Apr 30, 2021
See #75

Signed-off-by: Michael Mayer <michael@lastzero.net>
@lastzero
Copy link
Member

Updated our docs. Does this work for you?

CREATE USER 'photoprism'@'%' IDENTIFIED BY 'insecure';
GRANT ALL PRIVILEGES ON photoprism.* to 'photoprism'@'%';
FLUSH PRIVILEGES;

Setting the password again with GRANT ALL should not be needed IMO.

@lastzero lastzero self-assigned this Apr 30, 2021
@lastzero lastzero added enhancement Refactoring, improvement or maintenance task please-test Ready for acceptance test labels Apr 30, 2021
@teimahu
Copy link
Author

teimahu commented Apr 30, 2021

Yes you are right, I found tutorials with and without it yesterday.
But the MariaDB documentation clearly states that setting the password in GRANT ALL is optionnal,
I just tested again today creating a new user on my setup and of course, the "identified by" statement is not mandatory, I surely made a mistake yesterday.
Sorry I should have double checked before creating an issue.

@lastzero lastzero added done Implemented, merged and tested and removed please-test Ready for acceptance test labels May 14, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
done Implemented, merged and tested enhancement Refactoring, improvement or maintenance task
Projects
None yet
Development

No branches or pull requests

2 participants