fix: set request header before do (#11706)

Emyrk · web-flow · commit d67c9d1bb511 · 2024-01-19T16:14:08.000Z
diff --git a/coderd/externalauth/externalauth.go b/coderd/externalauth/externalauth.go
@@ -300,6 +300,7 @@ func (c *DeviceAuth) AuthorizeDevice(ctx context.Context) (*codersdk.ExternalAut
 	if err != nil {
 		return nil, err
 	}
+	req.Header.Set("Accept", "application/json")
 
 	do := http.DefaultClient.Do
 	if c.Config != nil {
@@ -310,7 +311,6 @@ func (c *DeviceAuth) AuthorizeDevice(ctx context.Context) (*codersdk.ExternalAut
 	}
 
 	resp, err := do(req)
-	req.Header.Set("Accept", "application/json")
 	if err != nil {
 		return nil, err
 	}

Original file line number	Diff line number	Diff line change
`@@ -300,6 +300,7 @@ func (c DeviceAuth) AuthorizeDevice(ctx context.Context) (codersdk.ExternalAut`
`300`	`300`	`if err != nil {`
`301`	`301`	`return nil, err`
`302`	`302`	`}`
	`303`	`+ req.Header.Set("Accept", "application/json")`
`303`	`304`
`304`	`305`	`do := http.DefaultClient.Do`
`305`	`306`	`if c.Config != nil {`
`@@ -310,7 +311,6 @@ func (c DeviceAuth) AuthorizeDevice(ctx context.Context) (codersdk.ExternalAut`
`310`	`311`	`}`
`311`	`312`
`312`	`313`	`resp, err := do(req)`
`313`		`- req.Header.Set("Accept", "application/json")`
`314`	`314`	`if err != nil {`
`315`	`315`	`return nil, err`
`316`	`316`	`}`