[Freeipa-devel] [PATCH] 763 use full name for gecos

[Rob Crittenden]

Stephen Gallagher wrote:
> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA1
>
> On 04/04/2011 10:41 AM, Sumit Bose wrote:
>> On Mon, Apr 04, 2011 at 10:01:29AM -0400, Stephen Gallagher wrote:
>> On 04/04/2011 09:58 AM, Stephen Gallagher wrote:
>>>>> On 04/01/2011 06:14 PM, Rich Megginson wrote:
>>>>>> On 04/01/2011 02:17 PM, Rob Crittenden wrote:
>>>>>>> Stephen Gallagher wrote:
>>>>>>>> -----BEGIN PGP SIGNED MESSAGE-----
>>>>>>>> Hash: SHA1
>>>>>>>>
>>>>>>>> On 04/01/2011 03:55 PM, Rob Crittenden wrote:
>>>>>>>>> Use fullname for gecos instead of uid.
>>>>>>>>>
>>>>>>>>> ticket 1146
>>>>>>>>
>>>>>>>>
>>>>>>>> While it probably doesn't actually matter for these tests, I should note
>>>>>>>> that the GECOS attribute in LDAP is IA5String, not UTF8.
>>>>>>>>
>>>>>>>> Otherwise, Ack.
>>>>>>>
>>>>>>> It is defined using Directory String syntax by 389-ds. According to
>>>>>>> rfc 2307 it should be an IA5String, not sure why it isn't but we
>>>>>>> inherit this schema.
>>>>>> It is defined in the original rfc 2307 as Directory String, but 2307bis
>>>>>> redefined it to IA5String.
>>>>>
>>>>>
>>>>> Actually, I just looked at both RFC 2307 and RFC 2307bis. It's defined
>>>>> as IA5String in both.
>>>>>
>>>>>  From http://www.ietf.org/rfc/rfc2307.txt:
>>>>> 3. Attribute definitions
>>>>>
>>>>>     This section contains attribute definitions to be implemented by DUAs
>>>>>     supporting this schema.
>>>>> ...
>>>>>          ( nisSchema.1.2 NAME 'gecos'
>>>>>            DESC 'The GECOS field; the common name'
>>>>>            EQUALITY caseIgnoreIA5Match
>>>>>            SUBSTRINGS caseIgnoreIA5SubstringsMatch
>>>>>            SYNTAX 'IA5String' SINGLE-VALUE )
>>>>>
>>>>>
>>>>> And from http://www.padl.com/~lukeh/rfc2307bis.txt
>>>>> 3.   Attribute definitions
>>>>>
>>>>>       This section contains attribute definitions to be implemented by
>>>>>       DUAs supporting this schema.
>>>>> ...
>>>>>               ( 1.3.6.1.1.1.1.2 NAME 'gecos'
>>>>>                 DESC 'The GECOS field; the common name'
>>>>>                 EQUALITY caseIgnoreIA5Match
>>>>>                 SUBSTRINGS caseIgnoreIA5SubstringsMatch
>>>>>                 SYNTAX 1.3.6.1.4.1.1466.115.121.1.26
>>>>>                 SINGLE-VALUE )
>>>>>
>>
>>
>> Hmm, strange. http://bgp.potaroo.net/ietf/idref/draft-howard-rfc2307bis/
>> defines it as DirectoryString.
>>
>> So I have no idea which one is "more correct" for RFC 2307bis.
>>
>>> Look at the diffs in
>>> http://bgp.potaroo.net/ietf/idref/draft-howard-rfc2307bis/
>>
>>> There are currently -00, -01 and -02 of the draft.
>
>
> With that in mind, Ack.

Pushed to master and ipa-2-0

rob