-
-
Notifications
You must be signed in to change notification settings - Fork 4k
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
handle postgres setup when we cant connect as admin #497
Conversation
@icewind1991, thanks for your PR! By analyzing the annotation information on this pull request, we identified @bartv2, @DeepDiver1975 and @nickvergessen to be potential reviewers |
963c0e8 behaves like the current release as described in #482: First, a connection attempt is made to database "postgres" instead of the one of Nextcloud which obviously fails. Next a connection attempt using correct database and database username is made but fails due to a wrong password besides the one filed in the wizard's form is definitely correct. The error message is
where at least |
Testing with 10 beta, I found the behaviour even more problematic. I've set up a postgres database upfront with full access granted to , including a specific pg_hba.conf entry, which should be sufficient for all nextcloud server access patterns. First, the setup tries to connect using the www-data to the database (1xssl, 1x nonssl), then it tries to connect using the to the postgres database. Both is wrong, it obviously should connect to using , nothing else (at least as a first attempt). |
Moved to NC11 since it is against master |
@icewind1991 Any comments on above remarks? |
Ran into the same issue with our Cloudron app, where the database and db user are already pre-setup. Using that patch only worked by also adding
to the |
Most likely |
963c0e8
to
787f0e5
Compare
It now sets the dbname explicitly |
Running 787f0e5 the error messages shown above and in #482 can no longer be seen. Also, The first connection attempt is still made to database 787f0e5 on Arch Linux x86_64, PostgreSQL 9.5.4, Apache 2.4.23, PHP 7.0.11 via mod_php. |
@rullzer Would be nice to get this in - I will try to test this today too ;) |
787f0e5
to
5365c1a
Compare
Rebased ontop of master ;) |
I tested this and setup via postgres works fine 👍 |
@rullzer @nickvergessen Mind to review this one here? It's not a huge diff: https://github.com/nextcloud/server/pull/497/files?w=1 |
@karlitschek We need more people testing postgres or we should kill the support for it. I would vote for only go for one nice solution instead of 4 different ones that only work half of the time 🙈 We have 3,7% Postgres users as per survey server |
hmm. not sure it is realistic to drop support. Maybe we should ask in the forum for more testers? |
We are should then check out how to get more traction on the postgres side. Because this is here sind end of July. But maybe something like the Nginx guys also happen for the postgres support :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fix the issue for me. I've installed Nextcloud 10.0.1 with non admin postgres user and non standard port.
@Threeseven Thanks for the feedback 👍 |
I'd consider dropping postgres support as a major drawback, please drop thinking about it... |
#1793 is another PGSQL issue that seems rather critical… |
Fixes #482
@pmattern can you check if this fixes the problem for you?