Check upgrade string for 1.1 upgrade

microsoftarchive / http2-katana

HTTP 2.0 with Katana

134 stars 34 forks source link

Check upgrade string for 1.1 upgrade #48

Closed jeffmendoza closed 11 years ago

jeffmendoza commented 11 years ago

src/Microsoft.Http2.Owin.Server/Adapters/Http11OwinMessageHandler.cs L164

Looks like we should be checking the upgrade string here (or somewhere else). We should only accept upgrade if it matches our protocol string (HTTP-draft-06/2.0).

sgrebnov commented 11 years ago

Right now we check this here https://github.com/MSOpenTech/http2-katana/blob/master/src/Microsoft.Http2.Owin.Middleware/Http2Middleware.cs#L88

jeffmendoza commented 11 years ago

Ah, thanks. Lets change to check for the Procools.Http2 string, instead of 'HTTP' and '2.0'

squirrelfm commented 11 years ago

Check updated. Closing.