diff --git a/coderd/audit/request.go b/coderd/audit/request.go index c73bedcd7d..a1bc0a4be9 100644 --- a/coderd/audit/request.go +++ b/coderd/audit/request.go @@ -135,7 +135,7 @@ func ResourceTarget[T Auditable](tgt T) string { case database.AiSeatState: return "AI Seat" case database.Chat: - return typed.Title + return typed.ID.String()[:8] default: panic(fmt.Sprintf("unknown resource %T for ResourceTarget", tgt)) } diff --git a/coderd/exp_chats_test.go b/coderd/exp_chats_test.go index 2693c5b43c..de3b94280c 100644 --- a/coderd/exp_chats_test.go +++ b/coderd/exp_chats_test.go @@ -272,10 +272,11 @@ func TestPostChats(t *testing.T) { } require.True(t, foundUserMessage) require.True(t, mAudit.Contains(t, database.AuditLog{ - Action: database.AuditActionCreate, - ResourceType: database.ResourceTypeChat, - ResourceID: chat.ID, - UserID: member.ID, + Action: database.AuditActionCreate, + ResourceType: database.ResourceTypeChat, + ResourceID: chat.ID, + ResourceTarget: chat.ID.String()[:8], + UserID: member.ID, })) }) @@ -4564,10 +4565,11 @@ func TestArchiveChat(t *testing.T) { require.True(t, archivedChats[0].Archived) require.True(t, mAudit.Contains(t, database.AuditLog{ - Action: database.AuditActionWrite, - ResourceType: database.ResourceTypeChat, - ResourceID: chatToArchive.ID, - UserID: firstUser.UserID, + Action: database.AuditActionWrite, + ResourceType: database.ResourceTypeChat, + ResourceID: chatToArchive.ID, + ResourceTarget: chatToArchive.ID.String()[:8], + UserID: firstUser.UserID, })) }) t.Run("NotFound", func(t *testing.T) { diff --git a/enterprise/audit/table.go b/enterprise/audit/table.go index e43bccae7a..890f3aa9e8 100644 --- a/enterprise/audit/table.go +++ b/enterprise/audit/table.go @@ -386,7 +386,7 @@ var auditableResourcesTypes = map[any]map[string]Action{ "workspace_id": ActionTrack, "build_id": ActionIgnore, // Internal lifecycle. "agent_id": ActionIgnore, // Internal lifecycle. - "title": ActionTrack, + "title": ActionSecret, "status": ActionIgnore, // Churns every message. "worker_id": ActionIgnore, // Internal. "started_at": ActionIgnore,