search

expressjs / vhost

virtual domain hosting
MIT License
761 stars 87 forks source link

Support Http2 #29

Open sogaani opened 6 years ago

sogaani commented 6 years ago

Support http2 to enable express tests with http2. expressjs/express#3390.

sogaani commented 6 years ago

I refered rfc and thought :authority header seems to have compatibility with Host header. And I'm not sure issues. But it's ok. I create this PR for express tests with http2. So we can disable tests depend on this. I feel free to close this.

sogaani commented 6 years ago

Above comment is my misunderstanding. Your concern is "There is possibility to refer Host header even if http2 request has :authority header." right? So, I addressed issue.

sogaani commented 6 years ago

I fixed PR installing deps only from npm.