Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

LDAP plugin spec #965

Closed
shashiranjan84 opened this issue Feb 8, 2016 · 3 comments
Closed

LDAP plugin spec #965

shashiranjan84 opened this issue Feb 8, 2016 · 3 comments

Comments

@shashiranjan84
Copy link
Contributor

shashiranjan84 commented Feb 8, 2016

This ticket is to track LDAP Plugin(#329) implementation strategy.

So far I come up with following spec

  • Plugin must parse the Authorization or Proxy-Authorization header to retrieve the encrypted signature
    ex. Authorization: ldap username:password
  • It must look for user in cache, if present it must create sha1 digest using username and stored password and validate it against user sent signature
  • If user missing in cache, it must be validated using either one of following strategies based on configuration
    • Bind authentication, authentication done directly by the LDAP server (seems like industry standard)
    • Password comparison, where the password supplied by the user is compared with the one stored in the repository
  • Plugin must cache the credential to save future trip to ldap server for same user unless caching is disabled
  • Must provide a path to invalidate an user from cache anytime
  • Must support encrypted communication between Kong and #LDAP server
@shashiranjan84
Copy link
Contributor Author

@ahmadnassri can you check with customer the use case for this plugin like how they generally use LDAP to authenticate user and what strategy they use to authenticate client (here Kong) connection to LDAP server?

@subnetmarco
Copy link
Member

@shashiranjan84 do you confirm it's closed with #1133? If so, let's close this.

@ahmadnassri ahmadnassri added the BC label May 13, 2016
@shashiranjan84
Copy link
Contributor Author

@thefosk Yes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants