240db4e2a7
Some checks failed
Server CI / Postgres (shard 0) (push) Has been skipped
Server CI / Postgres (shard 3) (push) Has been skipped
BuildEnv Docker Image / build-image (push) Failing after 4s
BuildEnv Docker Image / build-image-fips (push) Failing after 4s
Migration Review & Release Notes / Generate migration review & release notes (push) Failing after 3s
Server CI / Build Environment (push) Failing after 3s
Server CI / Postgres (shard 1) (push) Has been skipped
Tools CI / check-style (mattermost-govet) (push) Failing after 4s
Server CI / Postgres (shard 2) (push) Has been skipped
Tools CI / check-style (mmgotool) (push) Failing after 3s
Server CI / Elasticsearch v8 Compatibility (push) Has been skipped
Server CI / OpenSearch v2 Compatibility (push) Has been skipped
Server CI / Postgres FIPS (shard 0) (push) Has been skipped
Tools CI / Test (mattermost-govet) (push) Failing after 3s
Server CI / Postgres FIPS (shard 1) (push) Has been skipped
Server CI / Postgres FIPS (shard 2) (push) Has been skipped
Tools CI / Test (mmgotool) (push) Failing after 3s
Server CI / Postgres FIPS (shard 3) (push) Has been skipped
Server CI / Run mmctl tests (push) Has been skipped
Server CI / Run mmctl tests (FIPS) (push) Has been skipped
Server CI / Merge Postgres Test Results (push) Has been skipped
Server CI / Merge Postgres FIPS Test Results (push) Has been skipped
Server CI / Check backport migrations (push) Has been skipped
Server CI / Check generated files (push) Has been skipped
Server CI / check-style (push) Has been skipped
Server CI / Vet API (push) Has been skipped
Server CI / Build mattermost server app (push) Has been skipped
Server CI / Server CI Complete (push) Failing after 2s
Server CI / Server CI Artifacts (push) Has been skipped
Server CI / Sentry Upload (push) Has been skipped
Server CI / Server CI Report (push) Failing after 5m1s
Server CI Nightly Race / Race Detector (push) Has been skipped
CodeQL / Analyze (go) (push) Has been cancelled
CodeQL / Analyze (javascript) (push) Has been cancelled
Webapp CI Cache Warm / Warm Webapp node_modules Cache (push) Has been cancelled
E2E CI Cache Warm / Warm E2E npm Registry Cache (push) Has been cancelled
API / build (push) Has been cancelled
Web App CI / check-types (push) Has been cancelled
Web App CI / check-lint (push) Has been cancelled
Web App CI / check-i18n (push) Has been cancelled
Web App CI / check-external-links (push) Has been cancelled
Web App CI / test (platform) (push) Has been cancelled
Web App CI / test (mattermost-redux) (push) Has been cancelled
Web App CI / test (channels shard 1/4) (push) Has been cancelled
Web App CI / test (channels shard 2/4) (push) Has been cancelled
Web App CI / test (channels shard 3/4) (push) Has been cancelled
Web App CI / test (channels shard 4/4) (push) Has been cancelled
Web App CI / upload-coverage (push) Has been cancelled
Web App CI / build (push) Has been cancelled
YAML Lint / yamllint (push) Has been cancelled
Push mirrored docker images / cd/Push mirrored docker images (push) Has been cancelled
docs-cd / Build and deploy docs (push) Has been cancelled
Docs CI / build (push) Has been cancelled
E2E Cache Warm: Cypress deps / Warm Cypress deps cache (push) Has been cancelled
Scorecards supply-chain security / Scorecard analysis (push) Has been cancelled
Server CI Weekly / Postgres FIPS (push) Has been skipped
Server CI Weekly / Run mmctl tests (FIPS) (push) Has been skipped
Server CI Weekly / Postgres with binary parameters (push) Has been skipped
Server CI Weekly / Build Environment (push) Failing after 1s
Server CI Cache Warm / Warm Go Module and Build Cache (push) Failing after 2s
Server CI Nightly Race / Build Environment (push) Failing after 1s
197 lines
7.9 KiB
Go
197 lines
7.9 KiB
Go
// Copyright (c) 2015-present Mattermost, Inc. All Rights Reserved.
|
|
// See LICENSE.txt for license information.
|
|
|
|
package localcachelayer
|
|
|
|
import (
|
|
"testing"
|
|
|
|
"github.com/stretchr/testify/require"
|
|
|
|
"github.com/mattermost/mattermost/server/public/model"
|
|
"github.com/mattermost/mattermost/server/public/shared/mlog"
|
|
"github.com/mattermost/mattermost/server/v8/platform/services/cache"
|
|
)
|
|
|
|
func TestSessionAttributeStoreCache(t *testing.T) {
|
|
logger := mlog.CreateConsoleTestLogger(t)
|
|
|
|
t.Run("Refresh writes attributes that Get returns", func(t *testing.T) {
|
|
mockStore := getMockStore(t)
|
|
mockCacheProvider := getMockCacheProvider()
|
|
cachedStore, err := NewLocalCacheLayer(mockStore, nil, nil, mockCacheProvider, logger)
|
|
require.NoError(t, err)
|
|
|
|
sessionID := model.NewId()
|
|
err = cachedStore.SessionAttribute().Refresh(sessionID, map[string]any{
|
|
model.SessionAttributesPropertyFieldUserAgentBrowserName: "Chrome",
|
|
model.SessionAttributesPropertyFieldIPAddress: "192.0.2.10",
|
|
}, model.GetMillis())
|
|
require.NoError(t, err)
|
|
|
|
got, _, err := cachedStore.SessionAttribute().Get(sessionID)
|
|
require.NoError(t, err)
|
|
require.Equal(t, "Chrome", got[model.SessionAttributesPropertyFieldUserAgentBrowserName])
|
|
require.Equal(t, "192.0.2.10", got[model.SessionAttributesPropertyFieldIPAddress])
|
|
})
|
|
|
|
t.Run("Refresh merges partial attributes", func(t *testing.T) {
|
|
mockStore := getMockStore(t)
|
|
mockCacheProvider := getMockCacheProvider()
|
|
cachedStore, err := NewLocalCacheLayer(mockStore, nil, nil, mockCacheProvider, logger)
|
|
require.NoError(t, err)
|
|
|
|
sessionID := model.NewId()
|
|
require.NoError(t, cachedStore.SessionAttribute().Refresh(sessionID, map[string]any{
|
|
model.SessionAttributesPropertyFieldUserAgentBrowserName: "Chrome",
|
|
model.SessionAttributesPropertyFieldIPAddress: "192.0.2.10",
|
|
}, model.GetMillis()))
|
|
|
|
require.NoError(t, cachedStore.SessionAttribute().Refresh(sessionID, map[string]any{
|
|
model.SessionAttributesPropertyFieldUserAgentBrowserName: "Firefox",
|
|
}, model.GetMillis()))
|
|
|
|
got, timestamps, err := cachedStore.SessionAttribute().Get(sessionID)
|
|
require.NoError(t, err)
|
|
require.Equal(t, "Firefox", got[model.SessionAttributesPropertyFieldUserAgentBrowserName])
|
|
require.Equal(t, "192.0.2.10", got[model.SessionAttributesPropertyFieldIPAddress])
|
|
require.NotZero(t, timestamps[model.SessionAttributesPropertyFieldUserAgentBrowserName])
|
|
require.NotZero(t, timestamps[model.SessionAttributesPropertyFieldIPAddress])
|
|
})
|
|
|
|
t.Run("Get on missing session returns ErrKeyNotFound", func(t *testing.T) {
|
|
mockStore := getMockStore(t)
|
|
mockCacheProvider := getMockCacheProvider()
|
|
cachedStore, err := NewLocalCacheLayer(mockStore, nil, nil, mockCacheProvider, logger)
|
|
require.NoError(t, err)
|
|
|
|
got, _, err := cachedStore.SessionAttribute().Get(model.NewId())
|
|
require.ErrorIs(t, err, cache.ErrKeyNotFound)
|
|
require.Nil(t, got)
|
|
})
|
|
|
|
t.Run("cluster invalidation with session id removes single entry", func(t *testing.T) {
|
|
mockStore := getMockStore(t)
|
|
mockCacheProvider := getMockCacheProvider()
|
|
cachedStore, err := NewLocalCacheLayer(mockStore, nil, nil, mockCacheProvider, logger)
|
|
require.NoError(t, err)
|
|
|
|
sessionA := model.NewId()
|
|
sessionB := model.NewId()
|
|
require.NoError(t, cachedStore.SessionAttribute().Refresh(sessionA, map[string]any{
|
|
model.SessionAttributesPropertyFieldIPAddress: "192.0.2.10",
|
|
}, model.GetMillis()))
|
|
require.NoError(t, cachedStore.SessionAttribute().Refresh(sessionB, map[string]any{
|
|
model.SessionAttributesPropertyFieldIPAddress: "203.0.113.42",
|
|
}, model.GetMillis()))
|
|
|
|
cachedStore.sessionAttribute.handleClusterInvalidateSessionAttributes(&model.ClusterMessage{
|
|
Event: model.ClusterEventInvalidateCacheForSessionAttributes,
|
|
Data: []byte(sessionA),
|
|
})
|
|
|
|
_, _, err = cachedStore.SessionAttribute().Get(sessionA)
|
|
require.ErrorIs(t, err, cache.ErrKeyNotFound)
|
|
|
|
got, _, err := cachedStore.SessionAttribute().Get(sessionB)
|
|
require.NoError(t, err)
|
|
require.Equal(t, "203.0.113.42", got[model.SessionAttributesPropertyFieldIPAddress])
|
|
})
|
|
|
|
t.Run("Invalidate removes only the given session entry", func(t *testing.T) {
|
|
mockStore := getMockStore(t)
|
|
mockCacheProvider := getMockCacheProvider()
|
|
cachedStore, err := NewLocalCacheLayer(mockStore, nil, nil, mockCacheProvider, logger)
|
|
require.NoError(t, err)
|
|
|
|
sessionA := model.NewId()
|
|
sessionB := model.NewId()
|
|
require.NoError(t, cachedStore.SessionAttribute().Refresh(sessionA, map[string]any{
|
|
model.SessionAttributesPropertyFieldIPAddress: "192.0.2.10",
|
|
}, model.GetMillis()))
|
|
require.NoError(t, cachedStore.SessionAttribute().Refresh(sessionB, map[string]any{
|
|
model.SessionAttributesPropertyFieldIPAddress: "203.0.113.42",
|
|
}, model.GetMillis()))
|
|
|
|
require.NoError(t, cachedStore.SessionAttribute().Invalidate(sessionA))
|
|
|
|
_, _, err = cachedStore.SessionAttribute().Get(sessionA)
|
|
require.ErrorIs(t, err, cache.ErrKeyNotFound)
|
|
|
|
got, _, err := cachedStore.SessionAttribute().Get(sessionB)
|
|
require.NoError(t, err)
|
|
require.Equal(t, "203.0.113.42", got[model.SessionAttributesPropertyFieldIPAddress])
|
|
})
|
|
|
|
t.Run("Clear removes every entry", func(t *testing.T) {
|
|
mockStore := getMockStore(t)
|
|
mockCacheProvider := getMockCacheProvider()
|
|
cachedStore, err := NewLocalCacheLayer(mockStore, nil, nil, mockCacheProvider, logger)
|
|
require.NoError(t, err)
|
|
|
|
sessionA := model.NewId()
|
|
sessionB := model.NewId()
|
|
require.NoError(t, cachedStore.SessionAttribute().Refresh(sessionA, map[string]any{
|
|
model.SessionAttributesPropertyFieldIPAddress: "192.0.2.10",
|
|
}, model.GetMillis()))
|
|
require.NoError(t, cachedStore.SessionAttribute().Refresh(sessionB, map[string]any{
|
|
model.SessionAttributesPropertyFieldIPAddress: "203.0.113.42",
|
|
}, model.GetMillis()))
|
|
|
|
require.NoError(t, cachedStore.SessionAttribute().Clear())
|
|
|
|
_, _, err = cachedStore.SessionAttribute().Get(sessionA)
|
|
require.ErrorIs(t, err, cache.ErrKeyNotFound)
|
|
_, _, err = cachedStore.SessionAttribute().Get(sessionB)
|
|
require.ErrorIs(t, err, cache.ErrKeyNotFound)
|
|
})
|
|
|
|
t.Run("Refresh skips an update with an older timestamp", func(t *testing.T) {
|
|
mockStore := getMockStore(t)
|
|
mockCacheProvider := getMockCacheProvider()
|
|
cachedStore, err := NewLocalCacheLayer(mockStore, nil, nil, mockCacheProvider, logger)
|
|
require.NoError(t, err)
|
|
|
|
sessionID := model.NewId()
|
|
require.NoError(t, cachedStore.SessionAttribute().Refresh(sessionID, map[string]any{
|
|
model.SessionAttributesPropertyFieldIPAddress: "203.0.113.42",
|
|
}, model.GetMillis()))
|
|
|
|
// A stale update (older timestamp) for the same key must not win.
|
|
require.NoError(t, cachedStore.SessionAttribute().Refresh(sessionID,
|
|
map[string]any{model.SessionAttributesPropertyFieldIPAddress: "192.0.2.10"},
|
|
int64(1),
|
|
))
|
|
|
|
got, _, err := cachedStore.SessionAttribute().Get(sessionID)
|
|
require.NoError(t, err)
|
|
require.Equal(t, "203.0.113.42", got[model.SessionAttributesPropertyFieldIPAddress])
|
|
})
|
|
|
|
t.Run("cluster invalidation with clear-cache marker purges every entry", func(t *testing.T) {
|
|
mockStore := getMockStore(t)
|
|
mockCacheProvider := getMockCacheProvider()
|
|
cachedStore, err := NewLocalCacheLayer(mockStore, nil, nil, mockCacheProvider, logger)
|
|
require.NoError(t, err)
|
|
|
|
sessionA := model.NewId()
|
|
sessionB := model.NewId()
|
|
require.NoError(t, cachedStore.SessionAttribute().Refresh(sessionA, map[string]any{
|
|
model.SessionAttributesPropertyFieldIPAddress: "192.0.2.10",
|
|
}, model.GetMillis()))
|
|
require.NoError(t, cachedStore.SessionAttribute().Refresh(sessionB, map[string]any{
|
|
model.SessionAttributesPropertyFieldIPAddress: "203.0.113.42",
|
|
}, model.GetMillis()))
|
|
|
|
cachedStore.sessionAttribute.handleClusterInvalidateSessionAttributes(&model.ClusterMessage{
|
|
Event: model.ClusterEventInvalidateCacheForSessionAttributes,
|
|
Data: clearCacheMessageData,
|
|
})
|
|
|
|
_, _, err = cachedStore.SessionAttribute().Get(sessionA)
|
|
require.ErrorIs(t, err, cache.ErrKeyNotFound)
|
|
_, _, err = cachedStore.SessionAttribute().Get(sessionB)
|
|
require.ErrorIs(t, err, cache.ErrKeyNotFound)
|
|
})
|
|
}
|