ory-am/hydra

View on GitHub
oauth2/handler_test.go

Summary

Maintainability
D
2 days
Test Coverage
// Copyright © 2022 Ory Corp
// SPDX-License-Identifier: Apache-2.0

package oauth2_test

import (
    "context"
    "encoding/json"
    "fmt"
    "io"
    "net/http"
    "net/http/httptest"
    "net/url"
    "strings"
    "testing"
    "time"

    hydra "github.com/ory/hydra-client-go/v2"

    "github.com/ory/x/httprouterx"

    "github.com/ory/x/snapshotx"

    "github.com/ory/x/contextx"

    "github.com/ory/hydra/v2/jwk"
    "github.com/ory/hydra/v2/x"

    "github.com/golang/mock/gomock"
    "github.com/pkg/errors"
    "github.com/stretchr/testify/assert"
    "github.com/stretchr/testify/require"

    "github.com/ory/hydra/v2/driver/config"
    "github.com/ory/hydra/v2/internal"

    "github.com/ory/fosite"
    "github.com/ory/fosite/handler/openid"
    "github.com/ory/fosite/token/jwt"
    "github.com/ory/hydra/v2/client"
    "github.com/ory/hydra/v2/oauth2"
)

var lifespan = time.Hour

func TestHandlerDeleteHandler(t *testing.T) {
    ctx := context.Background()
    conf := internal.NewConfigurationWithDefaults()
    conf.MustSet(ctx, config.KeyIssuerURL, "http://hydra.localhost")
    reg := internal.NewRegistryMemory(t, conf, &contextx.Default{})

    cm := reg.ClientManager()
    store := reg.OAuth2Storage()

    h := oauth2.NewHandler(reg, conf)

    deleteRequest := &fosite.Request{
        ID:             "del-1",
        RequestedAt:    time.Now().Round(time.Second),
        Client:         &client.Client{ID: "foobar"},
        RequestedScope: fosite.Arguments{"fa", "ba"},
        GrantedScope:   fosite.Arguments{"fa", "ba"},
        Form:           url.Values{"foo": []string{"bar", "baz"}},
        Session:        &oauth2.Session{DefaultSession: &openid.DefaultSession{Subject: "bar"}},
    }
    require.NoError(t, cm.CreateClient(context.Background(), deleteRequest.Client.(*client.Client)))
    require.NoError(t, store.CreateAccessTokenSession(context.Background(), deleteRequest.ID, deleteRequest))

    r := x.NewRouterAdmin(conf.AdminURL)
    h.SetRoutes(r, &httprouterx.RouterPublic{Router: r.Router}, func(h http.Handler) http.Handler {
        return h
    })
    ts := httptest.NewServer(r)
    defer ts.Close()

    c := hydra.NewAPIClient(hydra.NewConfiguration())
    c.GetConfig().Servers = hydra.ServerConfigurations{{URL: ts.URL}}

    _, err := c.
        OAuth2API.DeleteOAuth2Token(context.Background()).
        ClientId("foobar").Execute()
    require.NoError(t, err)

    ds := new(oauth2.Session)
    _, err = store.GetAccessTokenSession(ctx, "del-1", ds)
    require.Error(t, err, "not_found")
}

