fix: Panic when loading coordination override

kylecarbs · kylecarbs · commit 17b6cecc1523 · 2022-09-22T10:50:39.000-05:00
This was broken because of browser-only. This should fix it!
diff --git a/coderd/workspaceagents.go b/coderd/workspaceagents.go
@@ -405,8 +405,10 @@ func (api *API) workspaceAgentClientCoordinate(rw http.ResponseWriter, r *http.R
 	}
 	// This is used by Enterprise code to control the functionality of this route.
 	override := api.WorkspaceClientCoordinateOverride.Load()
-	if override != nil && (*override)(rw) {
-		return
+	if override != nil {
+		if (*override)(rw) {
+			return
+		}
 	}
 
 	api.websocketWaitMutex.Lock()

Original file line number	Diff line number	Diff line change
`@@ -405,8 +405,10 @@ func (api API) workspaceAgentClientCoordinate(rw http.ResponseWriter, r http.R`
`405`	`405`	`}`
`406`	`406`	`// This is used by Enterprise code to control the functionality of this route.`
`407`	`407`	`override := api.WorkspaceClientCoordinateOverride.Load()`
`408`		`- if override != nil && (*override)(rw) {`
`409`		`- return`
	`408`	`+ if override != nil {`
	`409`	`+ if (*override)(rw) {`
	`410`	`+ return`
	`411`	`+ }`
`410`	`412`	`}`
`411`	`413`
`412`	`414`	`api.websocketWaitMutex.Lock()`