Skip to content

Conversation

@fabriziofortino
Copy link
Contributor

No description provided.

Copy link
Contributor

@reschke reschke left a comment

Choose a reason for hiding this comment

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

You should update httpcore as well.

@fabriziofortino
Copy link
Contributor Author

@reschke I did not because httpclient 4.5.13 is compiled/tested/released with httpcore 4.4.13 (https://github.com/apache/httpcomponents-client/blob/rel/v4.5.13/pom.xml#L69). The latest httpcore will be used in the next release of httpclient.

@thomasmueller
Copy link
Member

Test

@thomasmueller thomasmueller self-requested a review July 6, 2021 09:35
@fabriziofortino
Copy link
Contributor Author

@reschke correct, the update to v 4.5.13 is to avoid the vulnerability you mentioned (https://issues.apache.org/jira/browse/OAK-9482)

@reschke
Copy link
Contributor

reschke commented Jul 6, 2021

Yep, I misread the CVE.

Anyway; I understand the desire to use this combination, but given the fact that there are bugfixes in httpcore I really do not understand why we wouldn't use it, There's a reason why that release was made.

@fabriziofortino fabriziofortino requested a review from reschke July 6, 2021 11:54
@fabriziofortino fabriziofortino merged commit fb6986d into apache:trunk Jul 7, 2021
@fabriziofortino fabriziofortino deleted the OAK-9482 branch July 7, 2021 07:16
Joscorbe pushed a commit to Joscorbe/jackrabbit-oak that referenced this pull request Sep 3, 2021
* OAK-9482: upgrade httpclient to 4.5.13

* OAK-9482: align httpmime to v 4.5.13

* OAK-9482: upgrade httpcore to 4.4.14
reschke pushed a commit that referenced this pull request Mar 20, 2023
* OAK-9482: upgrade httpclient to 4.5.13

* OAK-9482: align httpmime to v 4.5.13

* OAK-9482: upgrade httpcore to 4.4.14
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