feat: add description to audit log responses (coder#3949)

coadler · web-flow · commit 7dc73ed6c6ac · 2022-09-08T09:36:34.000-05:00
diff --git a/coderd/audit.go b/coderd/audit.go
@@ -2,6 +2,7 @@ package coderd
 
 import (
 	"encoding/json"
+	"fmt"
 	"net"
 	"net/http"
 	"net/netip"
@@ -167,7 +168,14 @@ func convertAuditLog(dblog database.GetAuditLogsOffsetRow) codersdk.AuditLog {
 		Diff:             diff,
 		StatusCode:       dblog.StatusCode,
 		AdditionalFields: dblog.AdditionalFields,
-		Description:      "",
+		Description:      auditLogDescription(dblog),
 		User:             user,
 	}
 }
+
+func auditLogDescription(alog database.GetAuditLogsOffsetRow) string {
+	return fmt.Sprintf("{user} %s %s {target}",
+		codersdk.AuditAction(alog.Action).FriendlyString(),
+		codersdk.ResourceType(alog.ResourceType).FriendlyString(),
+	)
+}
diff --git a/coderd/database/databasefake/databasefake.go b/coderd/database/databasefake/databasefake.go
@@ -2308,7 +2308,7 @@ func (q *fakeQuerier) GetAuditLogsOffset(ctx context.Context, arg database.GetAu
 			OrganizationID:   alog.OrganizationID,
 			Ip:               alog.Ip,
 			UserAgent:        alog.UserAgent,
-			ResourceType:     database.ResourceType(alog.UserAgent),
+			ResourceType:     alog.ResourceType,
 			ResourceID:       alog.ResourceID,
 			ResourceTarget:   alog.ResourceTarget,
 			ResourceIcon:     alog.ResourceIcon,
diff --git a/coderd/database/dump.sql b/coderd/database/dump.sql
diff --git a/coderd/database/migrations/000018_provisioner_job_type_dry_run.down.sql b/coderd/database/migrations/000018_provisioner_job_type_dry_run.down.sql
@@ -1,9 +1,9 @@
 -- It's not possible to drop enum values from enum types, so the UP has "IF NOT
 -- EXISTS".
 
--- Delete all jobs that use the new enum value.
+-- Delete all audit logs that use the new enum values.
 DELETE FROM
-    provisioner_jobs
+    audit_logs
 WHERE
-    type = 'template_version_dry_run'
-;
+    resource_type = 'git_ssh_key' OR
+    resource_type = 'api_key';
diff --git a/coderd/database/migrations/000046_more_resource_types.down.sql b/coderd/database/migrations/000046_more_resource_types.down.sql
@@ -0,0 +1,8 @@
+-- It's not possible to drop enum values from enum types, so the UP has "IF NOT
+-- EXISTS".
+
+-- Delete all jobs that use the new enum value.
+DELETE FROM
+    provisioner_jobs
+WHERE
+    type = 'template_version_dry_run';
diff --git a/coderd/database/migrations/000046_more_resource_types.up.sql b/coderd/database/migrations/000046_more_resource_types.up.sql
@@ -0,0 +1,2 @@
+ALTER TYPE resource_type ADD VALUE IF NOT EXISTS 'git_ssh_key';
+ALTER TYPE resource_type ADD VALUE IF NOT EXISTS 'api_key';
diff --git a/coderd/database/models.go b/coderd/database/models.go
diff --git a/codersdk/audit.go b/codersdk/audit.go
@@ -18,8 +18,31 @@ const (
 	ResourceTypeTemplateVersion ResourceType = "template_version"
 	ResourceTypeUser            ResourceType = "user"
 	ResourceTypeWorkspace       ResourceType = "workspace"
+	ResourceTypeGitSSHKey       ResourceType = "git_ssh_key"
+	ResourceTypeAPIKey          ResourceType = "api_key"
 )
 
+func (r ResourceType) FriendlyString() string {
+	switch r {
+	case ResourceTypeOrganization:
+		return "organization"
+	case ResourceTypeTemplate:
+		return "template"
+	case ResourceTypeTemplateVersion:
+		return "template version"
+	case ResourceTypeUser:
+		return "user"
+	case ResourceTypeWorkspace:
+		return "workspace"
+	case ResourceTypeGitSSHKey:
+		return "git ssh key"
+	case ResourceTypeAPIKey:
+		return "api key"
+	default:
+		return "unknown"
+	}
+}
+
 type AuditAction string
 
 const (
@@ -28,6 +51,19 @@ const (
 	AuditActionDelete AuditAction = "delete"
 )
 
+func (a AuditAction) FriendlyString() string {
+	switch a {
+	case AuditActionCreate:
+		return "created"
+	case AuditActionWrite:
+		return "updated"
+	case AuditActionDelete:
+		return "deleted"
+	default:
+		return "unknown"
+	}
+}
+
 type AuditDiff map[string]AuditDiffField
 
 type AuditDiffField struct {
diff --git a/site/src/api/typesGenerated.ts b/site/src/api/typesGenerated.ts
@@ -692,7 +692,14 @@ export type ProvisionerStorageMethod = "file"
 export type ProvisionerType = "echo" | "terraform"
 
 // From codersdk/audit.go
-export type ResourceType = "organization" | "template" | "template_version" | "user" | "workspace"
+export type ResourceType =
+  | "api_key"
+  | "git_ssh_key"
+  | "organization"
+  | "template"
+  | "template_version"
+  | "user"
+  | "workspace"
 
 // From codersdk/users.go
 export type UserStatus = "active" | "suspended"
diff --git a/site/src/components/AuditLogRow/AuditLogRow.tsx b/site/src/components/AuditLogRow/AuditLogRow.tsx
@@ -38,6 +38,8 @@ const resourceLabelByResourceType: Record<AuditLog["resource_type"], string> = {
   template_version: "template version",
   user: "user",
   workspace: "workspace",
+  git_ssh_key: "git ssh key",
+  api_key: "api key",
 }
 
 const readableActionMessage = (auditLog: AuditLog) => {

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,2 @@`
	`1`	`+ALTER TYPE resource_type ADD VALUE IF NOT EXISTS 'git_ssh_key';`
	`2`	`+ALTER TYPE resource_type ADD VALUE IF NOT EXISTS 'api_key';`
Original file line number	Diff line number	Diff line change
`@@ -38,6 +38,8 @@ const resourceLabelByResourceType: Record<AuditLog["resource_type"], string> = {`
`38`	`38`	`template_version: "template version",`
`39`	`39`	`user: "user",`
`40`	`40`	`workspace: "workspace",`
	`41`	`+ git_ssh_key: "git ssh key",`
	`42`	`+ api_key: "api key",`
`41`	`43`	`}`
`42`	`44`
`43`	`45`	`const readableActionMessage = (auditLog: AuditLog) => {`