On 06/23/2014 10:01 AM, Martin Kosek wrote:
On 06/20/2014 02:14 PM, Petr Viktorin wrote:
My patch 0580 was wrong; non-POSIX groups obviously lack the posixgroup
objectclass. Actually the only objectclasses that all groups share are top and
ipaobject.

This makes permission plugin & updater join multiple
permission_filter_objectclasses filters with OR, and changes the --type group
filter to (|(objectclass=ipausergroup)(objectclass=posixgroup)).

The first patch has some unrelated test fixes.

This looks as the right thing to do, thanks. Access to group was restored,
group type also worked fine.

I found just one typo in 593 - "Retreive". When you fix this, it's an ACK.

Thanks for the review!
Typo fixed; pushed to master: ac8539bd344f2309f74efc6b42bddb3a925ff20f


--
PetrĀ³

_______________________________________________
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel

Reply via email to