fix: Always set DisconnectedAt if the agent isn't connected

kylecarbs · kylecarbs · commit 61713a06f077 · 2022-10-03T15:07:42.000Z
Fixes #4315.
diff --git a/coderd/workspaceagents.go b/coderd/workspaceagents.go
@@ -574,6 +574,8 @@ func convertWorkspaceAgent(derpMap *tailcfg.DERPMap, coordinator *tailnet.Coordi
 	case database.Now().Sub(dbAgent.LastConnectedAt.Time) > agentInactiveDisconnectTimeout:
 		// The connection died without updating the last connected.
 		workspaceAgent.Status = codersdk.WorkspaceAgentDisconnected
+		// Client code needs an accurate disconnected at if the agent has been inactive.
+		workspaceAgent.DisconnectedAt = &dbAgent.LastConnectedAt.Time
 	case dbAgent.LastConnectedAt.Valid:
 		// The agent should be assumed connected if it's under inactivity timeouts
 		// and last connected at has been properly set.
