Skip to content

Commit 6584331

Browse files
committed
Remove the host header before forwarding
With some hosts behind shared TLS (using SNI), keeping the Host header was causing the target host to be misidentified. I also observed issues related to TLS verification (as seen in probot#156). Fixes probot#156
1 parent 2c34dab commit 6584331

1 file changed

Lines changed: 3 additions & 0 deletions

File tree

index.ts

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -43,6 +43,9 @@ class Client {
4343

4444
delete data.query
4545

46+
// Remove the host header, leaving it causes issues with SNI and TLS verification
47+
delete data.host
48+
4649
const req = superagent.post(url.format(target)).send(data.body)
4750

4851
delete data.body

0 commit comments

Comments
 (0)