Skip to content
This repository has been archived by the owner on Dec 12, 2018. It is now read-only.

Delete AccessTokenAuthenticator access token #605

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Delete AccessTokenAuthenticator access token #605

wants to merge 3 commits into from

Conversation

the-overengineer
Copy link

Adds ability to direct tokens after running a StormpathAccessTokenAuthenticator.authenticate directly, by adding StormpathAccessTokenAuthenticatorResult.deleteAccessToken.

To implement this, Client.deleteResource was added and documented.

Fixes #603

Luka Skukan added 3 commits February 13, 2017 15:23
Add Client.deleteResource
d59053a 
Add ability to delete resources to the Client class directly.
Implement in same was as getResource and saveResource.
Add unit test for Client#deleteResource
e390152
Implement OAuth authentication result deleting
5437cbc 
Add functionality when the result of OAuth access token auth can
be deleted directly, without the need to fetch and construct a new
resource. Shave off several API calls.
@coveralls
Copy link

coveralls commented Feb 13, 2017 •
edited
Loading

Coverage Status

Coverage decreased (-0.05%) to 88.007% when pulling 5437cbc on feature/delete-oauth-result-token into 848654d on master.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
in progress
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@the-overengineer @coveralls