Skip to content

Bump async-http-client to 3.0.1 - #17646

Merged
kgyrtkirk merged 8 commits into
apache:masterfrom
infvg:bump-async
Apr 5, 2025
Merged

Bump async-http-client to 3.0.1#17646
kgyrtkirk merged 8 commits into
apache:masterfrom
infvg:bump-async

Conversation

@infvg

@infvg infvg commented Jan 20, 2025

Copy link
Copy Markdown
Contributor

Description

Bump async-http-client to 3.0.1 to resolve CVE-2024-53990.

Release note


General Changes

This PR has:

  • been self-reviewed.
  • [] added documentation for new or modified features or behaviors.
  • a release note entry in the PR description.
  • [] added Javadocs for most classes and all non-trivial methods. Linked related entities via Javadoc links.
  • added or updated version, license, or notice information in licenses.yaml
  • [] added comments explaining the "why" and the intent of the code wherever would not be obvious for an unfamiliar reader.
  • added unit tests or modified existing tests to cover new code paths, ensuring the threshold for code coverage is met.
  • added integration tests.
  • been tested in a test Druid cluster.

Bump async-http-client to 3.0.1 to resolve CVE-2024-53990 _.
@cryptoe

cryptoe commented Feb 25, 2025

Copy link
Copy Markdown
Contributor

Can you test this on a druid cluster ?

@cryptoe

cryptoe commented Feb 25, 2025

Copy link
Copy Markdown
Contributor

Seems like no CI tests have run. Could you please merge your branch with master and then push the commit again ?

@kgyrtkirk kgyrtkirk closed this Feb 26, 2025
@kgyrtkirk kgyrtkirk reopened this Feb 26, 2025
@gianm

gianm commented Apr 3, 2025

Copy link
Copy Markdown
Contributor

FYI, I merged master into the PR branch and added another couple of required dependencies to the licenses.yaml.

@gianm

gianm commented Apr 3, 2025

Copy link
Copy Markdown
Contributor

The update might have actually broken something. The compaction integration tests for both middleManager and indexer are showing a variety of failures. Seems odd that only those broke though, while other indexing-related tests passed.

@kgyrtkirk kgyrtkirk added this to the 33.0.0 milestone Apr 4, 2025
@gianm

gianm commented Apr 5, 2025

Copy link
Copy Markdown
Contributor

It's also possible those tests were just flaky. Will re-run them.

@kgyrtkirk
kgyrtkirk merged commit c60e3fb into apache:master Apr 5, 2025
kgyrtkirk pushed a commit to kgyrtkirk/druid that referenced this pull request Apr 7, 2025
* Bump async-http-client to 3.0.1

Bump async-http-client to 3.0.1 to resolve CVE-2024-53990 _.

* Additional licenses.yaml updates.

* Fix conflict.

---------

Co-authored-by: Gian Merlino <gianmerlino@gmail.com>
(cherry picked from commit c60e3fb)
basapuram-kumar pushed a commit to acceldata-io/druid that referenced this pull request Jun 19, 2025
* Bump async-http-client to 3.0.1

Bump async-http-client to 3.0.1 to resolve CVE-2024-53990 _.

* Additional licenses.yaml updates.

* Fix conflict.

---------

Co-authored-by: Gian Merlino <gianmerlino@gmail.com>
basapuram-kumar added a commit to acceldata-io/druid that referenced this pull request Jun 19, 2025
riovic918data pushed a commit to riovic918data/druid that referenced this pull request Jun 12, 2026
* Bump async-http-client to 3.0.1

Bump async-http-client to 3.0.1 to resolve CVE-2024-53990 _.

* Additional licenses.yaml updates.

* Fix conflict.

---------

Co-authored-by: Gian Merlino <gianmerlino@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants