On 12.08.2016 08:29, Jan Cholasta wrote:
On 11.8.2016 19:43, Martin Basti wrote:
On 11.08.2016 16:09, Jan Cholasta wrote:
On 11.8.2016 14:27, Martin Basti wrote:
On 01.08.2016 10:27, Jan Cholasta wrote:
On 1.8.2016 10:19, Jan Cholasta wrote:
the attached patches fix
Self-NACK, proper patches attached.
IMHO this is caused by your patches, test_cert_plugin.py:
Updated and rebased patches attached.
It works for me, but:
Is this py2/3 compatible?
ra_obj = ra.get_certificate(str(serial_number))
I don't see why not. Do you have any particular incompatibility in mind?
Because there is function str() used, where result is unicode in py3 but
not in py2
Are you sure you need tuple() here?
+ for key in tuple(six.iterkeys(result)):
Yes, I'm modifying `result` inside the loop.
I don't need the six.iterkeys() though.
sorry, I overlooked that.
if cert is not None:
filter = ldap.make_filter_from_attr('usercertificate',
Variable "value" may be referenced before assignment
Right, it should be `cert`, not `value`.
I haven't tested performace improvements yet, and it is quite big change
so I will continue with testing tomorrow.
Manage your subscription for the Freeipa-devel mailing list:
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code