As a rule, our INFO log statements should be curated

ammario · ammario · commit 4d3ff5fbead1 · 2023-05-09T19:20:25.000Z
So no JSON-dumped complex objects like the workspaceAgent
diff --git a/coderd/workspaceagents.go b/coderd/workspaceagents.go
@@ -930,7 +930,12 @@ func (api *API) workspaceAgentCoordinate(rw http.ResponseWriter, r *http.Request
 	}
 	api.publishWorkspaceUpdate(ctx, build.WorkspaceID)
 
-	api.Logger.Info(ctx, "accepting agent", slog.F("agent", workspaceAgent))
+	api.Logger.Info(ctx, "accepting agent",
+		slog.F("owner", owner.Username),
+		slog.F("workspace", workspace.Name),
+		slog.F("name", workspaceAgent.Name),
+	)
+	api.Logger.Debug(ctx, "accepting agent details", slog.F("agent", workspaceAgent))
 
 	defer conn.Close(websocket.StatusNormalClosure, "")
 
diff --git a/go.mod b/go.mod
@@ -51,7 +51,7 @@ replace github.com/gliderlabs/ssh => github.com/coder/ssh v0.0.0-20230421140225-
 replace github.com/imulab/go-scim/pkg/v2 => github.com/coder/go-scim/pkg/v2 v2.0.0-20230221055123-1d63c1222136
 
 require (
-	cdr.dev/slog v1.5.0
+	cdr.dev/slog v1.5.1
 	cloud.google.com/go/compute/metadata v0.2.3
 	github.com/AlecAivazis/survey/v2 v2.3.5
 	github.com/acarl005/stripansi v0.0.0-20180116102854-5a71ef0e047d
diff --git a/go.sum b/go.sum
@@ -14,6 +14,8 @@ cdr.dev/slog v1.4.3-0.20230509013323-fbe4576b00f6 h1:0yXyAI+9SDEfpi0nnu8wPL6Oor0
 cdr.dev/slog v1.4.3-0.20230509013323-fbe4576b00f6/go.mod h1:vW6Q4gGoDZSb4Db2wxAZoUba/HRUpen1g0fCu06zrjQ=
 cdr.dev/slog v1.5.0 h1:JE8gZeRLNb8wHj9PGR5IxuIx2eoa4vVB9Y2XCaOHJ+c=
 cdr.dev/slog v1.5.0/go.mod h1:vW6Q4gGoDZSb4Db2wxAZoUba/HRUpen1g0fCu06zrjQ=
+cdr.dev/slog v1.5.1 h1:C5YF6oMdmTfx/pClpqJ15KebRkMpA/mC0XltolKLMkA=
+cdr.dev/slog v1.5.1/go.mod h1:vW6Q4gGoDZSb4Db2wxAZoUba/HRUpen1g0fCu06zrjQ=
 cloud.google.com/go v0.26.0/go.mod h1:aQUYkXzVsufM+DwF1aE+0xfcU+56JwCaLick0ClmMTw=
 cloud.google.com/go v0.34.0/go.mod h1:aQUYkXzVsufM+DwF1aE+0xfcU+56JwCaLick0ClmMTw=
 cloud.google.com/go v0.38.0/go.mod h1:990N+gfupTy94rShfmMCWGDn0LpTmnzTp2qbd1dvSRU=
