On Thu, Oct 25, 2012 at 10:43:03AM +0200, Sumit Bose wrote:
On Wed, Oct 24, 2012 at 11:21:06AM -0400, Simo Sorce wrote:
> On Wed, 2012-10-24 at 10:57 +0200, Sumit Bose wrote:
> > From 13ff428b862b90fcbe2eb697565854442d489042 Mon Sep 17 00:00:00 2001
> > From: Sumit Bose <sbose(a)redhat.com>
> > Date: Tue, 9 Oct 2012 21:16:41 +0200
> > Subject: [PATCH 02/18] s2n: save user principal name of sub-domain
> > user
> >
Jakub, Simo,
thank you for the review.
As you suggested I opened
https://fedorahosted.org/sssd/ticket/1603 to
add the upn to the data which is send to the PAC responder.
Additionally I removed the second patch which adds a constructed upn to
the cache during the lookup of a sub-domain user. We discussed this on
irc can came to the conclusion that there should be only data in the
cache which have a fair chance to be correct.
To compensate this I enhanced patch 'Add new call find_or_guess_upn' to
handle sub-domain users as well.
In the attaches new series the old second patch is remove and 'Add new
call find_or_guess_upn' and 'Use find_or_guess_upn() where needed' are
modified. All other patches are unchanged.
bye,
Sumit
Simo was so kind to run some additional tests which detect some issues
mostly related to character cases.
A new version of the patch series is attached with the following
changes:
0005: a name alias is added to allow case-insensitive searches
0014: two variables are explicitly initialised to make
krb5_unparse_name_ext() happy
Additionally there are two new patches because the content was not
suited to squash it into other patches:
0018: internally sssd should always use the canonical domain name even
if the user used the flat/short name at the login prompt
0019: makes sub-domains case-insensitive
For your convenience I push the patches on top of the master tree to my
fedorapeople repo
http://fedorapeople.org/cgit/sbose/public_git/sssd.git/log/?h=subdomain-p...
bye,
Sumit
>
>
> --
> Simo Sorce * Red Hat, Inc * New York
>
> _______________________________________________
> sssd-devel mailing list
> sssd-devel(a)lists.fedorahosted.org
>
https://lists.fedorahosted.org/mailman/listinfo/sssd-devel