FS#69750 - [nextcloud] Keep NEXTCLOUD_CONFIG_DIR environment variable in occ wrapper

Attached to Project: Community Packages
Opened by Alexander Kobel (akobel) - Tuesday, 23 February 2021, 18:36 GMT
Last edited by David Runge (dvzrv) - Monday, 01 March 2021, 08:50 GMT
Task Type Feature Request
Category Packages
Status Closed
Assigned To Sergej Pupykin (sergej)
David Runge (dvzrv)
Architecture All
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

Please replace `sudo` by `sudo --preserve-env=NEXTCLOUD_CONFIG_DIR` in the /usr/bin/occ wrapper, to allow distinguishing between multi-tenant instances of nextcloud.

And: Thanks for the revamp of the package and the user separation. I cursed a lot, but in the long run it's the proper decision for everybody's benefit.
This task depends upon

Closed by  David Runge (dvzrv)
Monday, 01 March 2021, 08:50 GMT
Reason for closing:  Fixed
Additional comments about closing:  Fixed with nextcloud 21.0.0-8
Comment by David Runge (dvzrv) - Tuesday, 23 February 2021, 21:47 GMT
@akobel: Thanks for the report!

This is a very useful addition. I will definitely add an override in a pkgrel bump! :)
Comment by David Runge (dvzrv) - Tuesday, 23 February 2021, 23:12 GMT
@akobel: Please check out 21.0.0-8 in [community-testing].

Bonus points, if you test it with uwsgi-plugin-php 2.0.19.1-9 in [community-testing], which is now built against php >= 8.0.0. ;-)
Comment by Alexander Kobel (akobel) - Wednesday, 24 February 2021, 18:17 GMT
@dvzrv: Tested and confirmed to work as expected.

Terribly sorry that I have to miss out on bonus points, but I don't have nextcloud running with uwsgi (yet) and don't have the free time to deal with that setup soon-ish. Unfortunately, the 21.0.0 update already used up a tad more than my maintenance time budget for this week...
Comment by David Runge (dvzrv) - Wednesday, 24 February 2021, 20:52 GMT
@akobel: Thanks for testing! Don't worry about uwsgi, was just worth a shot in case your setup included that. :)

I'll move 21.0.0-8 alongside the rebuilt uwsgi.

Loading...