func TestUserinfo(t *testing.T) {
    ctx := context.Background()
    conf := internal.NewConfigurationWithDefaults()
    conf.MustSet(ctx, config.KeyScopeStrategy, "")
    conf.MustSet(ctx, config.KeyAuthCodeLifespan, lifespan)
    conf.MustSet(ctx, config.KeyIssuerURL, "http://hydra.localhost")
    reg := internal.NewRegistryMemory(t, conf, &contextx.Default{})
    internal.MustEnsureRegistryKeys(ctx, reg, x.OpenIDConnectKeyName)

    ctrl := gomock.NewController(t)
    op := NewMockOAuth2Provider(ctrl)
    defer ctrl.Finish()
    reg.WithOAuth2Provider(op)

    h := reg.OAuth2Handler()

    router := x.NewRouterAdmin(conf.AdminURL)
    h.SetRoutes(router, &httprouterx.RouterPublic{Router: router.Router}, func(h http.Handler) http.Handler {
        return h
    })
    ts := httptest.NewServer(router)
    defer ts.Close()

    for k, tc := range []struct {
        setup                func(t *testing.T)
        checkForSuccess      func(t *testing.T, body []byte)
        checkForUnauthorized func(t *testing.T, body []byte, header http.Header)
        expectStatusCode     int
    }{
        {
            setup: func(t *testing.T) {
                op.EXPECT().IntrospectToken(gomock.Any(), gomock.Eq("access-token"), gomock.Eq(fosite.AccessToken), gomock.Any()).Return(fosite.AccessToken, nil, errors.New("asdf"))
            },
            expectStatusCode: http.StatusInternalServerError,
        },
        {
            setup: func(t *testing.T) {
                op.EXPECT().
                    IntrospectToken(gomock.Any(), gomock.Eq("access-token"), gomock.Eq(fosite.AccessToken), gomock.Any()).
                    Return(fosite.RefreshToken, nil, nil)
            },
            checkForUnauthorized: func(t *testing.T, body []byte, headers http.Header) {
                assert.True(t, headers.Get("WWW-Authenticate") == `Bearer error="invalid_token",error_description="Only access tokens are allowed in the authorization header."`, "%s", headers)
            },
            expectStatusCode: http.StatusUnauthorized,
        },
        {
            setup: func(t *testing.T) {
                op.EXPECT().
                    IntrospectToken(gomock.Any(), gomock.Eq("access-token"), gomock.Eq(fosite.AccessToken), gomock.Any()).
                    Return(fosite.AccessToken, nil, fosite.ErrRequestUnauthorized)
            },
            checkForUnauthorized: func(t *testing.T, body []byte, headers http.Header) {
                assert.True(t, headers.Get("WWW-Authenticate") == `Bearer error="request_unauthorized",error_description="The request could not be authorized. Check that you provided valid credentials in the right format."`, "%s", headers)
            },
            expectStatusCode: http.StatusUnauthorized,
        },
        {
            setup: func(t *testing.T) {
                op.EXPECT().
                    IntrospectToken(gomock.Any(), gomock.Eq("access-token"), gomock.Eq(fosite.AccessToken), gomock.Any()).
                    DoAndReturn(func(_ context.Context, _ string, _ fosite.TokenType, _ fosite.Session, _ ...string) (fosite.TokenType, fosite.AccessRequester, error) {
                        session := &oauth2.Session{
                            DefaultSession: &openid.DefaultSession{
                                Claims: &jwt.IDTokenClaims{
                                    Subject: "alice",
                                },
                                Headers: new(jwt.Headers),
                                Subject: "alice",
                            },
                            Extra: map[string]interface{}{},
                        }

                        return fosite.AccessToken, &fosite.AccessRequest{
                            Request: fosite.Request{
                                Client: &client.Client{
                                    ID: "foobar",
                                },
                                Session: session,
                            },
                        }, nil
                    })
            },
            expectStatusCode: http.StatusOK,
            checkForSuccess: func(t *testing.T, body []byte) {
                bodyString := string(body)
                assert.True(t, strings.Contains(bodyString, `"sub":"alice"`), "%s", body)
                assert.True(t, strings.Contains(bodyString, `"aud":["foobar"]`), "%s", body)
            },
        },
        {
            setup: func(t *testing.T) {
                op.EXPECT().
                    IntrospectToken(gomock.Any(), gomock.Eq("access-token"), gomock.Eq(fosite.AccessToken), gomock.Any()).
                    DoAndReturn(func(_ context.Context, _ string, _ fosite.TokenType, _ fosite.Session, _ ...string) (fosite.TokenType, fosite.AccessRequester, error) {
                        session := &oauth2.Session{
                            DefaultSession: &openid.DefaultSession{
                                Claims: &jwt.IDTokenClaims{
                                    Subject:  "another-alice",
                                    Audience: []string{"something-else"},
                                },
                                Headers: new(jwt.Headers),
                                Subject: "alice",
                            },
                            Extra: map[string]interface{}{},
                        }

                        return fosite.AccessToken, &fosite.AccessRequest{
                            Request: fosite.Request{
                                Client: &client.Client{
                                    ID: "foobar",
                                },
                                Session: session,
                            },
                        }, nil
                    })
            },
            expectStatusCode: http.StatusOK,
            checkForSuccess: func(t *testing.T, body []byte) {
                bodyString := string(body)
                assert.False(t, strings.Contains(bodyString, `"sub":"alice"`), "%s", body)
                assert.True(t, strings.Contains(bodyString, `"sub":"another-alice"`), "%s", body)
                assert.True(t, strings.Contains(bodyString, `"aud":["something-else","foobar"]`), "%s", body)
            },
        },
        {
            setup: func(t *testing.T) {
                op.EXPECT().
                    IntrospectToken(gomock.Any(), gomock.Eq("access-token"), gomock.Eq(fosite.AccessToken), gomock.Any()).
                    DoAndReturn(func(_ context.Context, _ string, _ fosite.TokenType, _ fosite.Session, _ ...string) (fosite.TokenType, fosite.AccessRequester, error) {
                        session := &oauth2.Session{
                            DefaultSession: &openid.DefaultSession{
                                Claims: &jwt.IDTokenClaims{
                                    Subject:  "alice",
                                    Audience: []string{"foobar"},
                                },
                                Headers: new(jwt.Headers),
                                Subject: "alice",
                            },
                            Extra: map[string]interface{}{},
                        }

                        return fosite.AccessToken, &fosite.AccessRequest{
                            Request: fosite.Request{
                                Client: &client.Client{
                                    ID:                        "foobar",
                                    UserinfoSignedResponseAlg: "none",
                                },
                                Session: session,
                            },
                        }, nil
                    })
            },
            expectStatusCode: http.StatusOK,
            checkForSuccess: func(t *testing.T, body []byte) {
                bodyString := string(body)
                assert.True(t, strings.Contains(bodyString, `"sub":"alice"`), "%s", body)
                assert.True(t, strings.Contains(bodyString, `"aud":["foobar"]`), "%s", body)
            },
        },
        {
            setup: func(t *testing.T) {
                op.EXPECT().
                    IntrospectToken(gomock.Any(), gomock.Eq("access-token"), gomock.Eq(fosite.AccessToken), gomock.Any()).
                    DoAndReturn(func(_ context.Context, _ string, _ fosite.TokenType, _ fosite.Session, _ ...string) (fosite.TokenType, fosite.AccessRequester, error) {
                        session := &oauth2.Session{
                            DefaultSession: &openid.DefaultSession{
                                Claims: &jwt.IDTokenClaims{
                                    Subject: "alice",
                                },
                                Headers: new(jwt.Headers),
                                Subject: "alice",
                            },
                            Extra: map[string]interface{}{},
                        }

                        return fosite.AccessToken, &fosite.AccessRequest{
                            Request: fosite.Request{
                                Client: &client.Client{
                                    UserinfoSignedResponseAlg: "asdfasdf",
                                },
                                Session: session,
                            },
                        }, nil
                    })
            },
            expectStatusCode: http.StatusInternalServerError,
        },
        {
            setup: func(t *testing.T) {
                op.EXPECT().
                    IntrospectToken(gomock.Any(), gomock.Eq("access-token"), gomock.Eq(fosite.AccessToken), gomock.Any()).
                    DoAndReturn(func(_ context.Context, _ string, _ fosite.TokenType, _ fosite.Session, _ ...string) (fosite.TokenType, fosite.AccessRequester, error) {
                        session := &oauth2.Session{
                            DefaultSession: &openid.DefaultSession{
                                Claims: &jwt.IDTokenClaims{
                                    Subject: "alice",
                                },
                                Headers: new(jwt.Headers),
                                Subject: "alice",
                            },
                            Extra: map[string]interface{}{},
                        }

                        return fosite.AccessToken, &fosite.AccessRequest{
                            Request: fosite.Request{
                                Client: &client.Client{
                                    ID:                        "foobar-client",
                                    UserinfoSignedResponseAlg: "RS256",
                                },
                                Session: session,
                            },
                        }, nil
                    })
            },
            expectStatusCode: http.StatusOK,
            checkForSuccess: func(t *testing.T, body []byte) {
                claims, err := jwt.Parse(string(body), func(token *jwt.Token) (interface{}, error) {
                    keys, err := reg.KeyManager().GetKeySet(context.Background(), x.OpenIDConnectKeyName)
                    require.NoError(t, err)
                    t.Logf("%+v", keys)
                    key, _ := jwk.FindPublicKey(keys)
                    return key.Key, nil
                })
                require.NoError(t, err)
                assert.EqualValues(t, "alice", claims.Claims["sub"])
                assert.EqualValues(t, []interface{}{"foobar-client"}, claims.Claims["aud"], "%#v", claims.Claims)
                assert.NotEmpty(t, claims.Claims["jti"])
            },
        },
    } {
        t.Run(fmt.Sprintf("case=%d", k), func(t *testing.T) {
            tc.setup(t)

            req, err := http.NewRequest("GET", ts.URL+"/userinfo", nil)
            require.NoError(t, err)
            req.Header.Set("Authorization", "Bearer access-token")
            resp, err := http.DefaultClient.Do(req)
            require.NoError(t, err)
            defer resp.Body.Close()
            require.EqualValues(t, tc.expectStatusCode, resp.StatusCode)
            body, err := io.ReadAll(resp.Body)
            require.NoError(t, err)
            if tc.expectStatusCode == http.StatusOK {
                tc.checkForSuccess(t, body)
            } else if tc.expectStatusCode == http.StatusUnauthorized {
                tc.checkForUnauthorized(t, body, resp.Header)
            }
        })
    }
}

