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.

Sign up for GitHub

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

Jump to bottom

Add OAuth support for authentication #27

Open
pjeli opened this issue Jun 5, 2018 · 4 comments
Open

Add OAuth support for authentication #27

pjeli opened this issue Jun 5, 2018 · 4 comments
Labels
enhancement New feature or request security Change affects security

Comments

@pjeli
Copy link
Collaborator

pjeli commented Jun 5, 2018 •
edited
Loading

Today NNA only supports authentication support for LDAP.

It would be nice to add OAuth protocol support for those running SSO (Single-Sign-On).

pac4j should support it: http://www.pac4j.org/docs/clients/oauth.html
@pjeli pjeli added enhancement New feature or request security Change affects security labels Jun 5, 2018
@pjeli pjeli changed the title Add Auth0 support for authentication Add OAuth support for authentication Jun 27, 2018
@pjeli pjeli self-assigned this Jul 12, 2018
@pjeli pjeli added this to the Initial Release milestone Jul 12, 2018
@pjeli
Copy link
Collaborator Author

pjeli commented Jul 12, 2018

Support for this was requested.

pjeli added a commit that referenced this issue Jul 12, 2018
@pjeli
[#27] Add OAuth support for authentication
05ab5fc
@pjeli
Copy link
Collaborator Author

pjeli commented Jul 12, 2018

I've opened up a branch to work on this:git checkout oauth.
Need to test.

@pjeli pjeli mentioned this issue Jul 13, 2018
Closed
pjeli added a commit that referenced this issue Jul 20, 2018
@pjeli
[#27] Add OAuth support for authentication
19ca5ca
@pauljv92
Copy link
Contributor

pauljv92 commented Aug 9, 2018

what is the process for testing this branch? @Zero45

@pjeli pjeli removed their assignment Aug 9, 2018
@pjeli
Copy link
Collaborator Author

pjeli commented Aug 9, 2018

This branch is old now. I wasn't able to test it. Might be worthwhile to start over on this one.

@pjeli pjeli removed this from the Initial Release milestone Aug 31, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request security Change affects security
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@pjeli @pauljv92