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

oauth2server: token revocation endpoint #53

Open
2 tasks
jirikuncar opened this issue Jul 4, 2016 · 3 comments
Open
2 tasks

oauth2server: token revocation endpoint #53

jirikuncar opened this issue Jul 4, 2016 · 3 comments

Comments

@jirikuncar
Copy link
Member

jirikuncar commented Jul 4, 2016

@lnielsen commented on Tue Aug 19 2014

Flask-OAuthlib v0.7 (soon to be release) has added support for token revocation. This can rather easily be added to the current implementation.

@jirikuncar jirikuncar added this to the v1.0.0 milestone Oct 26, 2016
jirikuncar added a commit to jirikuncar/invenio-oauth2server that referenced this issue Oct 26, 2016
* Adds token revocation endpoint.  (closes inveniosoftware#53)

Signed-off-by: Jiri Kuncar <[email protected]>
@lnielsen lnielsen modified the milestones: v1.0.0, someday Feb 13, 2017
@slint
Copy link
Member

slint commented Feb 20, 2017

Are there any updates on this? Considering that the .revoke_token endpoint name is used, currently the settings view if a user has Authorized Applications is kind of broken.

@krzysztof
Copy link
Member

@slint It seems that the rendering now doesn't crash (both master of this repo as well as zenodo sandbox).

I can also revoke token using our endpoint - I guess this is complete w.r.t. functionality?

@slint
Copy link
Member

slint commented Jun 19, 2017

The endpoint that was introduced with e728a25 is not the OAuth2 spec implementation that is mentioned in this issue and provided by Flask-OAuthlib. There's still a need to implemented this in invenio_oauth2server.views.server.

(My comment was wrongly placed in this issue, since it's about the "internally" revoking Authorized Application tokens)

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

4 participants