Skip to content

Update httpclient to version 4.5.11#1234

Merged
bsideup merged 2 commits intodocker-java:masterfrom
phillip-h:master
Feb 18, 2020
Merged

Update httpclient to version 4.5.11#1234
bsideup merged 2 commits intodocker-java:masterfrom
phillip-h:master

Conversation

@phillip-h
Copy link
Copy Markdown
Contributor

@phillip-h phillip-h commented Aug 29, 2019

Httpclient 4.5.7 introduced a change to URL rewriting that broke AWS's Java SDK, prompting them to require versions >= 4.5.8 (see aws/aws-sdk-java#1919). Using this library in conjuncture with the AWS SDK (for example to interface with ECS) results in problems unless its httpclient is manually excluded. Upgrading to 4.5.9 does not seem to cause issues with docker-java. The unit tests all pass, and while the integration tests fail they seem to be failing on the upstream head already.


This change is Reviewable

@codecov-io
Copy link
Copy Markdown

Codecov Report

Merging #1234 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1234   +/-   ##
=======================================
  Coverage   56.69%   56.69%           
=======================================
  Files         467      467           
  Lines        9026     9026           
  Branches      535      535           
=======================================
  Hits         5117     5117           
  Misses       3631     3631           
  Partials      278      278

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7e781a0...5638a6d. Read the comment docs.

@bsideup bsideup changed the title Update httpclient to version 4.5.9 Update httpclient to version 4.5.11 Feb 17, 2020
@bsideup bsideup added this to the 3.2.0-rc5 milestone Feb 17, 2020
@bsideup
Copy link
Copy Markdown
Member

bsideup commented Feb 17, 2020

Resolved the conflict, will merge after the tests pass

@bsideup bsideup merged commit b4ce5ae into docker-java:master Feb 18, 2020
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.

3 participants