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

Allow public clients (e.g. SPAs using implicit flow or PKCE) to have redirect URLs other than localhost #1822

Merged
merged 4 commits into from
Nov 5, 2020
Merged

Allow public clients (e.g. SPAs using implicit flow or PKCE) to have redirect URLs other than localhost #1822

merged 4 commits into from
Nov 5, 2020

Commits on Nov 2, 2020

  1. Allow public clients (e.g. using implicit flow or PKCE) to have redir… 

    …ect URIs configured

Signed-off-by: Martin Heide <martin.heide@faro.com>
    @heidemn-faro
    heidemn-faro committed Nov 2, 2020
    Configuration menu
    Copy the full SHA
    b894d9c View commit details
    Browse the repository at this point in the history

  2. Fix gofmt in oauth2_test.go 

    Signed-off-by: Martin Heide <martin.heide@faro.com>
    @heidemn-faro
    heidemn-faro committed Nov 2, 2020
    Configuration menu
    Copy the full SHA
    1ea481b View commit details
    Browse the repository at this point in the history

  3. Add oob, device and localhost redirect URI tests 

    Signed-off-by: Martin Heide <martin.heide@faro.com>
    @heidemn-faro
    heidemn-faro committed Nov 2, 2020
    Configuration menu
    Copy the full SHA
    c15e288 View commit details
    Browse the repository at this point in the history

  4. No longer allow desktop/mobile redirect URIs implicitly if RedirectUR… 

    …Is is set

Signed-off-by: Martin Heide <martin.heide@faro.com>
    @heidemn-faro
    heidemn-faro committed Nov 2, 2020
    Configuration menu
    Copy the full SHA
    162073b View commit details
    Browse the repository at this point in the history