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

User Authentication #109

Closed
mark-velez opened this issue Jun 16, 2016 · 4 comments
Closed

User Authentication #109

mark-velez opened this issue Jun 16, 2016 · 4 comments
Milestone

Comments

@mark-velez
Copy link
Contributor

mark-velez commented Jun 16, 2016

As I mentioned here, Columbia may have more immediate needs for authentication. Specifically, I foresee us wanting authentication via LDAP to access WebAPI through Atlas. I am limiting the scope of this issue to authentication. If we are to implement a solution however, we don't want to preclude it from being more generally useful, becoming more robust. I'm listing here related objectives that have been mentioned so we keep this in sight:

  • Limit deployment overhead (Don't complicate WebAPI/Atlas install)
  • Allow for integration with existing infrastructure (AD/LDAP, OAuth providers)
  • If storing sensitive user information, take special care regarding sql injection
  • Make way for some flavor of authorization
  • Auditing

@fdefalco has mentioned Shiro as a possible framework and it appears that some efforts have been made already.

@mark-velez
Copy link
Contributor Author

mark-velez commented Jun 16, 2016

There is a pull request from @ganisimov. Has anyone reviewed this and do we intend for the shiro branch to be merged to master?

@t-abdul-basser
Copy link
Contributor

Related to issue #105 (Shiro PR)

@fdefalco
Copy link
Contributor

fdefalco commented Jun 16, 2016

The work done by @ganisimov builds off of some earlier work that had been done by @GlebMalikov. It isn't ready for integration yet however progress is being made towards its completion.

@anthonysena
Copy link
Collaborator

Tying to Atlas OHDSI/Atlas#86

@chrisknoll chrisknoll added this to the V2.0.0 milestone Jan 11, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants