Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add Revoke Refresh Token endpoint #170

Merged
merged 2 commits into from
Nov 27, 2018
Merged

Add Revoke Refresh Token endpoint #170

merged 2 commits into from
Nov 27, 2018

Conversation

lbalmaceda
Copy link
Contributor

Closes #115

@lbalmaceda lbalmaceda added this to the v3-Next milestone Nov 23, 2018
joshcanhelp
joshcanhelp previously approved these changes Nov 26, 2018
Copy link
Contributor

@joshcanhelp joshcanhelp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nothing holding back approval here but a few things ... will approve, make any changes you see fit.

been issued for the same user, application, and audience will be revoked.

Args:
client_id (str): your application's client Id
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Consider:

The Client ID for your Application

Args:
client_id (str): your application's client Id

token (str): the Refresh Token you want to revoke
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"The ..."


token (str): the Refresh Token you want to revoke

client_secret (str, optional): Your application's Client Secret.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Consider:

The Client Secret for your Application

token (str): the Refresh Token you want to revoke

client_secret (str, optional): Your application's Client Secret.
Required for confidential applications.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is confusing ... what is a "confidential application?" This doesn't tell me much about when I should be using this.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's explained like that in the api explorer linked. I'll add a more explicit link

@lbalmaceda lbalmaceda merged commit dc43a2b into master Nov 27, 2018
@lbalmaceda lbalmaceda deleted the add-revoke-token branch November 27, 2018 18:14
@lbalmaceda lbalmaceda modified the milestones: v3-Next, 3.5.0 Dec 27, 2018
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants