Skip to content

Commit

Permalink
Temporary fix for jetty/jetty.project#11841
Browse files Browse the repository at this point in the history
jetty/jetty.project#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
  • Loading branch information
Randgalt committed Jul 24, 2024
1 parent ee7dcb0 commit e2f2084
Show file tree
Hide file tree
Showing 4 changed files with 626 additions and 0 deletions.
35 changes: 35 additions & 0 deletions 000-temp-trino-aws-proxy/pom.xml
Original file line number Diff line number Diff line change
@@ -0,0 +1,35 @@
<?xml version="1.0" encoding="UTF-8"?>
<project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd">
<modelVersion>4.0.0</modelVersion>
<parent>
<groupId>io.trino</groupId>
<artifactId>trino-aws-proxy-root</artifactId>
<version>1-SNAPSHOT</version>
</parent>

<artifactId>000-temp-trino-aws-proxy</artifactId>

<properties>
<air.main.basedir>${project.parent.basedir}</air.main.basedir>
<air.check.skip-all>true</air.check.skip-all>
</properties>

<dependencies>
<dependency>
<groupId>io.airlift</groupId>
<artifactId>http-client</artifactId>
</dependency>
</dependencies>

<build>
<plugins>
<plugin>
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-surefire-plugin</artifactId>
<configuration>
<failIfNoTests>false</failIfNoTests>
</configuration>
</plugin>
</plugins>
</build>
</project>
Loading

0 comments on commit e2f2084

Please sign in to comment.