Skip to content

chore(deps): Bump jetty from 12.0.29 to 12.0.34#146

Merged
tdcmeehan merged 1 commit into
prestodb:masterfrom
ShahimSharafudeen:jetty-io_cve_fix
May 21, 2026
Merged

chore(deps): Bump jetty from 12.0.29 to 12.0.34#146
tdcmeehan merged 1 commit into
prestodb:masterfrom
ShahimSharafudeen:jetty-io_cve_fix

Conversation

@ShahimSharafudeen

@ShahimSharafudeen ShahimSharafudeen commented Mar 9, 2026

Copy link
Copy Markdown

Upgrade jetty version from 12.0.29 to 12.0.32 to address CVE-2025-11143, CVE-2026-1605 and CVE-2026-2332..

@linux-foundation-easycla

linux-foundation-easycla Bot commented Mar 9, 2026

Copy link
Copy Markdown

CLA Signed

The committers listed above are authorized under a signed CLA.

  • ✅ login: ShahimSharafudeen / name: Shahim Sharafudeen (8586fdd)

@sonarqubecloud

sonarqubecloud Bot commented Mar 9, 2026

Copy link
Copy Markdown

@ShahimSharafudeen ShahimSharafudeen marked this pull request as ready for review March 9, 2026 15:27
@ShahimSharafudeen ShahimSharafudeen requested a review from a team as a code owner March 9, 2026 15:27
@mblanco-denodo

Copy link
Copy Markdown

Prerrequisite: #148

@ShahimSharafudeen ShahimSharafudeen force-pushed the jetty-io_cve_fix branch 2 times, most recently from 277cf13 to d3fffdf Compare April 20, 2026 07:37
@ShahimSharafudeen ShahimSharafudeen changed the title chore(deps): Bump jetty from 12.0.29 to 12.0.32 chore(deps): Bump jetty from 12.0.29 to 12.0.34 Apr 20, 2026
@sonarqubecloud

Copy link
Copy Markdown

@sonarqubecloud

Copy link
Copy Markdown

@ShahimSharafudeen

Copy link
Copy Markdown
Author

Airlift local build :
Screenshot 2026-05-21 at 3 32 00 PM

Presto Local build with Airlift change :

Screenshot 2026-05-21 at 2 34 31 PM

Dependnecy tree after presto local build :
Screenshot 2026-05-21 at 3 10 03 PM

Testing Ailift changes in Presto using jitpack :

Unable to build Presto using the JitPack Airlift dependency due to multiple issues, such as the ones listed below.

  1. drift-maven-plugin resolves the wrong coordinates.
  2. Airlift Launcher resolves coordinates that JitPack does not publish

Error Screenshot :

Screenshot 2026-05-20 at 7 52 11 PM

This is an existing difficulty while testing the JitPack Airlift dependency in Presto:
Ref link for previous testing : #132 (comment)

@agrawalreetika

agrawalreetika commented May 21, 2026

Copy link
Copy Markdown
Member

Why are we not upgrading close to latest?

@ShahimSharafudeen

Copy link
Copy Markdown
Author

@agrawalreetika - Jetty 12.1.x encountered some serious issues in Airlift, which is why the Jetty version was previously downgraded to the 12.0.x series. For the current fix, I have used a recent stable CVE-patched version from the 12.0.x series.
Ref previous Jetty downgrade PR : #132

@agrawalreetika agrawalreetika left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code changes lgtm, I think if there is complexity in testing complete Presto CI without publishing, then we can try the Presto PR update after publishing the changes of airlift.

@tdcmeehan tdcmeehan merged commit 6b0a61a into prestodb:master May 21, 2026
3 checks passed
jja725 pushed a commit to prestodb/presto that referenced this pull request May 25, 2026
## Description
Upgrade jetty version from 12.0.29 to 12.0.32 to address CVE-2025-11143
and CVE-2026-1605.

Dependent Airlift PR : prestodb/airlift#146.

