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 Support for re-authentication of the access-token with Google #12

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

Conversation

drbobbeaty
Copy link

We needed to have the ability to re-authenticate (renew) an access-token from an initial call to get-access-token when the token timed out. In order to do this, there needed to be a few additions to the endpoint map to hold the query parameters for Google to generate a refresh-token, and then we had to add the function to actually call the service to generate a new token.

At the same time, I added an example for Google usage, as that's what we're using, so that the next guy won't have to do all this work that I needed to do to figure out how to get Google OAuth2 working.

Bob Beaty added 3 commits February 25, 2015 14:00
We needed to be able to re-authenticate on Google, and that meant that we
needed to add several infrastructural changes to the endpoints as well as
adding in a new 'defmethod' for the refreshing, and then we need to add in
the top-level function for refreshing and update the docs. This is all
tested out, and works like a charm.
We don't need to check in any of these files, and they just clutter the git
status command. Just keeping things clean.
When we are doing a refresh of the OAuth2 token, we need to add it back
into the map to make sure that it's there and available for the next time
we might want to call this function. This was something I was doing in the
calling code, but it really belongs here as everyone is likely to need 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.

1 participant