- Koha-bugs - lists.koha-community.org

[Bug 29957] Cookies not removed after logout
by bugzilla-daemon＠bugs.koha-community.org 25 Mar '22

25 Mar '22

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29957 --- Comment #108 from Nick Clemens <nick(a)bywatersolutions.com> --- Created attachment 132235 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=132235&action=e… Bug 29957: Adjust push @$cookie statements in Auth We can now use $cookie_mgr->replace_in_list instead. This effectively removes duplicates and keeps the newest cookie. Note: In the test plan below we are just verifying if this patch did not change behavior. The replace_in_list routine has been tested already in a unit test. Test plan: Run t/db_dependent/Auth.t Login at OPAC. Hit some opac and staff pages. Perform an Advanced search on OPAC. Check cookies in browser. Logout. Check cookies again. Verify with your do_not_remove_cookie lines in koha-conf.xml. Signed-off-by: Marcel de Rooy <m.de.rooy(a)rijksmuseum.nl> Signed-off-by: Nick Clemens <nick(a)bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 29957] Cookies not removed after logout
by bugzilla-daemon＠bugs.koha-community.org 25 Mar '22

25 Mar '22

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29957 --- Comment #107 from Nick Clemens <nick(a)bywatersolutions.com> --- Created attachment 132234 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=132234&action=e… Bug 29957: Replace clear_all_cookies in Auth.pm Test plan: Enable language selection and have two languages. Change language. Check cookie value in browser for KohaOpacLanguage. Logout. Verify that cookie has been cleared in browser. Add do_not_remove_cookie line for KohaOpacLanguage in koha-conf. Restart, flush. Login again. Change language. Check cookie value in browser for KohaOpacLanguage. Logout. Verify that cookie still contains the language. Signed-off-by: Marcel de Rooy <m.de.rooy(a)rijksmuseum.nl> Signed-off-by: Nick Clemens <nick(a)bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 29957] Cookies not removed after logout
by bugzilla-daemon＠bugs.koha-community.org 25 Mar '22

25 Mar '22

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29957 --- Comment #106 from Nick Clemens <nick(a)bywatersolutions.com> --- Created attachment 132233 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=132233&action=e… Bug 29957: (follow-up) Turn allow list into deny list See the discussion on the Bugzilla report. It might be easier to work with a deny list. Test plan: Run t/CookieManager.t Signed-off-by: Marcel de Rooy <m.de.rooy(a)rijksmuseum.nl> Signed-off-by: Nick Clemens <nick(a)bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 29957] Cookies not removed after logout
by bugzilla-daemon＠bugs.koha-community.org 25 Mar '22

25 Mar '22

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29957 --- Comment #105 from Nick Clemens <nick(a)bywatersolutions.com> --- Created attachment 132232 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=132232&action=e… Bug 29957: Introduce Koha::CookieManager Test plan: Run t/CookieManager.t Signed-off-by: Marcel de Rooy <m.de.rooy(a)rijksmuseum.nl> Signed-off-by: Nick Clemens <nick(a)bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 29957] Cookies not removed after logout
by bugzilla-daemon＠bugs.koha-community.org 25 Mar '22

25 Mar '22

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29957 Nick Clemens <nick(a)bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #132105|0 |1 is obsolete| | Attachment #132106|0 |1 is obsolete| | Attachment #132107|0 |1 is obsolete| | Attachment #132108|0 |1 is obsolete| | Attachment #132109|0 |1 is obsolete| | Attachment #132110|0 |1 is obsolete| | Attachment #132115|0 |1 is obsolete| | Attachment #132211|0 |1 is obsolete| | --- Comment #104 from Nick Clemens <nick(a)bywatersolutions.com> --- Created attachment 132231 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=132231&action=e… Bug 29957: Clear cookies on logout Signed-off-by: Martin Renvoize <martin.renvoize(a)ptfs-europe.com> Signed-off-by: Marcel de Rooy <m.de.rooy(a)rijksmuseum.nl> Amended: Reverted the change to $cookies to minimize changes. Fixed Auth.t where checkauth is being mocked. Signed-off-by: Nick Clemens <nick(a)bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 29957] Cookies not removed after logout
by bugzilla-daemon＠bugs.koha-community.org 25 Mar '22

25 Mar '22

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29957 Nick Clemens <nick(a)bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 20362] New: Direct link to authority records missing in staff detail view (1xx, 7xx)
by bugzilla-daemon＠bugs.koha-community.org 25 Mar '22

