mirror of
https://github.com/hanzoai/team.git
synced 2026-11-01 07:55:31 +00:00
The server module named github.com/hanzoai/team/server/v8 — a path that resolves to the retired workspace's repository, not this one — while server/public still published github.com/mattermost/mattermost/server/public. Both are github.com/hanzoteam/server/... now, which is where this code actually lives. That second half is what made the binary unbootable. Upstream's public path stayed claimed by upstream, mattermost-plugin-ai imported it, and the build linked public/model twice; public/plugin's gob.Register then ran twice under one name and panicked at init, before main. Reverting the rename traded the brand for a working binary. Rebranding the plugin alongside it needs no such trade: the agents fork is github.com/hanzoteam/agents/v2 against our public module, so no module named mattermost/mattermost is in the graph at all. Measured: `go list -deps ./cmd/mattermost` names exactly one public/model, and it is ours.
135 lines
4.6 KiB
Go
135 lines
4.6 KiB
Go
// Copyright (c) 2015-present Mattermost, Inc. All Rights Reserved.
|
|
// See LICENSE.txt for license information.
|
|
|
|
package cleanup_expired_access_tokens
|
|
|
|
import (
|
|
"github.com/hanzoteam/server/server/public/model"
|
|
"github.com/hanzoteam/server/server/public/shared/mlog"
|
|
"github.com/hanzoteam/server/server/public/shared/request"
|
|
"github.com/hanzoteam/server/server/v8/channels/jobs"
|
|
)
|
|
|
|
const (
|
|
workerName = "CleanupExpiredAccessTokens"
|
|
// batchLimit bounds both the number of rows fetched by GetExpiredBefore
|
|
// and the corresponding DeleteByIds call, keeping the transaction
|
|
// footprint bounded even when a large number of tokens expire at once.
|
|
batchLimit = 1000
|
|
// maxBatches caps the number of iterations per job execution so that very
|
|
// large expiry backlogs are drained across multiple scheduled runs rather
|
|
// than a single unbounded loop.
|
|
maxBatches = 1000
|
|
)
|
|
|
|
// expiredTokenStore is the subset of UserAccessTokenStore used by the worker.
|
|
// Defined here rather than depending on the full store interface so the
|
|
// orchestration logic can be unit-tested with a small fake.
|
|
type expiredTokenStore interface {
|
|
GetExpiredBefore(cutoff int64, limit int) ([]*model.UserAccessToken, error)
|
|
DeleteByIds(tokenIDs []string) (int64, error)
|
|
}
|
|
|
|
// MakeWorker creates a worker that periodically deletes personal access tokens
|
|
// whose ExpiresAt has passed, along with any sessions created from them.
|
|
// The work is done in batches to keep the transaction footprint bounded.
|
|
//
|
|
// clearSessionCache is called for each affected user after their tokens are
|
|
// deleted so that in-memory session caches don't serve stale sessions.
|
|
//
|
|
// notifyExpired is called with each batch of expired tokens immediately before
|
|
// they are deleted, so the owner can be DMed that their token has been removed.
|
|
// It is best-effort: any failure is handled internally and must not block the
|
|
// cleanup. Note that after this change PAT expiry notifications come from two
|
|
// jobs — the pre-expiry warning cascade (pat_expiry_notify) and this
|
|
// at-deletion notice — so a future reader shouldn't be surprised to find an
|
|
// expiry DM originating from the cleanup job.
|
|
func MakeWorker(jobServer *jobs.JobServer, clearSessionCache func(userID string), notifyExpired func(rctx request.CTX, tokens []*model.UserAccessToken)) *jobs.SimpleWorker {
|
|
isEnabled := func(cfg *model.Config) bool {
|
|
return *cfg.ServiceSettings.EnableUserAccessTokens
|
|
}
|
|
|
|
execute := func(logger mlog.LoggerIFace, job *model.Job) error {
|
|
defer jobServer.HandleJobPanic(logger, job)
|
|
return cleanupExpired(
|
|
request.EmptyContext(logger),
|
|
jobServer.Store.UserAccessToken(),
|
|
clearSessionCache,
|
|
notifyExpired,
|
|
model.GetMillis(),
|
|
batchLimit,
|
|
maxBatches,
|
|
)
|
|
}
|
|
|
|
return jobs.NewSimpleWorker(workerName, jobServer, execute, isEnabled)
|
|
}
|
|
|
|
// cleanupExpired drains expired personal access tokens in batches up to
|
|
// maxBatches iterations. It is extracted from MakeWorker so that the batching
|
|
// and error-propagation logic can be exercised by unit tests with a fake store.
|
|
//
|
|
// clearSessionCache is called for each unique user whose tokens were deleted so
|
|
// that in-memory session caches don't continue serving the removed sessions.
|
|
//
|
|
// notifyExpired is called with each batch immediately before deletion so token
|
|
// owners can be notified. It runs best-effort: it is invoked before the delete
|
|
// so the token→owner mapping is still available, and the delete proceeds
|
|
// regardless of what it does. The worst case (a crash between notify and
|
|
// delete) is a single duplicate DM on the next run, which is acceptable. The
|
|
// request context is passed through so the notifier logs under the job's logger.
|
|
func cleanupExpired(
|
|
rctx request.CTX,
|
|
store expiredTokenStore,
|
|
clearSessionCache func(userID string),
|
|
notifyExpired func(rctx request.CTX, tokens []*model.UserAccessToken),
|
|
cutoff int64,
|
|
limit int,
|
|
maxIter int,
|
|
) error {
|
|
var totalDeleted int64
|
|
|
|
for range maxIter {
|
|
expired, err := store.GetExpiredBefore(cutoff, limit)
|
|
if err != nil {
|
|
return err
|
|
}
|
|
if len(expired) == 0 {
|
|
break
|
|
}
|
|
|
|
ids := make([]string, len(expired))
|
|
userIDs := make(map[string]struct{}, len(expired))
|
|
for i, token := range expired {
|
|
ids[i] = token.Id
|
|
userIDs[token.UserId] = struct{}{}
|
|
}
|
|
|
|
if notifyExpired != nil {
|
|
notifyExpired(rctx, expired)
|
|
}
|
|
|
|
deleted, err := store.DeleteByIds(ids)
|
|
if err != nil {
|
|
return err
|
|
}
|
|
totalDeleted += deleted
|
|
|
|
for userID := range userIDs {
|
|
clearSessionCache(userID)
|
|
}
|
|
|
|
if len(expired) < limit {
|
|
break
|
|
}
|
|
}
|
|
|
|
rctx.Logger().Info(
|
|
"Cleaned up expired personal access tokens",
|
|
mlog.Int("deleted", int(totalDeleted)),
|
|
mlog.Int("cutoff", int(cutoff)),
|
|
)
|
|
|
|
return nil
|
|
}
|