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

Unable to change root password for database if provided via environment variable ORIENTDB_ROOT_PASSWORD #10273

Open
akshay-mahakalkar opened this issue Jul 22, 2024 · 4 comments
Milestone

Comments

@akshay-mahakalkar
Copy link

OrientDB Version: 3.2.32

Java Version: openjdk 11.0.23

OS: Alpine

Expected behavior

If the value of ORIENTDB_ROOT_PASSWORD is changed in the new password should be consumed.

Actual behavior

When ORIENTDB_ROOT_PASSWORD is changed, database is unable to use new password, instead continues to use old password.
When the OSystem database is deleted then only new password is consumed.

Steps to reproduce

  1. Download and start OrientDB server by running ORIENTDB_ROOT_PASSWORD="password" server.sh command.
  2. Try accessing the studio. Should be accessible.
  3. Change the password and run again. Should not be accessible.

Extra Findings

If password is provided via orientdb-server-config.xml file as provided below
<users> <user name="root" password="Root@1234" resources="*" /> <user name="guest" password="admin" resources="connect,server.listDatabases,server.dblist" /> </users>
Then I am able to login database with both passwords. One provided via environment variable and one provided in xml file at the same time.

@tglman tglman added this to the 3.2.x milestone Jul 22, 2024
@tglman
Copy link
Member

tglman commented Jul 25, 2024

Hi,
Yes I think that password as today is retrieve and stored somewhere else, I do agree this may not be the best behavior, I will see if this can be corrected.

@tglman
Copy link
Member

tglman commented Aug 29, 2024

Hi,

This should be fixed in 3.2.33 let me know if you can verify it.

Regards

@akshay-mahakalkar
Copy link
Author

Hi @tglman,

Thanks for resolving the issue. I verified it is now resolved. If new password set via environment variable it is considering the new password.
Only one thing I found new is, now the database console logs shows the database password.
image
Was this intentional as previously it was "INFO Found ORIENTDB_ROOT_PASSWORD variable, using this value as root's password [OServer]".

@tglman
Copy link
Member

tglman commented Sep 3, 2024

Hi,

Thank you for checking this, yes we should not log the password, that strange, I will double check this and make sure it doesn't happen, will leave this open till I make sure is not logged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

2 participants