[Date Prev][Date Next] [Chronological] [Thread] [Top]

Re: (ITS#6487) Nssov pam_authz authorizedUserService



I like what this offers administrators but I have a comment about how you 
handle "wildcards". They aren't wild at all, but "magic strings" with only 
one possible meaning: all users/services with the single magic character 
"*". If you have a defined user naming convention like i_whatever for 
interns and c_whatever for contractors, it would be useful to be able to 
either include or exclude such users from using certain services.

My patch at http://www.openldap.org/its/index.cgi?findid=6495 does this for 
userhost and userservices attributes, and includes negation. Would you be 
interested in working with me to expand this to support real wildcards? 
Also I suggest you make this two patches, as the patch submission 
guidelines clearly state that one patch for 1 feature, and the meaty parts 
of this are obfuscated by increasing the buffer sizes which should probably 
be in a separate patch.

Regards, Kean