25 Mar '22

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20362 Bug ID: 20362 Summary: Direct link to authority records missing in staff detail view (1xx, 7xx) Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: MARC Bibliographic data support Assignee: koha-bugs(a)lists.koha-community.org Reporter: katrin.fischer(a)bsz-bw.de QA Contact: testopia(a)bugs.koha-community.org Add a link to the authority record from the detail patch for authors and contributors (1xx, 7xx) like they exist in the staff page and like it's implemented for subjects on bug 16427. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

1 27

[Bug 30098] New: Patron search redirects when one result on any page of results
by bugzilla-daemon＠bugs.koha-community.org 25 Mar '22

25 Mar '22

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30098 Bug ID: 30098 Summary: Patron search redirects when one result on any page of results Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: normal Priority: P5 - low Component: Patrons Assignee: koha-bugs(a)lists.koha-community.org Reporter: nick(a)bywatersolutions.com QA Contact: testopia(a)bugs.koha-community.org CC: gmcharlt(a)gmail.com, kyle.m.hall(a)gmail.com The patron search has a redirect built in to send you directly to the patron if there is only a single result from your search. This is also triggered, however, when a later page has only 1 result To recreate: 1 - Perform a patron search that returns 41 results, on Koha testing docker, 'a' works 2 - Go to second page of results, works 3 - Go to third page of results, redirected to the patron -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

1 21

[Bug 29957] Cookies not removed after logout
by bugzilla-daemon＠bugs.koha-community.org 25 Mar '22

25 Mar '22

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29957 Marcel de Rooy <m.de.rooy(a)rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #132210|0 |1 is obsolete| | --- Comment #103 from Marcel de Rooy <m.de.rooy(a)rijksmuseum.nl> --- Created attachment 132211 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=132211&action=e… Bug 29957: Add --keep-cookie option to koha-create As requested on comment99. Personally I do not feel that it should be a requirement for adding a koha-conf entry though. Note that koha-create does/did not even support adding multiple line entries to koha-conf like we do here now. But it might be a start for adding them? Test plan: The following tweak might help you to test without running the complete koha-create, while still convincing you: [1] cp debian/templates/koha-conf-site.xml.in /etc/koha/test.conf [2] Add the following line in debian/scripts/koha-create. Append if after the BCRYPT_SETTINGS= line on L652: generate_config_file test.conf test.xml; exit [3] Run debian/scripts/koha-create --keep-cookie A [4] Check if you see <do_not_remove_cookie>A<..> in test.xml. [5] Run debian/scripts/koha-create --keep-cookie A --keep-cookie B [6] Check for two lines in test.xml [7] Run debian/scripts/koha-create clone_name [8] Check that the __KEEP_COOKIE__ line is gone in test.xml. [9] Undo the tweak by removing test.xml and /etc/koha/test.conf, and removing the added line from koha-create. Signed-off-by: Marcel de Rooy <m.de.rooy(a)rijksmuseum.nl> -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 29957] Cookies not removed after logout
by bugzilla-daemon＠bugs.koha-community.org 25 Mar '22

25 Mar '22

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29957 --- Comment #102 from Marcel de Rooy <m.de.rooy(a)rijksmuseum.nl> --- Created attachment 132210 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=132210&action=e… Bug 29957: Add --keep-cookie option to koha-create As requested on comment99. Personally I do not feel that it should be a requirement for adding a koha-conf entry though. Note that koha-conf does/did not even support adding multiple line entries to koha-conf like we do here now. But it might be a start for adding them? Test plan: The following tweak might help you to test without running the complete koha-create, while still convincing you: [1] cp debian/templates/koha-conf-site.xml.in /etc/koha/test.conf [2] Add the following line in debian/scripts/koha-create. Append if after the BCRYPT_SETTINGS= line on L652: generate_config_file test.conf test.xml; exit [3] Run debian/scripts/koha-create --keep-cookie A [4] Check if you see <do_not_remove_cookie>A<..> in test.xml. [5] Run debian/scripts/koha-create --keep-cookie A --keep-cookie B [6] Check for two lines in test.xml [7] Run debian/scripts/koha-create clone_name [8] Check that the __KEEP_COOKIE__ line is gone in test.xml. [9] Undo the tweak by removing test.xml and /etc/koha/test.conf, and removing the added line from koha-create. Signed-off-by: Marcel de Rooy <m.de.rooy(a)rijksmuseum.nl> -- You are receiving this mail because: You are watching all bug changes.

1 0