[Koha-bugs] [Bug 18025] Expired password recovery links cause sql crash
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18025 Mason Jameschanged: What|Removed |Added CC||m...@kohaaloha.com --- Comment #20 from Mason James --- Pushed to 16.05.x, for 16.05.10 release -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 18025] Expired password recovery links cause sql crash
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18025 Katrin Fischerchanged: What|Removed |Added CC||katrin.fisc...@bsz-bw.de Status|Pushed to Master|Pushed to Stable --- Comment #19 from Katrin Fischer --- These patches have been pushed to 16.11.x and will be in 16.11.04. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 18025] Expired password recovery links cause sql crash
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18025 Kyle M Hallchanged: What|Removed |Added Status|Passed QA |Pushed to Master CC||k...@bywatersolutions.com --- Comment #18 from Kyle M Hall --- Pushed to master for 17.05, thanks Liz, Jonathan! -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 18025] Expired password recovery links cause sql crash
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18025 Jonathan Druartchanged: What|Removed |Added Attachment #60305|0 |1 is obsolete|| --- Comment #17 from Jonathan Druart --- Created attachment 60313 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=60313=edit Bug 18025: Simplify logic and avoid 1 call to ValidateBorrowernumber Signed-off-by: Liz Rea This is fine with me. Signed-off-by: Jonathan Druart -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 18025] Expired password recovery links cause sql crash
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18025 Jonathan Druartchanged: What|Removed |Added Attachment #60304|0 |1 is obsolete|| --- Comment #16 from Jonathan Druart --- Created attachment 60312 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=60312=edit Bug 18025: Fix test Signed-off-by: Liz Rea Seems to work fine Signed-off-by: Jonathan Druart -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 18025] Expired password recovery links cause sql crash
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18025 Jonathan Druartchanged: What|Removed |Added Attachment #60303|0 |1 is obsolete|| --- Comment #15 from Jonathan Druart --- Created attachment 60311 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=60311=edit Bug 18025 - Expired password recovery links cause sql crash When a user gets an email, but doesn't act or visit it within two days, attempting to create a new one causes a collision. We should just delete the old one, assuming they still want to reset their password. To test: create yourself a borrower with a userid and password. Attempt a password recovery on the OPAC update the entry in the database for that user to have an expired token e.g. update borrower_password_recovery set valid_until = '2017-01-25 03:25:26' where borrowernumber = 12; Attempt another password recovery operation - should error apply the patch Try it again - no error, new token is generated and additional email with new link is sent. Issue reproduced - is resolved by patch Signed-off-by: Marc Véron Signed-off-by: Jonathan Druart -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 18025] Expired password recovery links cause sql crash
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18025 --- Comment #14 from Jonathan Druart--- Created attachment 60305 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=60305=edit Bug 18025: Simplify logic and avoid 1 call to ValidateBorrowernumber Signed-off-by: Liz Rea This is fine with me. Signed-off-by: Jonathan Druart -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 18025] Expired password recovery links cause sql crash
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18025 Jonathan Druartchanged: 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 http://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 18025] Expired password recovery links cause sql crash
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18025 Jonathan Druartchanged: What|Removed |Added Attachment #60245|0 |1 is obsolete|| Attachment #60295|0 |1 is obsolete|| Attachment #60296|0 |1 is obsolete|| --- Comment #12 from Jonathan Druart --- Created attachment 60303 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=60303=edit Bug 18025 - Expired password recovery links cause sql crash When a user gets an email, but doesn't act or visit it within two days, attempting to create a new one causes a collision. We should just delete the old one, assuming they still want to reset their password. To test: create yourself a borrower with a userid and password. Attempt a password recovery on the OPAC update the entry in the database for that user to have an expired token e.g. update borrower_password_recovery set valid_until = '2017-01-25 03:25:26' where borrowernumber = 12; Attempt another password recovery operation - should error apply the patch Try it again - no error, new token is generated and additional email with new link is sent. Issue reproduced - is resolved by patch Signed-off-by: Marc Véron Signed-off-by: Jonathan Druart -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 18025] Expired password recovery links cause sql crash
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18025 --- Comment #13 from Jonathan Druart--- Created attachment 60304 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=60304=edit Bug 18025: Fix test Signed-off-by: Liz Rea Seems to work fine Signed-off-by: Jonathan Druart -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 18025] Expired password recovery links cause sql crash
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18025 Liz Reachanged: What|Removed |Added Attachment #60282|0 |1 is obsolete|| --- Comment #11 from Liz Rea --- Created attachment 60296 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=60296=edit Bug 18025: Simplify logic and avoid 1 call to ValidateBorrowernumber Signed-off-by: Liz Rea This is fine with me. Current status: Signed Off -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 18025] Expired password recovery links cause sql crash
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18025 Liz Reachanged: What|Removed |Added Attachment #60281|0 |1 is obsolete|| --- Comment #10 from Liz Rea --- Created attachment 60295 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=60295=edit Bug 18025: Fix test Signed-off-by: Liz Rea Seems to work fine -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 18025] Expired password recovery links cause sql crash
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18025 Jonathan Druartchanged: What|Removed |Added Assignee|oleon...@myacpl.org |l...@catalyst.net.nz Status|Failed QA |Signed Off --- Comment #9 from Jonathan Druart --- Liz, Does the last patch sounds ok to you? -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 18025] Expired password recovery links cause sql crash
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18025 --- Comment #8 from Jonathan Druart--- Created attachment 60282 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=60282=edit Bug 18025: Simplify logic and avoid 1 call to ValidateBorrowernumber -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 18025] Expired password recovery links cause sql crash
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18025 --- Comment #7 from Jonathan Druart--- Created attachment 60281 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=60281=edit Bug 18025: Fix test -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 18025] Expired password recovery links cause sql crash
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18025 Jonathan Druartchanged: What|Removed |Added Status|Signed Off |Failed QA --- Comment #6 from Jonathan Druart --- Liz, could you double check please? The tests do not pass for me. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 18025] Expired password recovery links cause sql crash
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18025 Marc Véronchanged: What|Removed |Added Attachment #60223|0 |1 is obsolete|| --- Comment #5 from Marc Véron --- Created attachment 60245 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=60245=edit Bug 18025 - Expired password recovery links cause sql crash When a user gets an email, but doesn't act or visit it within two days, attempting to create a new one causes a collision. We should just delete the old one, assuming they still want to reset their password. To test: create yourself a borrower with a userid and password. Attempt a password recovery on the OPAC update the entry in the database for that user to have an expired token e.g. update borrower_password_recovery set valid_until = '2017-01-25 03:25:26' where borrowernumber = 12; Attempt another password recovery operation - should error apply the patch Try it again - no error, new token is generated and additional email with new link is sent. Issue reproduced - is resolved by patch Signed-off-by: Marc Véron -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 18025] Expired password recovery links cause sql crash
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18025 Marc Véronchanged: 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 http://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 18025] Expired password recovery links cause sql crash
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18025 Liz Reachanged: What|Removed |Added Attachment #60108|0 |1 is obsolete|| --- Comment #4 from Liz Rea --- Created attachment 60223 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=60223=edit Bug 18025 - Expired password recovery links cause sql crash When a user gets an email, but doesn't act or visit it within two days, attempting to create a new one causes a collision. We should just delete the old one, assuming they still want to reset their password. To test: create yourself a borrower with a userid and password. Attempt a password recovery on the OPAC update the entry in the database for that user to have an expired token e.g. update borrower_password_recovery set valid_until = '2017-01-25 03:25:26' where borrowernumber = 12; Attempt another password recovery operation - should error apply the patch Try it again - no error, new token is generated and additional email with new link is sent. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 18025] Expired password recovery links cause sql crash
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18025 Liz Reachanged: What|Removed |Added Status|Failed QA |Needs Signoff -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 18025] Expired password recovery links cause sql crash
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18025 Jonathan Druartchanged: What|Removed |Added Status|Signed Off |Failed QA CC||jonathan.dru...@bugs.koha-c ||ommunity.org --- Comment #3 from Jonathan Druart --- It sounds weird to make to have a validate/check subroutine that actually deletes stuffs. I think it would be better to delete it, if it exists, before regenerating a new one. Moreover the search argument does not make sense, there is a 'or' with only one condition. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 18025] Expired password recovery links cause sql crash
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18025 Jonathan Druartchanged: What|Removed |Added Severity|normal |major -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 18025] Expired password recovery links cause sql crash
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18025 Marc Véronchanged: What|Removed |Added Status|Needs Signoff |Signed Off CC||ve...@veron.ch -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 18025] Expired password recovery links cause sql crash
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18025 Marc Véronchanged: What|Removed |Added Attachment #59760|0 |1 is obsolete|| --- Comment #2 from Marc Véron --- Created attachment 60108 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=60108=edit Bug 18025 - Expired password recovery links cause sql crash When a user gets an email, but doesn't act or visit it within two days, attempting to create a new one causes a collision. We should just delete the old one, assuming they still want to reset their password. To test: create yourself a borrower with a userid and password. Attempt a password recovery on the OPAC update the entry in the database for that user to have an expired token e.g. update borrower_password_recovery set valid_until = '2017-01-25 03:25:26' where borrowernumber = 12; Attempt another password recovery operation - should error apply the patch Try it again - no error, new token is generated and additional email with new link is sent. Issue reproduced without patch; resolved with patch. Signed-off-by: Marc Véron -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 18025] Expired password recovery links cause sql crash
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18025 Liz Reachanged: What|Removed |Added Status|NEW |Needs Signoff -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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 18025] Expired password recovery links cause sql crash
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18025 --- Comment #1 from Liz Rea--- Created attachment 59760 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=59760=edit Bug 18025 - Expired password recovery links cause sql crash When a user gets an email, but doesn't act or visit it within two days, attempting to create a new one causes a collision. We should just delete the old one, assuming they still want to reset their password. To test: create yourself a borrower with a userid and password. Attempt a password recovery on the OPAC update the entry in the database for that user to have an expired token e.g. update borrower_password_recovery set valid_until = '2017-01-25 03:25:26' where borrowernumber = 12; Attempt another password recovery operation - should error apply the patch Try it again - no error, new token is generated and additional email with new link is sent. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://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/