Temporary fix for https://github.com/jetty/jetty.project/issues/11841

trinodb / aws-proxy

Proxy for S3

Apache License 2.0

7 stars 3 forks source link

Temporary fix for https://github.com/jetty/jetty.project/issues/11841 #115

Closed Randgalt closed 1 month ago

Randgalt commented 1 month ago

https://github.com/jetty/jetty.project/issues/11841 has an NPE when using 100-Continue. This hack is a workaround.

Create a new module that generates a JAR that will sort first in the classpath
Put a hacked version of Jetty's HttpSender in this module that has a fix
The Java classloader will load this version of the class first, thus working around the problem

This hack should be removed as soon as possible

Randgalt commented 1 month ago

Damn - we just got TestGenericRestRequests.testPutObject:206->doPutObject:273 » NullPointer Cannot invoke "org.eclipse.jetty.client.transport.HttpExchange.getRequest()" because "this.exchange" is null on this. I need to spend more time on this.