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 functionality to revoke tokens #150
base: master
Are you sure you want to change the base?
Conversation
neumayer
commented
Mar 29, 2017
- Add revoke method to provider interface
- Implement method for github and gplus
- Add tests
- Add noop implementation to other providers
* Add revoke method to provider interface * Implement method for github and gplus * Add tests * Add noop implementation to other providers
Anyone interested? |
@neumayer my apologies for not looking at this sooner. I think this is fantastic, and definitely wanted. Please continue with the implementations, we need implementations for all the providers that support it before it can be merged. Thanks. |
Hehe, that includes me getting accounts with up to 32 identity providers, so it might take a couple of days :-) But I'll try to keep working on it when I find the time. |
I looked at both twitch and amazon today. While I hunted down some secret amazon api endpoint, there's no documentation for it and I didn't manage to authenticate to it. Since it's not documented at all I think it's not worth pursuing that. I added a short comment about this. |
…unction getProviderName
… provder is not string
…unction getProviderName
… provder is not string
…session Provider name from session
* Add revoke method to provider interface * Implement method for github and gplus * Add tests * Add noop implementation to other providers
It's new in master.
This is needed because of changes to handling multiple logins introduced in 9885e53.