func TestHandlerWellKnown(t *testing.T) {
    ctx := context.Background()
    conf := internal.NewConfigurationWithDefaults()
    t.Run(fmt.Sprintf("hsm_enabled=%v", conf.HSMEnabled()), func(t *testing.T) {
        conf.MustSet(ctx, config.KeyScopeStrategy, "DEPRECATED_HIERARCHICAL_SCOPE_STRATEGY")
        conf.MustSet(ctx, config.KeyIssuerURL, "http://hydra.localhost")
        conf.MustSet(ctx, config.KeySubjectTypesSupported, []string{"pairwise", "public"})
        conf.MustSet(ctx, config.KeyOIDCDiscoverySupportedClaims, []string{"sub"})
        conf.MustSet(ctx, config.KeyOAuth2ClientRegistrationURL, "http://client-register/registration")
        conf.MustSet(ctx, config.KeyOIDCDiscoveryUserinfoEndpoint, "/userinfo")
        reg := internal.NewRegistryMemory(t, conf, &contextx.Default{})

        h := oauth2.NewHandler(reg, conf)

        r := x.NewRouterAdmin(conf.AdminURL)
        h.SetRoutes(r, &httprouterx.RouterPublic{Router: r.Router}, func(h http.Handler) http.Handler {
            return h
        })
        ts := httptest.NewServer(r)
        defer ts.Close()

        res, err := http.Get(ts.URL + "/.well-known/openid-configuration")
        require.NoError(t, err)
        defer res.Body.Close()

        var wellKnownResp hydra.OidcConfiguration
        err = json.NewDecoder(res.Body).Decode(&wellKnownResp)
        require.NoError(t, err, "problem decoding wellknown json response: %+v", err)
        snapshotx.SnapshotT(t, wellKnownResp)
    })
}