search

mpenet / jet

[not maintained, use at own risk] Jetty9 ring server adapter with WebSocket support via core.async and Jetty9 based HTTP & WebSocket clients (jvm based, no cljs)
http://mpenet.github.io/jet/
167 stars 19 forks source link

server: allow specifying parser compliance #40

Closed pyr closed 8 years ago

pyr commented 8 years ago

By default, Jetty standardizes headers to benefit from using a cache. Previously, this was overridable through the `org.eclipse.jetty.http.HttpParser.STRICT' system property.

In recent jetty releases this isn't the case and breaks signing mechanisms which rely on case-sensitive encoding of Http headers.

This PR brings back the ability to get original HTTP header values.

mpenet commented 8 years ago

LGTM. Minor nitpick, could you drop the :parser- prefix in the keywords?

Thanks!

mpenet commented 8 years ago

It's on clojars as 0.7.9

pyr commented 8 years ago

awesome, thanks a lot !

pyr commented 8 years ago

It's a bit sad that jetty never cared to see that org.eclipse.jetty.http.HttpParser.STRICT was set to true and didn't mention the fact that it was now deprecated.