[dev] Re: commit: horde/config hooks.php.dist passwd main.php passwd/lib/Driver sql.php

[Marcus I. Ryan]

If this is the way you intend to go, I think that should work, but you
need to fix the hook, then, too.  The hook right now takes Auth::getAuth
and passes it to the hook.  What's returned is passed to the form
generation and the default name is set.  When a password is actually
reset we use the name passed in, NOT whatever the hook returns.  That
needs to be adjusted (or the other stuff does) so that the hook still
works as expected.

Quoting Mike Cochrane <mike@graftonhall.co.nz>:

> > ----- Message from marcus@riboflavin.net ---------
> >
> > Frankly, this should be the opposite of what it is.  My problem is
> > Auth::getAuth() returns user@domain.com, which I need passed to the
> > backend, but passwd automatically removes the domain portion and
> sends
> > just the user.  I set up a hook to bypass that functionality.  It
> really
> > should be the other way though - assume Auth::getAuth() is right
> unless
> > they define a hook to properly convert.  Either way, the hook
> should be
> > translating Auth::getAuth() to fill in the form, not messing with
> what
> > is passed.
> >
> 
> Okay, getting a lot of conlicting views on this so:
> 
> LDAP has a realm paramter in it's config and I will add the same to
> the other
> drivers. Which will 'unbreak' the realms handling in the drivers.
> 
> The hook with the what the user has entered and convert it to what
> the backend
> wants. If this means you ignore that the user entered and return the
> first bit
> of Auth::getAuth() that's fine just make sure you haven't set allow
> guests to true.
> 
> One the backend gets this username it will append the realm to it, if
> it's been
> specified, and continue to update the backend.
> 
> How does that work for you guys?
> 
> Just so you have an idea where i'm going with password.. I'm making
> it a more
> generic password change application, so you can change password that
> aren't
> currently being used to login to horde.
> 
> - Mike :-)
> 
> --
> Horde developers mailing list
> Frequently Asked Questions: http://horde.org/faq/
> To unsubscribe, mail: dev-unsubscribe@lists.horde.org


-- 
Marcus I. Ryan, marcus@riboflavin.net
-----------------------------------------------------------------------
 "Love is a snowmobile racing across the tundra and then suddenly it
 flips over, pinning you underneath.  At night, the ice weasels come."
                 -- Matt Groening
-----------------------------------------------------------------------