fix(coderd): fix audit log resource link for tasks (coder#20545) (coder#20547)

johnstcn · web-flow · commit cb4ea1f39705 · 2025-11-03T09:00:42.000Z
Existing task audit log links were incorrect. As audit log links are generated on-the-fly, this does not require backfill. (cherry picked from commit 566146a)
diff --git a/coderd/audit.go b/coderd/audit.go
@@ -509,11 +509,11 @@ func (api *API) auditLogResourceLink(ctx context.Context, alog database.GetAudit
 		if err != nil {
 			return ""
 		}
-		workspace, err := api.Database.GetWorkspaceByID(ctx, task.WorkspaceID.UUID)
+		user, err := api.Database.GetUserByID(ctx, task.OwnerID)
 		if err != nil {
 			return ""
 		}
-		return fmt.Sprintf("/tasks/%s/%s", workspace.OwnerName, task.Name)
+		return fmt.Sprintf("/tasks/%s/%s", user.Username, task.ID)
 
 	default:
 		return ""

Original file line number	Diff line number	Diff line change
`@@ -509,11 +509,11 @@ func (api *API) auditLogResourceLink(ctx context.Context, alog database.GetAudit`
`509`	`509`	`if err != nil {`
`510`	`510`	`return ""`
`511`	`511`	`}`
`512`		`- workspace, err := api.Database.GetWorkspaceByID(ctx, task.WorkspaceID.UUID)`
	`512`	`+ user, err := api.Database.GetUserByID(ctx, task.OwnerID)`
`513`	`513`	`if err != nil {`
`514`	`514`	`return ""`
`515`	`515`	`}`
`516`		`- return fmt.Sprintf("/tasks/%s/%s", workspace.OwnerName, task.Name)`
	`516`	`+ return fmt.Sprintf("/tasks/%s/%s", user.Username, task.ID)`
`517`	`517`
`518`	`518`	`default:`
`519`	`519`	`return ""`