[PATCH] samba-tool group show

[Björn Baumbach]

On 4/2/20 12:35 PM, Rowland penny wrote:
> On 02/04/2020 10:58, Björn Baumbach wrote:
>> Hi Rowland,
>>
>> On 4/2/20 10:44 AM, Rowland penny via samba-technical wrote:
>>> Hi, <samba-tool group show 'groupname'> will only show the groups info
>>> if it is a global security group.
>>>
>>> The attached patch fixes this.
>>>
>>> See bug: https://bugzilla.samba.org/show_bug.cgi?id=14335
>> Thank you for fixing this!
>> I wonder about the use of "objectCategory=group". I would expect the use
>> of "objectClass=group" instead. But I tried the patch and it works - I
>> did not try to understand why, yet. The objectCategory looks like
>> "objectCategory: CN=Group,CN=Schema,CN=Configuration,..."
>> I checked the samba-tool code and see that we typically use the
>> objectClass attribute.
>> Is there a special reason for the use of "objectCategory"?
> 
> From my investigations, this is the recommended Windows way of carrying
> out searches
> 
> For instance, you can use '(&(objectCategory=person)(objectClass=user))'
> to search for all users.

Thank you and Alexander very much for the explanation!

>> If "objectClass" is also fine, I would adapt the patch, add my RB and
> 
> Nope, I am fairly sure my way is the best way, so can I still have your
> RB ?
> 
> If so, can I also ask you to push it, I haven't a clue how to do this. ;-)

Yes, RB+ and pushed to autobuild. :-)

Best regards,
Björn

-- 
SerNet GmbH, Bahnhofsallee 1b, 37081 Göttingen
phone: 0551-370000-0, mailto:kontakt at sernet.de
Gesch.F.: Dr. Johannes Loxen und Reinhild Jung
AG Göttingen: HR-B 2816 - http://www.sernet.de

Samba eXPerience 2020 - online edition!
sponsored by Google, Microsoft & SerNet
May 26th-28th 2020, https://sambaXP.org