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

Add more authentication strategies #7

Open
CSharpRU opened this issue Aug 18, 2017 · 4 comments
Open

Add more authentication strategies #7

CSharpRU opened this issue Aug 18, 2017 · 4 comments

Comments

@CSharpRU
Copy link
Owner

The title says for itself :)

@CSharpRU CSharpRU added this to the 3.4.0 milestone Aug 18, 2017
@LavaToaster
Copy link

I'm trying to work out how to add in a normal token strategy, but I'm having a little trouble with it right now. (See this for why I need a token strategy)

From what I can work out, the auth strategies require a request to be made, and then an "auth" in the response.

I initially thought I could just make a request to the Token Lookup (Self) endpoint, however, you don't get this "auth" endpoint. I then thought about renewing the token, however, for root tokens or non-renewable tokens the endpoint just returns a failure.

So 🤷‍♀️, I'm not sure where to go from here without consulting you. (To be fair, I could have completely missed something.)

@CSharpRU
Copy link
Owner Author

CSharpRU commented Nov 2, 2017

@Lavoaster You don't need to do any request within your Token strategy, just return Auth object with token assigned to it. After you call authenticate on client with you Token strategy it will automatically call lookup self to get info about token.

@CSharpRU CSharpRU modified the milestones: 3.4.0, 3.5.0 Nov 2, 2017
@CSharpRU
Copy link
Owner Author

CSharpRU commented Nov 2, 2017

But also I suppose that you want to check you really have info about your token after this line (because you'll not do any request to Vault and there will be no response for check): https://github.com/CSharpRU/vault-php/blob/master/src/Client.php#L221

@CSharpRU CSharpRU removed this from the 3.5.0 milestone May 1, 2018
@hexa2k9
Copy link
Contributor

hexa2k9 commented Mar 30, 2020

This would bring - at least - LDAP, Okta, RADIUS Authentication Methods as well as Userpass.
All of them can be mounted on non-default path segments.

#32

@CSharpRU CSharpRU linked a pull request May 23, 2020 that will close this issue
@CSharpRU CSharpRU removed a link to a pull request May 23, 2020
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

3 participants