Closed patter001 closed 8 years ago
Sorry, I totally missed this PR.
If it is still relevant to you, then please add a test. I'll also add a small comment on the commit now.
Thanks, it is ok. I ended up having to do more customization due to how my company uses nested groups in LDAP so the PR code got me 95% of the way there, but the last 5% (not in the PR) was something that wouldn't make sense for most folks. I will close.
In our environment, the resulting groupname contains complete LDAP information to the group objects. It would be nice to filter that extra information out using a regular expression. This would allow for more meaningful groupnames in devpi.