mdarifmustafa / vt-middleware

Automatically exported from code.google.com/p/vt-middleware
0 stars 0 forks source link

vt-ldap: ppolicy JNDI implementation #121

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
Provide a password policy control implementation for JNDI using the Bouncy 
Castle libraries.

  * Extend JndiControlHandler to provide this functionality
  * Comment on the design/extensibility of the ControlHandler API

Implementation can be attached to this issue, since we don't yet have a space 
for extensions or contrib.

Original issue reported on code.google.com by dfis...@gmail.com on 20 Oct 2011 at 6:38

GoogleCodeExporter commented 8 years ago

Original comment by marvin.addison@gmail.com on 7 Nov 2011 at 6:13

GoogleCodeExporter commented 8 years ago

Original comment by marvin.addison@gmail.com on 7 Nov 2011 at 6:45

GoogleCodeExporter commented 8 years ago
Link to ppolicy draft:
http://opends.java.net/public/standards/draft-behera-ldap-password-policy.txt

Original comment by dfis...@gmail.com on 7 Nov 2011 at 8:10

GoogleCodeExporter commented 8 years ago
Attached source for implementation that has been developed and tested against 
the ppolicy response controls provided by OpenLDAP.  The specific test cases 
are listed in JndiPasswordPolicyControlProcessorTest.  I'd appreciate feedback 
on the DER parsing strategy.  I think there are some novel ideas in there that 
may be generally beneficial for parsing DER types over much broader use cases.

Original comment by marvin.addison@gmail.com on 17 Nov 2011 at 5:02

Attachments:

GoogleCodeExporter commented 8 years ago
Patch committed in r2167.
Please review the javadocs, as I completed the documentation in several cases.

I like the DER parsing strategy. It seems easily extensible to other controls 
and I'm hopeful that all the supported controls can be handled in this fashion. 
It would reduce the complexity of the provider packages.

Original comment by dfis...@gmail.com on 21 Nov 2011 at 8:05

GoogleCodeExporter commented 8 years ago
I reviewed the vt-ldap commit that merged this in and javadocs look good.

Original comment by marvin.addison@gmail.com on 22 Nov 2011 at 1:05