Using sssm_*_init() as the name of the initialization function for identity providers was a holdover from earlier development when we thought we would only have a single "provider" entry in the config file.
As we have now separated out the initialization functions for auth, chpass and access, we should rename sssm_*_init() to sssm_*_id_init() for a cleaner interface.
On Thu, 15 Apr 2010 14:24:09 -0400 Stephen Gallagher sgallagh@redhat.com wrote:
Using sssm_*_init() as the name of the initialization function for identity providers was a holdover from earlier development when we thought we would only have a single "provider" entry in the config file.
As we have now separated out the initialization functions for auth, chpass and access, we should rename sssm_*_init() to sssm_*_id_init() for a cleaner interface.
Ack, to the change (haven't tested the actual patch though)
Simo.
On Thu, Apr 15, 2010 at 02:24:09PM -0400, Stephen Gallagher wrote:
Using sssm_*_init() as the name of the initialization function for identity providers was a holdover from earlier development when we thought we would only have a single "provider" entry in the config file.
As we have now separated out the initialization functions for auth, chpass and access, we should rename sssm_*_init() to sssm_*_id_init() for a cleaner interface.
ACK
bye, Sumit
-- Stephen Gallagher RHCE 804006346421761
Delivering value year after year. Red Hat ranks #1 in value among software vendors. http://www.redhat.com/promo/vendor/
On 04/16/2010 06:54 AM, Sumit Bose wrote:
On Thu, Apr 15, 2010 at 02:24:09PM -0400, Stephen Gallagher wrote:
Using sssm_*_init() as the name of the initialization function for identity providers was a holdover from earlier development when we thought we would only have a single "provider" entry in the config file.
As we have now separated out the initialization functions for auth, chpass and access, we should rename sssm_*_init() to sssm_*_id_init() for a cleaner interface.
ACK
bye, Sumit
Pushed to master and sssd-1-2.
sssd-devel@lists.fedorahosted.org