Avoid passing empty headers during upgrade

koding / websocketproxy

WebSocket reverse proxy handler for Go

http://godoc.org/github.com/koding/websocketproxy

MIT License

427 stars 120 forks source link

Avoid passing empty headers during upgrade #3

Closed databus23 closed 8 years ago

databus23 commented 8 years ago

This fixes the proxy for Safari which fails with a cryptic Invalid UTF-8 sequence in header value error when the upgrade request contains a header with an empty Value e.g. Set-Cookie:

I also removed the http.CanonicalHeaderKey function call which is not needed for Header.Get

fatih commented 8 years ago

Thanks @databus23 :+1: