[Koha-bugs] [Bug 28912] Pseudonymization should display a nice error message when brcypt_settings are not defined
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28912 Bug 28912 depends on bug 24151, which changed state. Bug 24151 Summary: Add a pseudonymization process for patrons and transactions https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24151 What|Removed |Added Status|Needs documenting |RESOLVED Resolution|--- |FIXED -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28912] Pseudonymization should display a nice error message when brcypt_settings are not defined
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28912 Bug 28912 depends on bug 24151, which changed state. Bug 24151 Summary: Add a pseudonymization process for patrons and transactions https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24151 What|Removed |Added Status|CLOSED |Needs documenting Resolution|FIXED |--- -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28912] Pseudonymization should display a nice error message when brcypt_settings are not defined
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28912 Victor Grousset/tuxayo changed: What|Removed |Added CC||vic...@tuxayo.net Status|Pushed to oldstable |RESOLVED Resolution|--- |FIXED --- Comment #16 from Victor Grousset/tuxayo --- Not backported to oldoldstable (20.05.x). Feel free to ask if it's needed. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28912] Pseudonymization should display a nice error message when brcypt_settings are not defined
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28912 Fridolin Somers changed: What|Removed |Added Version(s)|21.11.00,21.05.04 |21.11.00,21.05.04,20.11.10 released in|| CC||fridolin.som...@biblibre.co ||m Status|Pushed to stable|Pushed to oldstable --- Comment #15 from Fridolin Somers --- Pushed to 20.11.x for 20.11.10 -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28912] Pseudonymization should display a nice error message when brcypt_settings are not defined
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28912 Kyle M Hall changed: What|Removed |Added Version(s)|21.11.00|21.11.00,21.05.04 released in|| CC||k...@bywatersolutions.com Status|Pushed to master|Pushed to stable --- Comment #14 from Kyle M Hall --- Pushed to 21.05.x for 21.05.04 -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28912] Pseudonymization should display a nice error message when brcypt_settings are not defined
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28912 --- Comment #13 from Jonathan Druart --- Pushed to master for 21.11, thanks to everybody involved! -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28912] Pseudonymization should display a nice error message when brcypt_settings are not defined
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28912 Jonathan Druart changed: What|Removed |Added Version(s)||21.11.00 released in|| Status|Passed QA |Pushed to master -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28912] Pseudonymization should display a nice error message when brcypt_settings are not defined
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28912 Joonas Kylmälä changed: What|Removed |Added CC||joonas.kylm...@iki.fi QA Contact|testo...@bugs.koha-communit |joonas.kylm...@iki.fi |y.org | --- Comment #12 from Joonas Kylmälä --- This is a nice improvement and blocks the common case where the ISE might happen. If someone wants to further improve this the specification for the bcrypt setting string is defined in https://metacpan.org/pod/Crypt::Eksblowfish::Bcrypt#bcrypt(PASSWORD,-SETTINGS) and we could handle the error more gracefully if it doesn't follow the format. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28912] Pseudonymization should display a nice error message when brcypt_settings are not defined
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28912 --- Comment #11 from Joonas Kylmälä --- Created attachment 124520 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=124520=edit Bug 28912: Remove duplicated warnings on the about page Bad conflict resolution Signed-off-by: Andrew Fuerste-Henry Signed-off-by: Joonas Kylmälä -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28912] Pseudonymization should display a nice error message when brcypt_settings are not defined
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28912 Joonas Kylmälä changed: What|Removed |Added Attachment #124236|0 |1 is obsolete|| Attachment #124237|0 |1 is obsolete|| --- Comment #10 from Joonas Kylmälä --- Created attachment 124519 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=124519=edit Bug 28912: Prevent Pseudonymization to be set if bcrypt_settings not set If Pseudonymization is set but the bcrypt_settings config used by the feature is not set, then there is an ugly 500 on checking out. bad bcrypt settings at /kohadevbox/koha/Koha/PseudonymizedTransaction.pm line 116. However it's pretty hard to handle correctly this exception (and that's why it hasn't be done initially). However we could prevent the pref to be turned on if the config entry is not present. Test plan: Remove the bcrypt_settings from the config Try to turn the syspref on Add the config Try to turn the syspref on/off Signed-off-by: Andrew Fuerste-Henry JD amended patch: fix qa failures Signed-off-by: Joonas Kylmälä -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28912] Pseudonymization should display a nice error message when brcypt_settings are not defined
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28912 Joonas Kylmälä changed: What|Removed |Added Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28912] Pseudonymization should display a nice error message when brcypt_settings are not defined
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28912 Jonathan Druart changed: What|Removed |Added Status|Failed QA |Signed Off --- Comment #9 from Jonathan Druart --- Oops, fixed! -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28912] Pseudonymization should display a nice error message when brcypt_settings are not defined
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28912 --- Comment #8 from Jonathan Druart --- Created attachment 124237 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=124237=edit Bug 28912: Remove duplicated warnings on the about page Bad conflict resolution Signed-off-by: Andrew Fuerste-Henry -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28912] Pseudonymization should display a nice error message when brcypt_settings are not defined
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28912 Jonathan Druart changed: What|Removed |Added Attachment #124191|0 |1 is obsolete|| Attachment #124192|0 |1 is obsolete|| --- Comment #7 from Jonathan Druart --- Created attachment 124236 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=124236=edit Bug 28912: Prevent Pseudonymization to be set if bcrypt_settings not set If Pseudonymization is set but the bcrypt_settings config used by the feature is not set, then there is an ugly 500 on checking out. bad bcrypt settings at /kohadevbox/koha/Koha/PseudonymizedTransaction.pm line 116. However it's pretty hard to handle correctly this exception (and that's why it hasn't be done initially). However we could prevent the pref to be turned on if the config entry is not present. Test plan: Remove the bcrypt_settings from the config Try to turn the syspref on Add the config Try to turn the syspref on/off Signed-off-by: Andrew Fuerste-Henry JD amended patch: fix qa failures -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28912] Pseudonymization should display a nice error message when brcypt_settings are not defined
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28912 --- Comment #6 from Katrin Fischer --- Hm, updated QA test tools to make sure, but it just added one more to the list: FAIL admin/preferences.pl OK critic FAIL forbidden patterns forbidden pattern: Use of Data::Printer, should only be used for devlopment purpose (line 238) OK git manipulation -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28912] Pseudonymization should display a nice error message when brcypt_settings are not defined
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28912 Katrin Fischer changed: What|Removed |Added Status|Signed Off |Failed QA --- Comment #5 from Katrin Fischer --- There are 2 fails here in the QA script: FAIL koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences.tt FAIL filters missing_filter at line 118 () missing_filter at line 187 ([% CASE %]Unknown warning "[% w %]") OK forbidden patterns OK git manipulation OK js_in_body OK spelling FAIL tt_valid lines 118 Please fix! -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28912] Pseudonymization should display a nice error message when brcypt_settings are not defined
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28912 Andrew Fuerste-Henry changed: What|Removed |Added Attachment #124187|0 |1 is obsolete|| --- Comment #4 from Andrew Fuerste-Henry --- Created attachment 124192 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=124192=edit Bug 28912: Remove duplicated warnings on the about page Bad conflict resolution Signed-off-by: Andrew Fuerste-Henry -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28912] Pseudonymization should display a nice error message when brcypt_settings are not defined
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28912 Andrew Fuerste-Henry changed: What|Removed |Added Attachment #124186|0 |1 is obsolete|| --- Comment #3 from Andrew Fuerste-Henry --- Created attachment 124191 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=124191=edit Bug 28912: Prevent Pseudonymization to be set if bcrypt_settings not set If Pseudonymization is set but the bcrypt_settings config used by the feature is not set, then there is an ugly 500 on checking out. bad bcrypt settings at /kohadevbox/koha/Koha/PseudonymizedTransaction.pm line 116. However it's pretty hard to handle correctly this exception (and that's why it hasn't be done initially). However we could prevent the pref to be turned on if the config entry is not present. Test plan: Remove the bcrypt_settings from the config Try to turn the syspref on Add the config Try to turn the syspref on/off Signed-off-by: Andrew Fuerste-Henry -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28912] Pseudonymization should display a nice error message when brcypt_settings are not defined
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28912 Andrew Fuerste-Henry changed: What|Removed |Added Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28912] Pseudonymization should display a nice error message when brcypt_settings are not defined
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28912 Andrew Fuerste-Henry changed: What|Removed |Added CC||and...@bywatersolutions.com -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28912] Pseudonymization should display a nice error message when brcypt_settings are not defined
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28912 Jonathan Druart changed: What|Removed |Added Assignee|koha-b...@lists.koha-commun |jonathan.druart+koha@gmail. |ity.org |com Depends on||24151 CC||jonathan.druart+koha@gmail. ||com Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24151 [Bug 24151] Add a pseudonymization process for patrons and transactions -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28912] Pseudonymization should display a nice error message when brcypt_settings are not defined
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28912 --- Comment #2 from Jonathan Druart --- Created attachment 124187 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=124187=edit Bug 28912: Remove duplicated warnings on the about page Bad conflict resolution -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28912] Pseudonymization should display a nice error message when brcypt_settings are not defined
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28912 --- Comment #1 from Jonathan Druart --- Created attachment 124186 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=124186=edit Bug 28912: Prevent Pseudonymization to be set if bcrypt_settings not set If Pseudonymization is set but the bcrypt_settings config used by the feature is not set, then there is an ugly 500 on checking out. bad bcrypt settings at /kohadevbox/koha/Koha/PseudonymizedTransaction.pm line 116. However it's pretty hard to handle correctly this exception (and that's why it hasn't be done initially). However we could prevent the pref to be turned on if the config entry is not present. Test plan: Remove the bcrypt_settings from the config Try to turn the syspref on Add the config Try to turn the syspref on/off -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28912] Pseudonymization should display a nice error message when brcypt_settings are not defined
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28912 Jonathan Druart changed: What|Removed |Added Status|NEW |Needs Signoff -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 28912] Pseudonymization should display a nice error message when brcypt_settings are not defined
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28912 Nick Clemens changed: What|Removed |Added See Also||https://bugs.koha-community ||.org/bugzilla3/show_bug.cgi ||?id=28911 -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
