fix: Handle all method types for app proxying (coder#2868)

kylecarbs · web-flow · commit 52fa1f246499 · 2022-07-08T15:45:28.000-05:00
All methods need to be accepted on app routes. Some apps
may POST (like Jupyter).
diff --git a/coderd/coderd.go b/coderd/coderd.go
@@ -124,7 +124,7 @@ func New(options *Options) *API {
 			apiKeyMiddleware,
 			httpmw.ExtractUserParam(api.Database),
 		)
-		r.Get("/*", api.workspaceAppsProxyPath)
+		r.HandleFunc("/*", api.workspaceAppsProxyPath)
 	}
 	// %40 is the encoded character of the @ symbol. VS Code Web does
 	// not handle character encoding properly, so it's safe to assume

Original file line number	Diff line number	Diff line change
`@@ -124,7 +124,7 @@ func New(options Options) API {`
`124`	`124`	`apiKeyMiddleware,`
`125`	`125`	`httpmw.ExtractUserParam(api.Database),`
`126`	`126`	`)`
`127`		`- r.Get("/*", api.workspaceAppsProxyPath)`
	`127`	`+ r.HandleFunc("/*", api.workspaceAppsProxyPath)`
`128`	`128`	`}`
`129`	`129`	`// %40 is the encoded character of the @ symbol. VS Code Web does`
`130`	`130`	`// not handle character encoding properly, so it's safe to assume`