Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upgrade Apache HttpComponents Core to 5.0.3. #1375

Merged
merged 5 commits into from
Apr 9, 2021

Conversation

maciejwalkowiak
Copy link
Contributor

@maciejwalkowiak maciejwalkowiak commented Apr 6, 2021

📜 Description

Upgrade Apache HttpComponents Core to 5.0.3.

💡 Motivation and Context

HttpComponents Core 5.0.3 contains a bugfix for a potential infinite loop and CPU exhaustion.

According to library authors, HttpComponents Core 5.0.3 is compatible with HttpComponents Client 5.0.3.

💚 How did you test it?

Covered by existing tests. Could not reproduce the infinite loop.

📝 Checklist

  • I reviewed the submitted code
  • I added tests to verify the changes
  • I updated the docs if needed
  • No breaking changes

HttpComponents Core 5.0.3 contains a bugfix for a potential infinite loop and CPU exhaustion.
@maciejwalkowiak
Copy link
Contributor Author

cc @bruno-garcia

Comment on lines +24 to +25
api(Config.Libs.apacheHttpCore)
api(Config.Libs.apacheHttpCoreH2)
Copy link
Contributor

Choose a reason for hiding this comment

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

do people need to access its classes? otherwise only implementation is fine

Copy link
Contributor Author

Choose a reason for hiding this comment

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

They don't need to but they may want to. We bring ApacheTransportFactory but they may want to create their custom factory with custom client configuration. I am not 100% sure if at some level of configuration they would not need to use lower level classes.

@maciejwalkowiak maciejwalkowiak marked this pull request as draft April 6, 2021 09:39
@maciejwalkowiak
Copy link
Contributor Author

Converting to draft until we decide that this is the way to go. The alternative is to upgrade httpcomponents-core directly to 5.1

@marandaneto
Copy link
Contributor

Converting to draft until we decide that this is the way to go. The alternative is to upgrade httpcomponents-core directly to 5.1

if 5.0.3 fixes the bug, even though they said 5.1 is also compatible, not sure how extensible this has been tested as its a new release, it'd be safer just bumping the patch.

@maciejwalkowiak
Copy link
Contributor Author

5.0.3 fixes a bug, but i am not sure if this is the one we were facing. Just to double check I've filed an issue: https://issues.apache.org/jira/browse/HTTPCLIENT-2146

@maciejwalkowiak maciejwalkowiak marked this pull request as ready for review April 9, 2021 09:28
@marandaneto marandaneto merged commit a4949a2 into main Apr 9, 2021
@marandaneto marandaneto deleted the upgrade-apache-transport branch April 9, 2021 11:45
@github-actions
Copy link
Contributor

github-actions bot commented Apr 9, 2021

Fails
🚫 Please consider adding a changelog entry for the next release.
`Instructions and example for changelog`$

Please add an entry to CHANGELOG.md` to the "Unreleased" section under the following heading:

To the changelog entry, please add a link to this PR (consider a more descriptive message):`

- Upgrade Apache HttpComponents Core to 5.0.3(#1375)

If none of the above apply, you can opt out by adding _#skip-changelog_ to the PR description.

Generated by 🚫 dangerJS against eb74fd2

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.

2 participants