Commitedd982e

authored

fix(vpn/tunnel): fix panic when starting tunnel with headers (#16565)

`http.Header` is a map so it needs to be initialized ..

1 parentb23e3f9 commitedd982eCopy full SHA for edd982e

File tree

-1

lines changed

-1

lines changed

Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -230,7 +230,7 @@ func (t Tunnel) start(req StartRequest) error {`
`230`	`230`	`ifapiToken=="" {`
`231`	`231`	`returnxerrors.New("missing api token")`
`232`	`232`	`}`
`233`		`-varheader http.Header`
	`233`	`+header:=make(http.Header)`
`234`	`234`	`for_,h:=rangereq.GetHeaders() {`
`235`	`235`	`header.Add(h.GetName(),h.GetValue())`
`236`	`236`	`}`

Lines changed: 3 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -100,6 +100,9 @@ func TestTunnel_StartStop(t *testing.T) {`
`100`	`100`	`TunnelFileDescriptor:2,`
`101`	`101`	`CoderUrl:"https://coder.example.com",`
`102`	`102`	`ApiToken:"fakeToken",`
	`103`	`+Headers: []*StartRequest_Header{`
	`104`	`+{Name:"X-Test-Header",Value:"test"},`
	`105`	`+},`
`103`	`106`	`},`
`104`	`107`	`},`
`105`	`108`	`})`

Comments

(0)