## Motivation and Context
<!---Why is this change required? What problem does it solve?-->
<!---If it fixes an open issue, please link to the issue here.-->

## Impact
<!---Describe any public API or user-facing feature change or any
performance impact-->

## Test Plan
<!---Please fill in how you tested your change-->

## Contributor checklist

- [ ] Please make sure your submission complies with our [contributing
guide](https://github.com/prestodb/presto/blob/master/CONTRIBUTING.md),
in particular [code
style](https://github.com/prestodb/presto/blob/master/CONTRIBUTING.md#code-style)
and [commit
standards](https://github.com/prestodb/presto/blob/master/CONTRIBUTING.md#commit-standards).
- [ ] PR description addresses the issue accurately and concisely. If
the change is non-trivial, a GitHub Issue is referenced.
- [ ] Documented new properties (with its default value), SQL syntax,
functions, or other functionality.
- [ ] If release notes are required, they follow the [release notes
guidelines](https://github.com/prestodb/presto/wiki/Release-Notes-Guidelines).
- [ ] Adequate tests were added if applicable.
- [ ] CI passed.
- [ ] If adding new dependencies, verified they have an [OpenSSF
Scorecard](https://securityscorecards.dev/#the-checks) score of 5.0 or
higher (or obtained explicit TSC approval for lower scores).

## Release Notes
Please follow [release notes
guidelines](https://github.com/prestodb/presto/wiki/Release-Notes-Guidelines)
and fill in the release notes below.

```
== RELEASE NOTES ==

Security Changes
* Upgrade jetty dependency from 0.27 to version 2.0.2 to address `CVE-2025-11143 <https://github.com/advisories/GHSA-wjpw-4j6x-6rwh>` and `CVE-2026-1605 <https://github.com/advisories/GHSA-xxh7-fcf3-rj7f>`_
```
msmygit pushed a commit to msmygit/presto that referenced this pull request Jun 3, 2026
## Description
Upgrade jetty version from 12.0.29 to 12.0.32 to address CVE-2025-11143
and CVE-2026-1605.

Dependent Airlift PR : prestodb/airlift#146.

## Motivation and Context
<!---Why is this change required? What problem does it solve?-->
<!---If it fixes an open issue, please link to the issue here.-->

## Impact
<!---Describe any public API or user-facing feature change or any
performance impact-->

## Test Plan
<!---Please fill in how you tested your change-->

## Contributor checklist

- [ ] Please make sure your submission complies with our [contributing
guide](https://github.com/prestodb/presto/blob/master/CONTRIBUTING.md),
in particular [code
style](https://github.com/prestodb/presto/blob/master/CONTRIBUTING.md#code-style)
and [commit
standards](https://github.com/prestodb/presto/blob/master/CONTRIBUTING.md#commit-standards).
- [ ] PR description addresses the issue accurately and concisely. If
the change is non-trivial, a GitHub Issue is referenced.
- [ ] Documented new properties (with its default value), SQL syntax,
functions, or other functionality.
- [ ] If release notes are required, they follow the [release notes
guidelines](https://github.com/prestodb/presto/wiki/Release-Notes-Guidelines).
- [ ] Adequate tests were added if applicable.
- [ ] CI passed.
- [ ] If adding new dependencies, verified they have an [OpenSSF
Scorecard](https://securityscorecards.dev/#the-checks) score of 5.0 or
higher (or obtained explicit TSC approval for lower scores).

## Release Notes
Please follow [release notes
guidelines](https://github.com/prestodb/presto/wiki/Release-Notes-Guidelines)
and fill in the release notes below.

```
== RELEASE NOTES ==

Security Changes
* Upgrade jetty dependency from 0.27 to version 2.0.2 to address `CVE-2025-11143 <https://github.com/advisories/GHSA-wjpw-4j6x-6rwh>` and `CVE-2026-1605 <https://github.com/advisories/GHSA-xxh7-fcf3-rj7f>`_
```
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants