Tickets :: [#4456] Allow preauthenticate hook to modify parameters

6.0.0-alpha14

7/1/25

Summary	Allow preauthenticate hook to modify parameters
Queue	Horde Framework Packages
Queue Version	HEAD
Type	Enhancement
State	Resolved
Priority	1. Low
Owners	slusarz (at) horde (dot) org
Requester	bklang (at) horde (dot) org
Created	09/23/2006 (6856 days ago)
Due
Updated	12/10/2009 (5682 days ago)
Assigned	11/09/2008 (6078 days ago)
Resolved	12/09/2009 (5683 days ago)
Milestone	4
Patch	No

12/10/2009 12:21:46 AM	Jan Schneider	Milestone ⇒ 4

12/09/2009 11:11:28 PM	Michael Slusarz	Comment #6 Assigned to Michael Slusarz Taken from Ben Klang State ⇒ Resolved	Reply to this comment
Credentials can be modified in Horde 4 via the preauthenticate hook.

11/09/2008 01:35:47 AM	Chuck Hagenbuch	Comment #5 Assigned to Ben Klang State ⇒ Assigned	Reply to this comment
Ben, you can go ahead and do this in HEAD now.

11/14/2006 05:11:12 AM	Chuck Hagenbuch	Comment #4 State ⇒ Stalled	Reply to this comment
Stalling with a Horde 4.0 milestone for now.

11/13/2006 05:13:35 PM	Ben Klang	Comment #3	Reply to this comment
You're right, sorry about that. I looked into trying to do the same modification to Horde::callHook() but it is not possible with PHP4. callHook() sets $args to a default value of array(). If the function signature is changed to specify $args as a pass-by-reference, a parse error is thrown. According to the PHP documentation this is valid in PHP5 but not PHP4. Unless I'm missing something, this change isn't going to be possible (at least, not while maintaining backward compatibility).

09/27/2006 11:50:31 AM	Jan Schneider	Comment #2 State ⇒ Feedback	Reply to this comment
This is not against the latest HEAD, the hook call has been replaced with callHook().

09/23/2006 03:53:34 AM	Ben Klang	Comment #1 Priority ⇒ 1. Low State ⇒ New New Attachment: horde-hook-preauth-pbr.patch Queue ⇒ Horde Framework Packages Summary ⇒ Allow preauthenticate hook to modify parameters Type ⇒ Enhancement	Reply to this comment
Recently I found a need to rewrite the username sent to Horde on authentication. While the tobackend and frombackend hooks looked appealing at first, they are not called in the appropriate locations to be useable for this goal. After discussing alternatives with Chuck, I settled on this approach: Pass the values by reference to the preauthenticate hook. This will allow rewriting of the parameters if required. The defualt case does not do that so this should not impact BC. It does however let me rewrite the username to include the domain name of the HTTP Vhost when the user authenticates.