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

Support for OAuth2 sessions revocation #515

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Louson
Copy link

@Louson Louson commented Jul 13, 2023

Add support for revocation in OAuth2 sessions

Suggestion for #337

Do you know how it's possible to test this ?

@Louson Louson force-pushed the louson/revocation branch 2 times, most recently from ee7e0af to f2c3c6e Compare July 13, 2023 12:52
@JonathanHuot
Copy link
Contributor

JonathanHuot commented Feb 25, 2024

This looks interesting, shall we add an example in docs & rebase ? I'll be happy to merge it.

@Louson
Copy link
Author

Louson commented Mar 19, 2024

I made an example with the gitlab API as I'm not sure the google API or the GitHub are compliant.

@Louson Louson marked this pull request as ready for review March 19, 2024 21:11
Add support for revocation in OAuth2 sessions.
Make an example with GitLab.
Documentation.

Signed-off-by: Louis Rannou <louis.rannou@syslinbit.com>
@jtroussard
Copy link
Contributor

@JonathanHuot would this PR be a good candidate to have the secrets stored? Should we get #538 tested and merged before this one? Then we can move the keys out of the source code?

@Louson
Copy link
Author

Louson commented May 28, 2024

Hello, any news on this ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants