-
Notifications
You must be signed in to change notification settings - Fork 8
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
Additional Keystone Strategies #1
Comments
:: shakes dust off project :: Thanks for the detailed issue! Originally just needed something to plug and play with an express web app. So many assumptions, glad you still checked this out and gave feedback.
These items are a great proposal for a v1 release and in general a solid direction for this project. I added you as a collaborator if you've got some of these ideas to push to a branch. Also, wouldn't mind pushing this somewhere a bit more visible like github.com/rackerlabs. just give me a 👍 and I'll move it over. |
It exposes the service catalog, the current user, and information about the token.
It probably won't be in my first pass of work anyway since we don't actually need it for my project, so we can defer the decision. |
Hi |
When attempting to use this library with on a few Rackspace projects, I realized the provided strategy is really intended for a web app where the username and password are provided on a login screen and the result of authentication is stored in the session. While this works great for a web app with a UI, it doesn't translate to building an API where an auth token is provided and must be validated using the Keystone API. In order to allow this library to work well with APIs, I'm proposing the following changes to this library:
Strategy
toAuthenticationStrategy
that corresponds to Token AuthenticationAuthenticationStrategy
that corresponds to Example 4.57.TokenValidationStrategy
that corresponds to the token validation call in the admin API.Additionally, we might want some additional changes in the future:
The text was updated successfully, but these errors were encountered: