CGI::Compile 0.24 has a bug: https://github.com/miyagawa/CGI-Compile/issues/25 It can cause weird bugs, like breaking the authority detail display after trying to modify an authority. Steps to reproduce: 1. Install CGI::Compile 0.24 2. Restart starman with only 1 worker 3. Go to the detail page of an authority (authorities/detail.pl) 4. Go to the edit page (Edit » Edit record) 5. Check starman logs, you should see "Subroutine build_tabs redefined at [...]" 6. Click on cancel to go back to the detail page, which should now show a "blank" authority. Now the only way to get the authority back is to restart starman. I suggest to just forbid CGI::Compile 0.24 in cpanfile (patch coming soon) Warning for Ubuntu Focal users: the packaged version of CGI::Compile is 0.24
Created attachment 120704 [details] [review] Bug 28302: Forbid CGI::Compile 0.24 CGI::Compile 0.24 has a bug: https://github.com/miyagawa/CGI-Compile/issues/25 It can cause weird bugs, like breaking the authority detail display after trying to modify an authority. Steps to reproduce: 1. Install CGI::Compile 0.24 2. Restart starman with only 1 worker 3. Go to the detail page of an authority (authorities/detail.pl) 4. Go to the edit page (Edit » Edit record) 5. Check starman logs, you should see "Subroutine build_tabs redefined at [...]" 6. Click on cancel to go back to the detail page, which should now show a "blank" authority. Now the only way to get the authority back is to restart starman. Test plan: 1. Reproduce the bug 2. Install CGI::Compile 0.25 and restart starman 3. Make sure the bug is gone :)
(In reply to Julian Maurice from comment #0) > Warning for Ubuntu Focal users: the packaged version of CGI::Compile is 0.24 Yikes... According to https://packages.ubuntu.com/search?keywords=libcgi-compile-perl&searchon=names, the only bad one is focal 20.04. I'll raise an issue with Ubuntu. I don't think forbidding CGI::Compile 0.24 is going to be practical for Koha users.
https://bugs.launchpad.net/ubuntu/+source/libcgi-compile-perl/+bug/1927882
So... I think that it would be OK to sign this off. For Ubuntu Focal, we should package CGI::Compile 0.25. That doesn't need to happen in the Koha codebase, so it's not a blocker here.
Created attachment 120952 [details] [review] Bug 28302: Forbid CGI::Compile 0.24 CGI::Compile 0.24 has a bug: https://github.com/miyagawa/CGI-Compile/issues/25 It can cause weird bugs, like breaking the authority detail display after trying to modify an authority. Steps to reproduce: 1. Install CGI::Compile 0.24 2. Restart starman with only 1 worker 3. Go to the detail page of an authority (authorities/detail.pl) 4. Go to the edit page (Edit » Edit record) 5. Check starman logs, you should see "Subroutine build_tabs redefined at [...]" 6. Click on cancel to go back to the detail page, which should now show a "blank" authority. Now the only way to get the authority back is to restart starman. Test plan: 1. Reproduce the bug 2. Install CGI::Compile 0.25 and restart starman 3. Make sure the bug is gone :) Signed-off-by: Victor Grousset/tuxayo <victor@tuxayo.net>
It works :) about.pl shows as expected that 0.24 is forbidden 0.24 vs 0.25 give the expected authority test results and logs.
Test notes: install version 0.24: cpanm RKITOVER/CGI-Compile-0.24.tar.gz remove version from package manager without removing dependencies and packages whose CGI::Compile is a dep (basically uninstalls all Koha stuff) sudo dpkg -r --force-depends libcgi-compile-perl
Created attachment 120976 [details] [review] Bug 28302: Forbid CGI::Compile 0.24 CGI::Compile 0.24 has a bug: https://github.com/miyagawa/CGI-Compile/issues/25 It can cause weird bugs, like breaking the authority detail display after trying to modify an authority. Steps to reproduce: 1. Install CGI::Compile 0.24 2. Restart starman with only 1 worker 3. Go to the detail page of an authority (authorities/detail.pl) 4. Go to the edit page (Edit » Edit record) 5. Check starman logs, you should see "Subroutine build_tabs redefined at [...]" 6. Click on cancel to go back to the detail page, which should now show a "blank" authority. Now the only way to get the authority back is to restart starman. Test plan: 1. Reproduce the bug 2. Install CGI::Compile 0.25 and restart starman 3. Make sure the bug is gone :) Signed-off-by: Victor Grousset/tuxayo <victor@tuxayo.net> Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Pushed to master for 21.05, thanks to everybody involved!
Pushed to 20.11.x for 20.11.06
Created attachment 121183 [details] [review] Bug 28302: Update packages to ignore CGI::Compile 0.24 to test... 1/ build package 2/ install package on focal 3/ check that libcgi-compile-perl 0.24 is not installed
Should it not be backported to 20.05 meanwhile?
(In reply to Victor Grousset/tuxayo from comment #12) > Should it not be backported to 20.05 meanwhile? yes, this should be backported to 20.05 and 19.11 too
Follow-up pushed to master.
Created attachment 121243 [details] [review] Bug 28302: Update packages to ignore CGI::Compile 0.24 (2) to test... 1/ build package 2/ install package on focal 3/ check that libcgi-compile-perl 0.24 is not installed
2 Follow-ups pushed to 20.11.x for 20.11.06
Pushed to 20.05.x for 20.05.12
No cpanfile on 19.11.x so I only backported: Bug 28302: Update packages to ignore CGI::Compile 0.24 Bug 28302: Update packages to ignore CGI::Compile 0.24 (2) Was that the right choice? Backported: Pushed to 19.11.x branch for 19.11.18
(In reply to Victor Grousset/tuxayo from comment #19) > No cpanfile on 19.11.x so I only backported: > Bug 28302: Update packages to ignore CGI::Compile 0.24 > Bug 28302: Update packages to ignore CGI::Compile 0.24 (2) > > Was that the right choice? yes :)
(In reply to Mason James from comment #20) > (In reply to Victor Grousset/tuxayo from comment #19) > > No cpanfile on 19.11.x so I only backported: > > Bug 28302: Update packages to ignore CGI::Compile 0.24 > > Bug 28302: Update packages to ignore CGI::Compile 0.24 (2) > > > > Was that the right choice? > > yes :) Shouldn't it have been added to PerlDependencies then?
Created attachment 121465 [details] [review] Bug 28302: Update PerlDependencies.pm to CGI::Compile 0.25 (19.11.x) to test... 1/ apply patch 2/ build package 3/ view about.pl to check CGI::Compile 0.25 version is satisfied
(In reply to Jonathan Druart from comment #21) > (In reply to Mason James from comment #20) > > (In reply to Victor Grousset/tuxayo from comment #19) > > > No cpanfile on 19.11.x so I only backported: > > > Bug 28302: Update packages to ignore CGI::Compile 0.24 > > > Bug 28302: Update packages to ignore CGI::Compile 0.24 (2) > > > > > > Was that the right choice? > > > > yes :) > > Shouldn't it have been added to PerlDependencies then? ah yes, thanks. i've added a patch Victor, if you push this patch for a tiny 19.11.19 release, i can build the package
Pushed in 19.11.18-2
(In reply to Victor Grousset/tuxayo from comment #24) > Pushed in 19.11.18-2 thanks Victor, closing :)