Only warn on SSE failure when there is an actual failure

dannykopping · dannykopping · commit d1708cb1347e · 2025-03-12T16:39:20.000Z
Signed-off-by: Danny Kopping &lt;dannykopping@gmail.com&gt;
diff --git a/coderd/workspaceagents.go b/coderd/workspaceagents.go
@@ -1140,7 +1140,9 @@ func (api *API) workspaceAgentReinit(rw http.ResponseWriter, r *http.Request) {
 					Reason:  agentsdk.ReinitializeReasonPrebuildClaimed,
 				},
 			})
-			log.Warn(ctx, "failed to send SSE response to trigger reinit", slog.Error(err))
+			if err != nil {
+				log.Warn(ctx, "failed to send SSE response to trigger reinit", slog.Error(err))
+			}
 		}
 	}
 }

Original file line number	Diff line number	Diff line change
`@@ -1140,7 +1140,9 @@ func (api API) workspaceAgentReinit(rw http.ResponseWriter, r http.Request) {`
`1140`	`1140`	`Reason: agentsdk.ReinitializeReasonPrebuildClaimed,`
`1141`	`1141`	`},`
`1142`	`1142`	`})`
`1143`		`- log.Warn(ctx, "failed to send SSE response to trigger reinit", slog.Error(err))`
	`1143`	`+ if err != nil {`
	`1144`	`+ log.Warn(ctx, "failed to send SSE response to trigger reinit", slog.Error(err))`
	`1145`	`+ }`
`1144`	`1146`	`}`
`1145`	`1147`	`}`
`1146`	`1148`	`}`