OpenVPN Auth. per AD Group Membership
-
It's not a bug, I think the default behaviour is to match on OUs membership, not group membership.
It's good to hear it's working :-) -
huh,
I thought it could be a bug because of this post. Another user was looking for the same thing.
http://forum.pfsense.org/index.php/topic,25166.msg150474.html#msg150474
If not a bug, I think a feature request should be added.To me, not to be able to pick Group Membership leaves issues due to the different ways admin's configure AD.
With your patch, can an OU still be checked for users and not group? (did not test that)
(pretty much the way it was before) Just so i know.Again thanks. it is appreciated
:) -
No problem, the code checks if you are providing an OU DN or a group DN.
It also checks if you are providing a special known builtin container which is not an OU nor a group (eg: cn=users, cn=computers etc.). -
Juve,
Can you open a ticket on redmine with your patch?
It could be imported in to the base system if enough people test it out and say it works well, so long as it doesn't break other functionality.
-
I applied this patch and it worked great.
Thanks!
The issue that I still notice is that I can't nest groups within my openvpn-allowed group in Active Directory. When I add a user directly to the group, they are allowed to auth, but when they are in a group (say "Staff") that is in the openvpn-allowed group, they cannot.
This isn't a big issue for me, but it may affect some users with large domains.
Thanks again!
-
Recursive group membership checking is not yet included.
It could be added, I'll look as soon as possible on how to add this feature. This feature will be LDAP query intensive, since group nesting will require checking if group members are group objects and so on.@Jimp, the ticket is open and assigned to you.
-
Looks like we need a patch for that instead of a whole file.
And it will have to wait for 2.1 it looks like, but if you have a patch then others can always patch it in if they want the feature in the meantime.
-
jimp,
do you mean Recursive group membership checking as a patch for 2.1 or the original patch juve added?
thanks! -
The original code for AD group matching.
-
thanks jimp
not sure if you saw juve patch file on the first page.
he attached two files to the post. -
Yes but the file has changed quite a bit since then, not sure if it still applies cleanly. (It may, I haven't tried)
-
I updated to the latest snap to test the patch file.
I am not having any luck, but I am not a programmer. :)
The original auth.inc file that Juve posted does work in the new snap for open VPN, but not sure if anything else is broken since the file was changed allot. (As jimp noted)
Juve, I hate to ask but have you worked on this with new snaps? Please let me know if I can help.
Without group searches, I think this severely hurts the AD lookup function. (IMHO)
