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.

Sign up for GitHub

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

Jump to bottom

Handle OAuth responses as querystring as well as JSON on Android #171

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

Handle OAuth responses as querystring as well as JSON on Android #171

wants to merge 4 commits into from

Commits on Sep 12, 2017

  1. Update OAuthManager.h

    @exentrich
    exentrich authored Sep 12, 2017
    Configuration menu
    Copy the full SHA
    f727a7c View commit details
    Browse the repository at this point in the history

Commits on Sep 27, 2017

  1. Update build.gradle

    @exentrich
    exentrich authored Sep 27, 2017
    Configuration menu
    Copy the full SHA
    470ffaa View commit details
    Browse the repository at this point in the history

Commits on Oct 16, 2017

  1. Twitter, for example, returns OAuth token data as a querystring 

    (x-www-form-urlencoded), not as JSON, so signing in breaks as per
fullstackreact#165.
Correct this by catching the JSON parse error and falling back to
extracting the parameters we need from the token instead.
    @stuartlangridge
    stuartlangridge committed Oct 16, 2017
    Configuration menu
    Copy the full SHA
    c0af6d8 View commit details
    Browse the repository at this point in the history

Commits on Oct 18, 2017

  1. Merge pull request #1 from exentrich/master 

    Merge in OAuthManager.h iOS fixes
    @stuartlangridge
    stuartlangridge authored Oct 18, 2017
    Configuration menu
    Copy the full SHA
    298727a View commit details
    Browse the repository at this point in the history