mirror of https://github.com/portainer/portainer
You can not select more than 25 topics
Topics must start with a letter or number, can include dashes ('-') and can be up to 35 characters long.
335 lines
9.3 KiB
335 lines
9.3 KiB
package security
|
|
|
|
import (
|
|
"fmt"
|
|
"net/http"
|
|
"net/http/httptest"
|
|
"testing"
|
|
|
|
portainer "github.com/portainer/portainer/api"
|
|
"github.com/portainer/portainer/api/apikey"
|
|
"github.com/portainer/portainer/api/dataservices"
|
|
"github.com/portainer/portainer/api/datastore"
|
|
httperrors "github.com/portainer/portainer/api/http/errors"
|
|
"github.com/portainer/portainer/api/jwt"
|
|
|
|
"github.com/stretchr/testify/assert"
|
|
)
|
|
|
|
// testHandler200 is a simple handler which returns HTTP status 200 OK
|
|
var testHandler200 = http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
|
|
w.WriteHeader(http.StatusOK)
|
|
})
|
|
|
|
func tokenLookupSucceed(dataStore dataservices.DataStore, jwtService dataservices.JWTService) tokenLookup {
|
|
return func(r *http.Request) *portainer.TokenData {
|
|
uid := portainer.UserID(1)
|
|
dataStore.User().Create(&portainer.User{ID: uid})
|
|
jwtService.GenerateToken(&portainer.TokenData{ID: uid})
|
|
return &portainer.TokenData{ID: 1}
|
|
}
|
|
}
|
|
|
|
func tokenLookupFail(r *http.Request) *portainer.TokenData {
|
|
return nil
|
|
}
|
|
|
|
func Test_mwAuthenticateFirst(t *testing.T) {
|
|
is := assert.New(t)
|
|
|
|
_, store := datastore.MustNewTestStore(t, true, true)
|
|
|
|
jwtService, err := jwt.NewService("1h", store)
|
|
assert.NoError(t, err, "failed to create a copy of service")
|
|
|
|
apiKeyService := apikey.NewAPIKeyService(nil, nil)
|
|
|
|
bouncer := NewRequestBouncer(store, jwtService, apiKeyService)
|
|
|
|
tests := []struct {
|
|
name string
|
|
verificationMiddlwares []tokenLookup
|
|
wantStatusCode int
|
|
}{
|
|
{
|
|
name: "mwAuthenticateFirst middleware passes with no middleware",
|
|
verificationMiddlwares: nil,
|
|
wantStatusCode: http.StatusUnauthorized,
|
|
},
|
|
{
|
|
name: "mwAuthenticateFirst middleware succeeds with passing middleware",
|
|
verificationMiddlwares: []tokenLookup{
|
|
tokenLookupSucceed(store, jwtService),
|
|
},
|
|
wantStatusCode: http.StatusOK,
|
|
},
|
|
{
|
|
name: "mwAuthenticateFirst fails with failing middleware",
|
|
verificationMiddlwares: []tokenLookup{
|
|
tokenLookupFail,
|
|
},
|
|
wantStatusCode: http.StatusUnauthorized,
|
|
},
|
|
{
|
|
name: "mwAuthenticateFirst succeeds if first middleware successfully handles request",
|
|
verificationMiddlwares: []tokenLookup{
|
|
tokenLookupSucceed(store, jwtService),
|
|
tokenLookupFail,
|
|
},
|
|
wantStatusCode: http.StatusOK,
|
|
},
|
|
{
|
|
name: "mwAuthenticateFirst succeeds if last middleware successfully handles request",
|
|
verificationMiddlwares: []tokenLookup{
|
|
tokenLookupFail,
|
|
tokenLookupSucceed(store, jwtService),
|
|
},
|
|
wantStatusCode: http.StatusOK,
|
|
},
|
|
}
|
|
|
|
for _, tt := range tests {
|
|
t.Run(tt.name, func(t *testing.T) {
|
|
req := httptest.NewRequest(http.MethodGet, "/", nil)
|
|
rr := httptest.NewRecorder()
|
|
|
|
h := bouncer.mwAuthenticateFirst(tt.verificationMiddlwares, testHandler200)
|
|
h.ServeHTTP(rr, req)
|
|
|
|
is.Equal(tt.wantStatusCode, rr.Code, fmt.Sprintf("Status should be %d", tt.wantStatusCode))
|
|
})
|
|
}
|
|
}
|
|
|
|
func Test_extractBearerToken(t *testing.T) {
|
|
is := assert.New(t)
|
|
|
|
tt := []struct {
|
|
name string
|
|
requestHeader string
|
|
requestHeaderValue string
|
|
wantToken string
|
|
succeeds bool
|
|
}{
|
|
{
|
|
name: "missing request header",
|
|
requestHeader: "",
|
|
requestHeaderValue: "",
|
|
wantToken: "",
|
|
succeeds: false,
|
|
},
|
|
{
|
|
name: "invalid authorization request header",
|
|
requestHeader: "authorisation", // note: `s`
|
|
requestHeaderValue: "abc",
|
|
wantToken: "",
|
|
succeeds: false,
|
|
},
|
|
{
|
|
name: "valid authorization request header",
|
|
requestHeader: "AUTHORIZATION",
|
|
requestHeaderValue: "abc",
|
|
wantToken: "abc",
|
|
succeeds: true,
|
|
},
|
|
{
|
|
name: "valid authorization request header case-insensitive canonical check",
|
|
requestHeader: "authorization",
|
|
requestHeaderValue: "def",
|
|
wantToken: "def",
|
|
succeeds: true,
|
|
},
|
|
}
|
|
|
|
for _, test := range tt {
|
|
req := httptest.NewRequest(http.MethodGet, "/", nil)
|
|
req.Header.Set(test.requestHeader, test.requestHeaderValue)
|
|
apiKey, err := extractBearerToken(req)
|
|
is.Equal(test.wantToken, apiKey)
|
|
if !test.succeeds {
|
|
is.Error(err, "Should return error")
|
|
is.ErrorIs(err, httperrors.ErrUnauthorized)
|
|
} else {
|
|
is.NoError(err)
|
|
}
|
|
}
|
|
}
|
|
|
|
func Test_extractAPIKeyHeader(t *testing.T) {
|
|
is := assert.New(t)
|
|
|
|
tt := []struct {
|
|
name string
|
|
requestHeader string
|
|
requestHeaderValue string
|
|
wantApiKey string
|
|
succeeds bool
|
|
}{
|
|
{
|
|
name: "missing request header",
|
|
requestHeader: "",
|
|
requestHeaderValue: "",
|
|
wantApiKey: "",
|
|
succeeds: false,
|
|
},
|
|
{
|
|
name: "invalid api-key request header",
|
|
requestHeader: "api-key",
|
|
requestHeaderValue: "abc",
|
|
wantApiKey: "",
|
|
succeeds: false,
|
|
},
|
|
{
|
|
name: "valid api-key request header",
|
|
requestHeader: apiKeyHeader,
|
|
requestHeaderValue: "abc",
|
|
wantApiKey: "abc",
|
|
succeeds: true,
|
|
},
|
|
{
|
|
name: "valid api-key request header case-insensitive canonical check",
|
|
requestHeader: "x-api-key",
|
|
requestHeaderValue: "def",
|
|
wantApiKey: "def",
|
|
succeeds: true,
|
|
},
|
|
}
|
|
|
|
for _, test := range tt {
|
|
req := httptest.NewRequest(http.MethodGet, "/", nil)
|
|
req.Header.Set(test.requestHeader, test.requestHeaderValue)
|
|
apiKey, ok := extractAPIKey(req)
|
|
is.Equal(test.wantApiKey, apiKey)
|
|
is.Equal(test.succeeds, ok)
|
|
}
|
|
}
|
|
|
|
func Test_extractAPIKeyQueryParam(t *testing.T) {
|
|
is := assert.New(t)
|
|
|
|
tt := []struct {
|
|
name string
|
|
queryParam string
|
|
queryParamValue string
|
|
wantApiKey string
|
|
succeeds bool
|
|
}{
|
|
{
|
|
name: "missing request header",
|
|
queryParam: "",
|
|
queryParamValue: "",
|
|
wantApiKey: "",
|
|
succeeds: false,
|
|
},
|
|
{
|
|
name: "invalid api-key request header",
|
|
queryParam: "api-key",
|
|
queryParamValue: "abc",
|
|
wantApiKey: "",
|
|
succeeds: false,
|
|
},
|
|
{
|
|
name: "valid api-key request header",
|
|
queryParam: apiKeyHeader,
|
|
queryParamValue: "abc",
|
|
wantApiKey: "abc",
|
|
succeeds: true,
|
|
},
|
|
{
|
|
name: "valid api-key request header case-insensitive canonical check",
|
|
queryParam: "x-api-key",
|
|
queryParamValue: "def",
|
|
wantApiKey: "def",
|
|
succeeds: true,
|
|
},
|
|
}
|
|
|
|
for _, test := range tt {
|
|
req := httptest.NewRequest(http.MethodGet, "/", nil)
|
|
q := req.URL.Query()
|
|
q.Add(test.queryParam, test.queryParamValue)
|
|
req.URL.RawQuery = q.Encode()
|
|
|
|
apiKey, ok := extractAPIKey(req)
|
|
is.Equal(test.wantApiKey, apiKey)
|
|
is.Equal(test.succeeds, ok)
|
|
}
|
|
}
|
|
|
|
func Test_apiKeyLookup(t *testing.T) {
|
|
is := assert.New(t)
|
|
|
|
_, store := datastore.MustNewTestStore(t, true, true)
|
|
|
|
// create standard user
|
|
user := &portainer.User{ID: 2, Username: "standard", Role: portainer.StandardUserRole}
|
|
err := store.User().Create(user)
|
|
is.NoError(err, "error creating user")
|
|
|
|
// setup services
|
|
jwtService, err := jwt.NewService("1h", store)
|
|
is.NoError(err, "Error initiating jwt service")
|
|
apiKeyService := apikey.NewAPIKeyService(store.APIKeyRepository(), store.User())
|
|
bouncer := NewRequestBouncer(store, jwtService, apiKeyService)
|
|
|
|
t.Run("missing x-api-key header fails api-key lookup", func(t *testing.T) {
|
|
req := httptest.NewRequest(http.MethodGet, "/", nil)
|
|
// req.Header.Add("Authorization", fmt.Sprintf("Bearer %s", jwt))
|
|
token := bouncer.apiKeyLookup(req)
|
|
is.Nil(token)
|
|
})
|
|
|
|
t.Run("invalid x-api-key header fails api-key lookup", func(t *testing.T) {
|
|
req := httptest.NewRequest(http.MethodGet, "/", nil)
|
|
req.Header.Add("x-api-key", "random-failing-api-key")
|
|
token := bouncer.apiKeyLookup(req)
|
|
is.Nil(token)
|
|
})
|
|
|
|
t.Run("valid x-api-key header succeeds api-key lookup", func(t *testing.T) {
|
|
rawAPIKey, _, err := apiKeyService.GenerateApiKey(*user, "test")
|
|
is.NoError(err)
|
|
|
|
req := httptest.NewRequest(http.MethodGet, "/", nil)
|
|
req.Header.Add("x-api-key", rawAPIKey)
|
|
|
|
token := bouncer.apiKeyLookup(req)
|
|
|
|
expectedToken := &portainer.TokenData{ID: user.ID, Username: user.Username, Role: portainer.StandardUserRole}
|
|
is.Equal(expectedToken, token)
|
|
})
|
|
|
|
t.Run("valid x-api-key header succeeds api-key lookup", func(t *testing.T) {
|
|
rawAPIKey, apiKey, err := apiKeyService.GenerateApiKey(*user, "test")
|
|
is.NoError(err)
|
|
defer apiKeyService.DeleteAPIKey(apiKey.ID)
|
|
|
|
req := httptest.NewRequest(http.MethodGet, "/", nil)
|
|
req.Header.Add("x-api-key", rawAPIKey)
|
|
|
|
token := bouncer.apiKeyLookup(req)
|
|
|
|
expectedToken := &portainer.TokenData{ID: user.ID, Username: user.Username, Role: portainer.StandardUserRole}
|
|
is.Equal(expectedToken, token)
|
|
})
|
|
|
|
t.Run("successful api-key lookup updates token last used time", func(t *testing.T) {
|
|
rawAPIKey, apiKey, err := apiKeyService.GenerateApiKey(*user, "test")
|
|
is.NoError(err)
|
|
defer apiKeyService.DeleteAPIKey(apiKey.ID)
|
|
|
|
req := httptest.NewRequest(http.MethodGet, "/", nil)
|
|
req.Header.Add("x-api-key", rawAPIKey)
|
|
|
|
token := bouncer.apiKeyLookup(req)
|
|
|
|
expectedToken := &portainer.TokenData{ID: user.ID, Username: user.Username, Role: portainer.StandardUserRole}
|
|
is.Equal(expectedToken, token)
|
|
|
|
_, apiKeyUpdated, err := apiKeyService.GetDigestUserAndKey(apiKey.Digest)
|
|
is.NoError(err)
|
|
|
|
is.True(apiKeyUpdated.LastUsed > apiKey.LastUsed)
|
|
})
|
|
}
|