fix: add span.SetStatus to codersdk.Request (coder#6964)

coadler · web-flow · commit f4d16a1ae50b · 2023-04-03T05:52:42.000Z
diff --git a/codersdk/client.go b/codersdk/client.go
@@ -183,6 +183,7 @@ func (c *Client) Request(ctx context.Context, method, path string, body interfac
 	}
 
 	span.SetAttributes(httpconv.ClientResponse(resp)...)
+	span.SetStatus(httpconv.ClientStatus(resp.StatusCode))
 
 	// Copy the response body so we can log it if it's a loggable mime type.
 	var respBody []byte

Original file line number	Diff line number	Diff line change
`@@ -183,6 +183,7 @@ func (c *Client) Request(ctx context.Context, method, path string, body interfac`
`183`	`183`	`}`
`184`	`184`
`185`	`185`	`span.SetAttributes(httpconv.ClientResponse(resp)...)`
	`186`	`+ span.SetStatus(httpconv.ClientStatus(resp.StatusCode))`
`186`	`187`
`187`	`188`	`// Copy the response body so we can log it if it's a loggable mime type.`
`188`	`189`	`var respBody []byte`