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

Enabled to run in-cluster, added Expiry for Tokens, updated to go 1.9 etc. #6

Open
wants to merge 47 commits into
base: master
Choose a base branch
from

Conversation

christianhuening
Copy link

Hi there,
I changed the service to be able to run inside of Kubernetes and serve the /ldapAuth endpoint separately from the /authenticate endpoint.
Other changes:

  • Uses RSASSA-PSS (SHA512) for token signing with 4096 bit tokens
  • Features 'exp' field in tokens and invalidates tokens after 12h
  • Updated to Go 1.9
  • Switched to Go Dep as package manager
  • Added Deployment instructions

ToDo:

  • Make EXP configurable

Christian Hüning added 30 commits July 17, 2017 11:49
@dnascimento
Copy link

dnascimento commented Nov 29, 2017

@christianhuening Kudos for the great work, exactly what I was starting to change in the project.

@alexbrand Can review the pull request and merge? It needs a considerable review :(

@christianhuening
Copy link
Author

@dnascimento Thanks!
We're using this service to provide login for ~500 student users to our cluster atm. Working great so far.

@christianhuening
Copy link
Author

I made a commit which enables the service to be replicated for improved performance and H/A

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

Successfully merging this pull request may close these issues.

3 participants