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

SQLite: Both PDO and vendor specific driver needed according to installation wizard and running instance #481

Closed
pmattern opened this issue Jul 20, 2016 · 6 comments · Fixed by #3246
Labels
3. to review Waiting for reviews bug

Comments

@pmattern
Copy link
Contributor

pmattern commented Jul 20, 2016

Both the installation wizard and running Nextcloud instances are claiming that both the PDO ("pdo_sqlite") and vendor specific ("sqlite3") driver are needed.

If the vendor specific driver ("sqlite3") is missing there's a warning
nextcloud_wizard-sqlite_no-sqlite3
right when the wizard should come up or the URL of a running instance (which had been setup with both drivers available before) is opened.

If the PDO driver is missing there's a warning
nextcloud_wizard-sqlite_no-pdo_sqlite
upon completing the wizard while a running instance (set up as depicted in previous paragraph) simply turns into a blank page.

To my knowledge it isn't common practice to use the two drivers in parallel suggesting this is a bug.
It looks somewhat similar to the problem of PostgreSQL addressed in #367.

Seen running 09e1218 or release 10.0.0 on Arch Linux x86_64, SQLite 3.14.2, Apache 2.4.23, PHP 7.0.11 via mod_php.

@MorrisJobke
Copy link
Member

cc @icewind1991

@icewind1991
Copy link
Member

I can't find any part in the setup code that uses the sqlite extension

@nickvergessen
Copy link
Member

still an issue with nextcloud 10 @pmattern ?

@pmattern pmattern changed the title SQLite: installation wizard claims both PDO and vendor specific driver are needed SQLite: Both PDO and vendor specific driver needed according to installation wizard and running instance Sep 20, 2016
@pmattern
Copy link
Contributor Author

@icewind1991, @nickvergessen
The problem persists but having another look made me realize it's even worse. I took the liberty to edit both title and first comment accordingly. You may want to have a look.

@MorrisJobke
Copy link
Member

@LukasReschke maybe we should adjust the tests?

MorrisJobke added a commit that referenced this issue Jan 24, 2017
* we only require the PDO driver
* fixes #481

Signed-off-by: Morris Jobke <hey@morrisjobke.de>
@MorrisJobke MorrisJobke added 3. to review Waiting for reviews and removed needs info labels Jan 24, 2017
@MorrisJobke MorrisJobke added this to the Nextcloud 12.0 milestone Jan 24, 2017
MorrisJobke added a commit that referenced this issue Jan 26, 2017
* we only require the PDO driver
* fixes #481

Signed-off-by: Morris Jobke <hey@morrisjobke.de>
@pmattern
Copy link
Contributor Author

Confirming the problem is fixed in latest master under the same conditions as depicted in the first posting (except the various components were of course updated in the meantime).

pmattern added a commit to pmattern/documentation that referenced this issue Jan 27, 2017
* PHP: Both PostgreSQL (nextcloud/server#367) and SQLite (nextcloud/server#481)
  are handled by PDO in recent Nextcloud versions
* Apache: Distributed configuration files (".htaccess") need to be readable
  only
pmattern added a commit to pmattern/documentation that referenced this issue Jan 27, 2017
…he configuration

* PHP: Both PostgreSQL (nextcloud/server#367) and SQLite (nextcloud/server#481)
  are handled by PDO in recent Nextcloud versions
* Apache: Distributed configuration files (".htaccess") need to be readable
  only
pmattern added a commit to pmattern/documentation that referenced this issue Jan 27, 2017
…he configuration

* PHP: Both PostgreSQL (nextcloud/server#367) and SQLite (nextcloud/server#481)
  are handled by PDO in recent Nextcloud versions
* Apache: Distributed configuration files (".htaccess") need to be readable
  only
pmattern added a commit to pmattern/documentation that referenced this issue Feb 6, 2017
Both PostgreSQL (nextcloud/server#367) and SQLite (nextcloud/server#481)
are handled by PDO in recent Nextcloud versions.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review Waiting for reviews bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants