[SSSD] [PATCH] SYSDB: Save only lowercased aliases in case-insensitive domains

Stephen Gallagher sgallagh at redhat.com
Fri Mar 16 18:11:17 UTC 2012


On Fri, 2012-03-16 at 15:28 +0100, Jan Zelený wrote:
> > On Fri, 2012-03-16 at 08:48 +0100, Pavel Březina wrote:
> > > On 03/15/2012 09:02 PM, Stephen Gallagher wrote:
> > > > Fixes https://fedorahosted.org/sssd/ticket/1253
> > > > 
> > > > Previously, we were saving both the lower-cased alias names as well as
> > > > the original capitalization. This was done in order to allow switching
> > > > case-sensitive/insensitive modes safely, but it had the negative effect
> > > > of producing duplicate alias entries (differing only by case) on maps
> > > > like 'services' which have an 'aliases' field.
> > > > 
> > > > Jakub and I discussed it and we agreed that switching case-sensitivity
> > > > should always warrant a cache-purge, as it's a fundamental change to
> > > > the cache contents.
> > > 
> > > You forgot to attach the patch. :)
> > 
> > So I did. Thanks.
> 
> Just one small thing. For the sake of nice output when debugging, would it be 
> possible to move the DEBUG line out of the for cycle?
> 
> Other than that the patch is fine

I moved the DEBUG message and pushed to master and sssd-1-8.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: This is a digitally signed message part
URL: <https://lists.fedorahosted.org/pipermail/sssd-devel/attachments/20120316/652f565d/attachment.sig>


More information about the sssd-devel mailing list