[Freeipa-devel] [PATCH 0050] ACI plugin: correctly parse bind rules enclosed in parentheses

Martin Babinsky mbabinsk at redhat.com
Tue Jul 28 11:33:57 UTC 2015


On 07/27/2015 05:10 PM, Martin Basti wrote:
> On 23/07/15 16:06, Martin Babinsky wrote:
>> This is a quick fix for https://fedorahosted.org/freeipa/ticket/5037
>>
>>
>>
> NACK
>
> I do not like your change in first regexp too much.
>
> Can you try this instead?
>
> PermPat = re.compile(r'(\w+)\s*\(([^()]*)\)\s*(.*)', re.UNICODE)
>
> This just removes '(' and ') ' from pattern and accept all other characters.
>
> --
> Martin Basti
>

Attaching updated patch.

-- 
Martin^3 Babinsky
-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-mbabinsk-0050.1-ACI-plugin-correctly-parse-bind-rules-enclosed-in-pa.patch
Type: text/x-patch
Size: 1636 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20150728/af92798e/attachment.bin>


More information about the Freeipa-devel mailing list