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 refresh_token_request compliance hook #214

Conversation

jimcortez
Copy link

This hook is to be able to modify refresh token requests. It is useful when doing automatic refreshes.

Also added example for the fitbit developer api, which requires a Authorization header at refresh time.

@jimcortez
Copy link
Author

This should address issue #182

@Lukasa
Copy link
Member

Lukasa commented Dec 20, 2015

Do you really mean #128?

@jimcortez
Copy link
Author

@Lukasa no, I meant 182. 😊

edited.

@sigmavirus24
Copy link
Contributor

This has merge conflicts that need to be resolved.

This hook is to be able to modify refresh token requests. It is useful when doing automatic refreshes.

Also added example for the fitbit developer api, which requires a Authorization header at refresh time.
@jimcortez jimcortez force-pushed the refresh_token_request_compliance_hook branch from 21f848e to 7cafa8b Compare December 30, 2015 23:26
@singingwolfboy
Copy link
Member

Looks like the Fitbit compliance fix was merged in #261. As a result, I'm closing this pull request.

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.

None yet

4 participants