ai-workspace-services/accounts
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix(agent-server): include all active users in sync

Browse Source
This commit is contained in:
Haitao Pan 2026-02-07 03:32:39 +08:00
parent 414a9a417f
commit 5d5e3ce0f8
2 changed files with 4 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
api/agent_server.go
View File

@ -82,14 +82,6 @@ func (h *handler) listAgentUsers(c *gin.Context) {
continue
}
// Default behavior: only sync eligible (active, verified, non-expired) users.
if !u.EmailVerified {
continue
}
if u.ProxyUUIDExpiresAt != nil && now.After(*u.ProxyUUIDExpiresAt) {
continue
}
id := strings.TrimSpace(u.ProxyUUID)
if id == "" {
id = strings.TrimSpace(u.ID)

8
api/api_test.go
View File

@ -140,11 +140,11 @@ func TestAgentServerUsers_DefaultSyncIncludesSandboxAndRegularUsers(t *testing.T
t.Fatalf("create sandbox user: %v", err)
}
// normal user
// normal user (not verified + expired) should still be synced by default.
if err := st.CreateUser(ctx, &store.User{
Name: "User",
Email: "user@example.com",
EmailVerified: true,
EmailVerified: false,
Role: store.RoleUser,
Level: store.LevelUser,
Active: true,
@ -152,12 +152,12 @@ func TestAgentServerUsers_DefaultSyncIncludesSandboxAndRegularUsers(t *testing.T
t.Fatalf("create normal user: %v", err)
}
// Ensure normal user has a non-expired proxy UUID.
// Ensure normal user is "expired" per proxy UUID expiry metadata.
normal, err := st.GetUserByEmail(ctx, "user@example.com")
if err != nil {
t.Fatalf("get normal user: %v", err)
}
exp := time.Now().UTC().Add(24 * time.Hour)
exp := time.Now().UTC().Add(-24 * time.Hour)
normal.ProxyUUIDExpiresAt = &exp
if err := st.UpdateUser(ctx, normal); err != nil {
t.Fatalf("update normal user: %v", err)