Summary: | starting plack on reboot can fail on non-standard apache configs | ||
---|---|---|---|
Product: | Koha | Reporter: | Liz Rea <liz> |
Component: | System Administration | Assignee: | Liz Rea <liz> |
Status: | CLOSED WONTFIX | QA Contact: | Testopia <testopia> |
Severity: | enhancement | ||
Priority: | P5 - low | ||
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
GIT URL: | Change sponsored?: | --- | |
Patch complexity: | Trivial patch | Documentation contact: | |
Documentation submission: | Text to go in the release notes: | ||
Version(s) released in: | Circulation function: | ||
Attachments: |
Bug 18617 - starting plack on reboot can fail on non-standard apache configs
Bug 18617: Follow up - correctly check plack.enabled file first |
Description
Liz Rea
2017-05-16 22:54:59 UTC
Created attachment 63493 [details] [review] Bug 18617 - starting plack on reboot can fail on non-standard apache configs To test: apply this patch copy debian/scripts/koha-functions.sh to /usr/share/koha/bin copy debian/scripts/koha-plack to /usr/sbin create a dummy instance if you need to (koha-create --create-db whatever) enable plack - koha-plack --enable whatever check for existence of /var/lib/koha/whatever/plack.enabled do a koha-list --enabled --plack and your instance should be listed as enabled. rename your /etc/apache2/sites-available/whatever.conf to something else do a koha-list --enabled --plack - it should still be enabled do a koha-plack --disable whatever - it should remove /var/lib/koha/whatever/plack.enabled do a koha-list --enabled --plack - it should not be listed. Note: This does not deal with the issue of disabling plack not doing the right thing for config files that are not "/etc/apache2/sites-available/whatever.conf" - this only ensures that koha-list --enabled and --disabled reports correctly in that circumstance. Bump, we'll need this if we want to ever split up the apache configuration files as we should. (In reply to Liz Rea from comment #2) > Bump, we'll need this if we want to ever split up the apache configuration > files as we should. I vaguely recall that LetsEncrypt does not like split config files. Do you have objections that are further to the actual function here, because I don't really want to look up and document every bit of certbot to defend that. I have directly experienced the plack problem with vhosts in separate files, it ought to be enough to justify the change. Cheers, Liz I like the idea of not having to grep the apache configuration files. I was testing this, and discovered that this doesn't fully work. I'm going to attach a follow up which should. If that makes you happy, Liz, then sign off, and I'll sign off, and we can set this to signed off. Created attachment 70971 [details] [review] Bug 18617: Follow up - correctly check plack.enabled file first This patch was required, because without it, the apache2 config grepping logic was preventing the elif from checking the existence of the enabled file. |