[Freeipa-devel] [PATCHES] 106-113 Access raw LDAP values directly from LDAPEntry

[Jan Cholasta]

On 24.9.2013 15:35, Jan Cholasta wrote:
> On 27.2.2013 16:31, Jan Cholasta wrote:
>> Hi,
>>
>> these patches add the ability to access and manipulate raw attribute
>> values as they are returned from python-ldap to the LDAPEntry class.
>> This is useful for comparing entries, computing modlists for the modify
>> operation, deleting values that don't match the syntax of an attribute,
>> etc., because we don't need to care what syntax does a particular
>> attribute use, or what Python type is used for it in the framework (raw
>> values are always list of str). It should also make interaction with
>> non-389 DS LDAP servers easier in the framework.
>>
>> (It might be too late for this kind of changes to get into 3.2 now, I'm
>> posting these patches mainly so that you are aware that they exist.)
>>
>> Honza
>>
>
> This is now planned for 3.4: <https://fedorahosted.org/freeipa/ticket/3521>
>
> I fixed some issues in these patches and refined the API. Updated
> patches attached.
>
> Also added a patch to use raw values when adding new entries and a patch
> which refines LDAPEntry.single_value, so that it is consistent with the
> rest of the changes introduced in the patches.
>
> Patch 110 will probably be dropped in favor of Petr Viktorin's schema
> update patches, but I included it anyway.
>
> Incidentally, this also fixes
> <https://fedorahosted.org/freeipa/ticket/3927> and possibly also
> <https://fedorahosted.org/freeipa/ticket/2131>.
>
> Honza
>

Noticed a couple more issues and fixed them. Updated patches attached.

Honza

-- 
Jan Cholasta
-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-jcholast-106.3-Make-LDAPEntry-a-wrapper-around-dict-rather-than-a-d.patch
Type: text/x-patch
Size: 7272 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20130926/78962840/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-jcholast-107.3-Introduce-IPASimpleLDAPObject.decode-method-for-deco.patch
Type: text/x-patch
Size: 3308 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20130926/78962840/attachment-0001.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-jcholast-108.3-Always-use-lists-for-values-in-LDAPEntry-internally.patch
Type: text/x-patch
Size: 4001 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20130926/78962840/attachment-0002.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-jcholast-109.3-Decode-and-encode-attribute-values-in-LDAPEntry-on-d.patch
Type: text/x-patch
Size: 17310 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20130926/78962840/attachment-0003.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-jcholast-110.3-Make-sure-attributeTypes-updates-are-done-before-obj.patch
Type: text/x-patch
Size: 1088 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20130926/78962840/attachment-0004.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-jcholast-111.3-Remove-legacy-toDict-and-origDataDict-methods-of-LDA.patch
Type: text/x-patch
Size: 4878 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20130926/78962840/attachment-0005.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-jcholast-112.3-Store-encoded-attribute-values-from-search-results-d.patch
Type: text/x-patch
Size: 886 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20130926/78962840/attachment-0006.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-jcholast-113.3-Use-encoded-values-from-entry-objects-directly-when-.patch
Type: text/x-patch
Size: 3253 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20130926/78962840/attachment-0007.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-jcholast-121.3-Use-encoded-values-from-entry-objects-directly-when-.patch
Type: text/x-patch
Size: 1478 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20130926/78962840/attachment-0008.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-jcholast-169.3-Turn-LDAPEntry.single_value-into-a-dictionary-like-p.patch
Type: text/x-patch
Size: 49542 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20130926/78962840/attachment-0009.bin>