-
-
Notifications
You must be signed in to change notification settings - Fork 179
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
Exception while initializing db after installation #1493
Comments
When I do the same installation procedure (with the old data directory) with the 3.4.0 release, the result is the same. |
The issue here is that the installer has a bug with regards to setting the password. The facility for setting the password, does not recognize the situation when you are installing into a directory with an existing data dir. When there is an existing data-dir, that database may have a different database password. The installer always expects the database to have an empty admin password, because that is what is there when you install into an empty directory. The password screen in the installer needs to modified to recognize an existing data-dir and either prompt for the password of the existing database, before offering to let you set a new password, or probably just skip offering to set the password altogether (as the existing database already has a password set). |
It would be nice that installer can except all the data and the admin username and password from the old data directory. In that case upgrade procedure would be quite smoother if the new version is binary compatible. I presume this procedure of update is safe and that exist accepts all data this way. |
So it looks like to be able to do more complex conditional processing in IzPack, we need to be using IzPack 5 first. I have done the majority of the work for that already here: #1489 Once we are on IzPack5, the User Input Panel for Memory and Admin Password, needs to be split into two separate panels, where the Password Panel is conditionally shown if an existing database is not present. For the conditions see: |
@adamretter I see that this bug-fix is planned for the eXist-3.6.0, but I can see improvements (thanks for your work) in eXist-3.5.0 and just want to inform you about what I am getting during update from eXist-3.4.0 to eXist-3.5.0. |
@miljanuscumlic Okay thanks. Unfortunately the work with IzPack is stuck as the IzPack developers seem to be ignoring issues. I don't know how else to contact that, but here is the issue I raised with them: https://izpack.atlassian.net/browse/IZPACK-1582 |
Okay we have now mover to IzPack 5 for eXist-db 5, so we could again continue with this issue... |
What is the problem
What did you expect
Describe how to reproduce or add a test
Context information
This is the report I got:
The part from the report
Caught an exception while initializing db: Wrong password for user [admin]
is pretty unclear because it is shown no matter what password is entered during installation whether it is the same or not as in the former installation.After installation is done, the database does not seem to be broken and I can login as admin without any problems.
When I choose some empty data directory in my second installation, the report I am getting is:
The only difference in these two installation procedures is the chosen data directory.
Btw, I do not know why I am getting these
The system cannot find the file
errors because both mentioned directories exist.The text was updated successfully, but these errors were encountered: