6.0.0-beta1
7/22/25

[#6496] Fail to update vacation msg (using sql driver)
Summary Fail to update vacation msg (using sql driver)
Queue Vacation
Queue Version 3.0.1
Type Bug
State Resolved
Priority 1. Low
Owners chuck (at) horde (dot) org
Requester pola (at) sol (dot) info (dot) unlp (dot) edu (dot) ar
Created 03/18/2008 (6335 days ago)
Due 03/18/2008 (6335 days ago)
Updated 04/25/2008 (6297 days ago)
Assigned 04/24/2008 (6298 days ago)
Resolved 04/25/2008 (6297 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch No

History
04/25/2008 02:52:21 PM Chuck Hagenbuch Comment #6
Assigned to Chuck Hagenbuch
State ⇒ Resolved
Reply to this comment
Excellent, thanks for the feedback.
04/25/2008 02:46:56 PM pola (at) sol (dot) info (dot) unlp (dot) edu (dot) ar Comment #5 Reply to this comment
The patch worked flawlessly, using crypt algorithm.

Thank you very much.

Best regards...
04/25/2008 04:29:48 AM Chuck Hagenbuch Comment #4 Reply to this comment
04/24/2008 01:14:44 PM pola (at) sol (dot) info (dot) unlp (dot) edu (dot) ar Comment #3 Reply to this comment
If it's not for some reason, we could also make it work by fetching
the current password value and passing it in to
Auth::getCryptedPassword as the seed.
That seems a better solution, my fix was a fast and dirty hack.

Best regards.
04/24/2008 03:26:10 AM Chuck Hagenbuch Comment #2
State ⇒ Feedback
Reply to this comment
It looks like that code has been there from the beginning of this 
driver; I have to say I'm not sure why the password would be used that 
way (trying to support multiple people with the same email address? I 
doubt that works...), so removing the password checks seems like a 
legit option to me.



If it's not for some reason, we could also make it work by fetching 
the current password value and passing it in to 
Auth::getCryptedPassword as the seed.



Anyone know anything about this driver or have an opinion here?
03/18/2008 01:47:32 PM pola (at) sol (dot) info (dot) unlp (dot) edu (dot) ar Comment #1
Priority ⇒ 1. Low
Type ⇒ Bug
Summary ⇒ Fail to update vacation msg (using sql driver)
Due ⇒ 03/18/2008
Queue ⇒ Vacation
Milestone ⇒
Patch ⇒ No
State ⇒ Unconfirmed
Reply to this comment
Vacation fails to update vacation msg using sql driver and crypt as 
encryption method, thats because the salt used to generate the crypted 
passwd for the query (WHERE part) does not match the salt in the 
crypted passwd in the database.



A possible solution should be to remove this lines

' AND ' . $this->_params[$realm]['pass_col'] . ' = ?';

(lines 89, 107, 147, 189)

from the sql query and just use the user email as the primary search key.



Best regards

Saved Queries