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 method to revoke token #390

Merged
merged 3 commits into from
Apr 13, 2023
Merged

add method to revoke token #390

merged 3 commits into from
Apr 13, 2023

Conversation

hardyrarso
Copy link
Contributor

Add a method to send a request to the /openid-connect/revoke endpoint

Let me know if I need to change anything! Thank you for maintaining gocloak.

@Nerzal
Copy link
Owner

Nerzal commented Jan 9, 2023

Hi, if you merge the main branch the tests should be able to run now :)

@Nerzal
Copy link
Owner

Nerzal commented Jan 10, 2023

From the pipeline:

Warning: exported: exported type AuthorizationResponse should have comment or be unexported (revive)

Guess that lead to the failure. Does not look like you touched that, now i wonder, how it went through earlier

@Nerzal
Copy link
Owner

Nerzal commented Apr 13, 2023

I have resolved the issue in go.mod by myself.
Thank you for the contribution! :)

@Nerzal Nerzal merged commit aeb6613 into Nerzal:main Apr 13, 2023
@hardyrarso
Copy link
Contributor Author

Hello

Apologies, sorry for not responding to the comms

I opened this PR because my team needed the functionality at the time but priorities got shifted and I did not get back to you about this.

I'm glad this was able to be useful.

Thank you! And sorry again for going AWOL

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.

2 participants