bouncestorage / chaos-http-proxy

Introduce failures into HTTP requests via a proxy server
Apache License 2.0
144 stars 12 forks source link

Work around Jetty Content-Length removal bug #6

Closed flandr closed 9 years ago

flandr commented 9 years ago

Trying to use chaos-http-proxy to test s3fs-fuse, I found that it's stripping out content-length headers. This turns out to be an apparent bug in Jetty (see referenced issue in the commit message).

gaul commented 9 years ago

Amended to fix modernizer violation and pushed to master as 0838bb5e59f7b3c630a5e7885c9dca6ac1fc8e64.

flandr commented 9 years ago

Would you consider publishing a point release that includes this one? Would like to use the prepackaged jar in some tests that care about content-length ;)

gaul commented 9 years ago

I pushed a snapshot -- does this suffice in the short-term:

https://oss.sonatype.org/content/repositories/snapshots/com/bouncestorage/chaos-http-proxy/1.1.0-SNAPSHOT/chaos-http-proxy-1.1.0-20150824.023506-2-jar-with-dependencies.jar

I am investigating another Content-Length issue surfaced in #4 that I would like to fix before publishing 1.1.

flandr commented 9 years ago

That'll work fine, thanks.

flandr commented 9 years ago

If your issue is that Jetty will stomp over explicit content-length no matter what, then that's the same issue.