Skip to content

Commit

Permalink
Force downgrading to org.apache.httpcomponents:httpclient:4.5.6 preve…
Browse files Browse the repository at this point in the history
…nt URI normalization (#1924)

* Force org.apache.httpcomponents:httpclient:4.5.6
* Update CHANGELOGs
  • Loading branch information
chanseokoh authored Aug 22, 2019
1 parent 7bb9167 commit 50bc8f0
Show file tree
Hide file tree
Showing 7 changed files with 47 additions and 9 deletions.
2 changes: 2 additions & 0 deletions jib-core/CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,8 @@ All notable changes to this project will be documented in this file.

### Fixed

- Fixed an issue interacting with certain registries due to changes to URL handling in the underlying Apache HttpClient library. ([#1924](https:/GoogleContainerTools/jib/issues/1924))

## 0.10.1

### Added
Expand Down
18 changes: 15 additions & 3 deletions jib-core/build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -41,9 +41,21 @@ dependencies {
// For Google libraries, check <http-client-bom.version>, <google.auth.version>, <guava.version>,
// ... in https:/googleapis/google-cloud-java/blob/master/google-cloud-clients/pom.xml
// for best compatibility.
implementation 'com.google.http-client:google-http-client:1.31.0'
implementation 'com.google.http-client:google-http-client-apache-v2:1.31.0'
implementation 'com.google.auth:google-auth-library-oauth2-http:0.16.2'
//
// TODO: remove once https:/googleapis/google-http-java-client/issues/795 is fixed and released.
// Forcing to downgrade this to 4.5.6 fixes https:/GoogleContainerTools/jib/issues/1914
// However, #795 and upgrading httpclient alone may not fix #1914. We may need to explicitly disable URI
// normalization as discussed in #795.
implementation('com.google.http-client:google-http-client:1.31.0') {
exclude group: "org.apache.httpcomponents", module: "httpclient"
}
implementation('com.google.http-client:google-http-client-apache-v2:1.31.0') {
exclude group: "org.apache.httpcomponents", module: "httpclient"
}
implementation('com.google.auth:google-auth-library-oauth2-http:0.16.2') {
exclude group: "org.apache.httpcomponents", module: "httpclient"
}
implementation 'org.apache.httpcomponents:httpclient:4.5.6'
implementation 'com.google.guava:guava:28.0-jre'

implementation 'org.apache.commons:commons-compress:1.18'
Expand Down
2 changes: 2 additions & 0 deletions jib-gradle-plugin/CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,8 @@ All notable changes to this project will be documented in this file.

### Fixed

- Fixed an issue interacting with certain registries due to changes to URL handling in the underlying Apache HttpClient library. ([#1924](https:/GoogleContainerTools/jib/issues/1924))

## 1.5.0

### Added
Expand Down
13 changes: 10 additions & 3 deletions jib-gradle-plugin/build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -61,9 +61,16 @@ configurations {

dependencies {
// These are copied over from jib-core and are necessary for the jib-core sourcesets.
compile 'com.google.http-client:google-http-client:1.31.0'
compile 'com.google.http-client:google-http-client-apache-v2:1.31.0'
compile 'com.google.auth:google-auth-library-oauth2-http:0.16.2'
compile('com.google.http-client:google-http-client:1.31.0') {
exclude group: "org.apache.httpcomponents", module: "httpclient"
}
compile('com.google.http-client:google-http-client-apache-v2:1.31.0') {
exclude group: "org.apache.httpcomponents", module: "httpclient"
}
compile('com.google.auth:google-auth-library-oauth2-http:0.16.2') {
exclude group: "org.apache.httpcomponents", module: "httpclient"
}
compile 'org.apache.httpcomponents:httpclient:4.5.6'
compile 'com.google.guava:guava:28.0-jre'

compile 'org.apache.commons:commons-compress:1.18'
Expand Down
2 changes: 2 additions & 0 deletions jib-maven-plugin/CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,8 @@ All notable changes to this project will be documented in this file.

### Fixed

- Fixed an issue interacting with certain registries due to changes to URL handling in the underlying Apache HttpClient library. ([#1924](https:/GoogleContainerTools/jib/issues/1924))

## 1.5.0

### Added
Expand Down
6 changes: 6 additions & 0 deletions jib-maven-plugin/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -62,6 +62,12 @@
<version>1.31.0</version>
<scope>compile</scope>
</dependency>
<dependency>
<groupId>org.apache.httpcomponents</groupId>
<artifactId>httpclient</artifactId>
<version>4.5.6</version>
<scope>compile</scope>
</dependency>
<dependency>
<groupId>com.google.auth</groupId>
<artifactId>google-auth-library-oauth2-http</artifactId>
Expand Down
13 changes: 10 additions & 3 deletions jib-plugins-common/build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -29,9 +29,16 @@ sourceSets {

dependencies {
// Make sure these are consistent with jib-maven-plugin.
compile 'com.google.http-client:google-http-client:1.31.0'
compile 'com.google.http-client:google-http-client-apache-v2:1.31.0'
compile 'com.google.auth:google-auth-library-oauth2-http:0.16.2'
compile('com.google.http-client:google-http-client:1.31.0') {
exclude group: "org.apache.httpcomponents", module: "httpclient"
}
compile('com.google.http-client:google-http-client-apache-v2:1.31.0') {
exclude group: "org.apache.httpcomponents", module: "httpclient"
}
compile('com.google.auth:google-auth-library-oauth2-http:0.16.2') {
exclude group: "org.apache.httpcomponents", module: "httpclient"
}
compile 'org.apache.httpcomponents:httpclient:4.5.6'
compile 'com.google.guava:guava:28.0-jre'

compile 'org.apache.commons:commons-compress:1.18'
Expand Down

0 comments on commit 50bc8f0

Please sign in to comment.