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

Already on GitHub? Sign in to your account

Changes in BaseFacebook#getUserAccessToken() #39

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
2 participants

ethanl commented Sep 24, 2012

Avoid a roundtrip to facebook.com if the code in the signed request is the same as what's in the persistent store.

avoid a roundtrip to facebook.com if the code in the signed request i…
…s the same as what's in the persistent store
Contributor

oyvindkinsey commented Dec 11, 2012

Hey ethanl - would you mind rebasing and re-request this pull?

ethanl commented Dec 12, 2012

Sorry for the late reply. But I believe you've committed a similar fix for this issue. Although my changes also include removing some duplicate code (L447-452 and L463-469).

@ethanl ethanl closed this Dec 12, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment