From 8b5ea85b44c5302d62351789e1bdd41ef7665344 Mon Sep 17 00:00:00 2001 From: buildmaster Date: Tue, 11 Jun 2019 18:37:07 +0000 Subject: [PATCH 01/12] Update SNAPSHOT to 2.1.2.RELEASE --- README.adoc | 6 +++++- docs/pom.xml | 2 +- pom.xml | 4 ++-- spring-cloud-commons-dependencies/pom.xml | 4 ++-- spring-cloud-commons/pom.xml | 2 +- spring-cloud-context-integration-tests/pom.xml | 2 +- spring-cloud-context/pom.xml | 2 +- spring-cloud-loadbalancer/pom.xml | 2 +- spring-cloud-starter/pom.xml | 2 +- spring-cloud-test-support/pom.xml | 2 +- 10 files changed, 16 insertions(+), 12 deletions(-) diff --git a/README.adoc b/README.adoc index 9dccadc0..45159743 100644 --- a/README.adoc +++ b/README.adoc @@ -1,4 +1,8 @@ -// Do not edit this file (e.g. go instead to src/main/asciidoc) +//// +DO NOT EDIT THIS FILE. IT WAS GENERATED. +Manual changes to this file will be lost when it is generated again. +Edit the files in the src/main/asciidoc/ directory instead. +//// image::https://circleci.com/gh/spring-cloud/spring-cloud-commons.svg?style=svg[Build Status, link=https://circleci.com/gh/spring-cloud/spring-cloud-commons] diff --git a/docs/pom.xml b/docs/pom.xml index de6a1861..2281b586 100644 --- a/docs/pom.xml +++ b/docs/pom.xml @@ -8,7 +8,7 @@ org.springframework.cloud spring-cloud-commons-parent - 2.1.2.BUILD-SNAPSHOT + 2.1.2.RELEASE pom Spring Cloud Commons Docs diff --git a/pom.xml b/pom.xml index a2664378..680092c0 100644 --- a/pom.xml +++ b/pom.xml @@ -5,7 +5,7 @@ 4.0.0 org.springframework.cloud spring-cloud-commons-parent - 2.1.2.BUILD-SNAPSHOT + 2.1.2.RELEASE pom Spring Cloud Commons Parent Spring Cloud Commons Parent @@ -13,7 +13,7 @@ org.springframework.cloud spring-cloud-build - 2.1.4.BUILD-SNAPSHOT + 2.1.5.RELEASE diff --git a/spring-cloud-commons-dependencies/pom.xml b/spring-cloud-commons-dependencies/pom.xml index af349d61..685709f7 100644 --- a/spring-cloud-commons-dependencies/pom.xml +++ b/spring-cloud-commons-dependencies/pom.xml @@ -6,11 +6,11 @@ spring-cloud-dependencies-parent org.springframework.cloud - 2.1.4.BUILD-SNAPSHOT + 2.1.5.RELEASE spring-cloud-commons-dependencies - 2.1.2.BUILD-SNAPSHOT + 2.1.2.RELEASE pom spring-cloud-commons-dependencies Spring Cloud Commons Dependencies diff --git a/spring-cloud-commons/pom.xml b/spring-cloud-commons/pom.xml index 408520c8..80c4875e 100644 --- a/spring-cloud-commons/pom.xml +++ b/spring-cloud-commons/pom.xml @@ -7,7 +7,7 @@ org.springframework.cloud spring-cloud-commons-parent - 2.1.2.BUILD-SNAPSHOT + 2.1.2.RELEASE .. spring-cloud-commons diff --git a/spring-cloud-context-integration-tests/pom.xml b/spring-cloud-context-integration-tests/pom.xml index de29c8ed..00748f53 100644 --- a/spring-cloud-context-integration-tests/pom.xml +++ b/spring-cloud-context-integration-tests/pom.xml @@ -7,7 +7,7 @@ org.springframework.cloud spring-cloud-commons-parent - 2.1.2.BUILD-SNAPSHOT + 2.1.2.RELEASE .. spring-cloud-context-integration-tests diff --git a/spring-cloud-context/pom.xml b/spring-cloud-context/pom.xml index 063f5b88..35160693 100644 --- a/spring-cloud-context/pom.xml +++ b/spring-cloud-context/pom.xml @@ -7,7 +7,7 @@ org.springframework.cloud spring-cloud-commons-parent - 2.1.2.BUILD-SNAPSHOT + 2.1.2.RELEASE .. spring-cloud-context diff --git a/spring-cloud-loadbalancer/pom.xml b/spring-cloud-loadbalancer/pom.xml index 852a296d..30e0a1e2 100644 --- a/spring-cloud-loadbalancer/pom.xml +++ b/spring-cloud-loadbalancer/pom.xml @@ -7,7 +7,7 @@ org.springframework.cloud spring-cloud-commons-parent - 2.1.2.BUILD-SNAPSHOT + 2.1.2.RELEASE .. spring-cloud-loadbalancer diff --git a/spring-cloud-starter/pom.xml b/spring-cloud-starter/pom.xml index 64c17ed9..e96b9e85 100644 --- a/spring-cloud-starter/pom.xml +++ b/spring-cloud-starter/pom.xml @@ -6,7 +6,7 @@ org.springframework.cloud spring-cloud-commons-parent - 2.1.2.BUILD-SNAPSHOT + 2.1.2.RELEASE spring-cloud-starter spring-cloud-starter diff --git a/spring-cloud-test-support/pom.xml b/spring-cloud-test-support/pom.xml index 7a2036f3..02b13a3a 100644 --- a/spring-cloud-test-support/pom.xml +++ b/spring-cloud-test-support/pom.xml @@ -7,7 +7,7 @@ org.springframework.cloud spring-cloud-commons-parent - 2.1.2.BUILD-SNAPSHOT + 2.1.2.RELEASE .. spring-cloud-test-support From a21ed507cb81c7fbf62adb567624f186adb1bd4a Mon Sep 17 00:00:00 2001 From: buildmaster Date: Tue, 11 Jun 2019 18:38:29 +0000 Subject: [PATCH 02/12] Going back to snapshots --- README.adoc | 6 +----- docs/pom.xml | 2 +- pom.xml | 4 ++-- spring-cloud-commons-dependencies/pom.xml | 4 ++-- spring-cloud-commons/pom.xml | 2 +- spring-cloud-context-integration-tests/pom.xml | 2 +- spring-cloud-context/pom.xml | 2 +- spring-cloud-loadbalancer/pom.xml | 2 +- spring-cloud-starter/pom.xml | 2 +- spring-cloud-test-support/pom.xml | 2 +- 10 files changed, 12 insertions(+), 16 deletions(-) diff --git a/README.adoc b/README.adoc index 45159743..9dccadc0 100644 --- a/README.adoc +++ b/README.adoc @@ -1,8 +1,4 @@ -//// -DO NOT EDIT THIS FILE. IT WAS GENERATED. -Manual changes to this file will be lost when it is generated again. -Edit the files in the src/main/asciidoc/ directory instead. -//// +// Do not edit this file (e.g. go instead to src/main/asciidoc) image::https://circleci.com/gh/spring-cloud/spring-cloud-commons.svg?style=svg[Build Status, link=https://circleci.com/gh/spring-cloud/spring-cloud-commons] diff --git a/docs/pom.xml b/docs/pom.xml index 2281b586..de6a1861 100644 --- a/docs/pom.xml +++ b/docs/pom.xml @@ -8,7 +8,7 @@ org.springframework.cloud spring-cloud-commons-parent - 2.1.2.RELEASE + 2.1.2.BUILD-SNAPSHOT pom Spring Cloud Commons Docs diff --git a/pom.xml b/pom.xml index 680092c0..a2664378 100644 --- a/pom.xml +++ b/pom.xml @@ -5,7 +5,7 @@ 4.0.0 org.springframework.cloud spring-cloud-commons-parent - 2.1.2.RELEASE + 2.1.2.BUILD-SNAPSHOT pom Spring Cloud Commons Parent Spring Cloud Commons Parent @@ -13,7 +13,7 @@ org.springframework.cloud spring-cloud-build - 2.1.5.RELEASE + 2.1.4.BUILD-SNAPSHOT diff --git a/spring-cloud-commons-dependencies/pom.xml b/spring-cloud-commons-dependencies/pom.xml index 685709f7..af349d61 100644 --- a/spring-cloud-commons-dependencies/pom.xml +++ b/spring-cloud-commons-dependencies/pom.xml @@ -6,11 +6,11 @@ spring-cloud-dependencies-parent org.springframework.cloud - 2.1.5.RELEASE + 2.1.4.BUILD-SNAPSHOT spring-cloud-commons-dependencies - 2.1.2.RELEASE + 2.1.2.BUILD-SNAPSHOT pom spring-cloud-commons-dependencies Spring Cloud Commons Dependencies diff --git a/spring-cloud-commons/pom.xml b/spring-cloud-commons/pom.xml index 80c4875e..408520c8 100644 --- a/spring-cloud-commons/pom.xml +++ b/spring-cloud-commons/pom.xml @@ -7,7 +7,7 @@ org.springframework.cloud spring-cloud-commons-parent - 2.1.2.RELEASE + 2.1.2.BUILD-SNAPSHOT .. spring-cloud-commons diff --git a/spring-cloud-context-integration-tests/pom.xml b/spring-cloud-context-integration-tests/pom.xml index 00748f53..de29c8ed 100644 --- a/spring-cloud-context-integration-tests/pom.xml +++ b/spring-cloud-context-integration-tests/pom.xml @@ -7,7 +7,7 @@ org.springframework.cloud spring-cloud-commons-parent - 2.1.2.RELEASE + 2.1.2.BUILD-SNAPSHOT .. spring-cloud-context-integration-tests diff --git a/spring-cloud-context/pom.xml b/spring-cloud-context/pom.xml index 35160693..063f5b88 100644 --- a/spring-cloud-context/pom.xml +++ b/spring-cloud-context/pom.xml @@ -7,7 +7,7 @@ org.springframework.cloud spring-cloud-commons-parent - 2.1.2.RELEASE + 2.1.2.BUILD-SNAPSHOT .. spring-cloud-context diff --git a/spring-cloud-loadbalancer/pom.xml b/spring-cloud-loadbalancer/pom.xml index 30e0a1e2..852a296d 100644 --- a/spring-cloud-loadbalancer/pom.xml +++ b/spring-cloud-loadbalancer/pom.xml @@ -7,7 +7,7 @@ org.springframework.cloud spring-cloud-commons-parent - 2.1.2.RELEASE + 2.1.2.BUILD-SNAPSHOT .. spring-cloud-loadbalancer diff --git a/spring-cloud-starter/pom.xml b/spring-cloud-starter/pom.xml index e96b9e85..64c17ed9 100644 --- a/spring-cloud-starter/pom.xml +++ b/spring-cloud-starter/pom.xml @@ -6,7 +6,7 @@ org.springframework.cloud spring-cloud-commons-parent - 2.1.2.RELEASE + 2.1.2.BUILD-SNAPSHOT spring-cloud-starter spring-cloud-starter diff --git a/spring-cloud-test-support/pom.xml b/spring-cloud-test-support/pom.xml index 02b13a3a..7a2036f3 100644 --- a/spring-cloud-test-support/pom.xml +++ b/spring-cloud-test-support/pom.xml @@ -7,7 +7,7 @@ org.springframework.cloud spring-cloud-commons-parent - 2.1.2.RELEASE + 2.1.2.BUILD-SNAPSHOT .. spring-cloud-test-support From c9d42117be0456dfde0649d6b406e128441f5537 Mon Sep 17 00:00:00 2001 From: buildmaster Date: Tue, 11 Jun 2019 18:38:29 +0000 Subject: [PATCH 03/12] Bumping versions to 2.1.3.BUILD-SNAPSHOT after release --- docs/pom.xml | 2 +- pom.xml | 2 +- spring-cloud-commons-dependencies/pom.xml | 2 +- spring-cloud-commons/pom.xml | 2 +- spring-cloud-context-integration-tests/pom.xml | 2 +- spring-cloud-context/pom.xml | 2 +- spring-cloud-loadbalancer/pom.xml | 2 +- spring-cloud-starter/pom.xml | 2 +- spring-cloud-test-support/pom.xml | 2 +- 9 files changed, 9 insertions(+), 9 deletions(-) diff --git a/docs/pom.xml b/docs/pom.xml index de6a1861..991cd090 100644 --- a/docs/pom.xml +++ b/docs/pom.xml @@ -8,7 +8,7 @@ org.springframework.cloud spring-cloud-commons-parent - 2.1.2.BUILD-SNAPSHOT + 2.1.3.BUILD-SNAPSHOT pom Spring Cloud Commons Docs diff --git a/pom.xml b/pom.xml index a2664378..dc54b920 100644 --- a/pom.xml +++ b/pom.xml @@ -5,7 +5,7 @@ 4.0.0 org.springframework.cloud spring-cloud-commons-parent - 2.1.2.BUILD-SNAPSHOT + 2.1.3.BUILD-SNAPSHOT pom Spring Cloud Commons Parent Spring Cloud Commons Parent diff --git a/spring-cloud-commons-dependencies/pom.xml b/spring-cloud-commons-dependencies/pom.xml index af349d61..00fffb29 100644 --- a/spring-cloud-commons-dependencies/pom.xml +++ b/spring-cloud-commons-dependencies/pom.xml @@ -10,7 +10,7 @@ spring-cloud-commons-dependencies - 2.1.2.BUILD-SNAPSHOT + 2.1.3.BUILD-SNAPSHOT pom spring-cloud-commons-dependencies Spring Cloud Commons Dependencies diff --git a/spring-cloud-commons/pom.xml b/spring-cloud-commons/pom.xml index 408520c8..368deda5 100644 --- a/spring-cloud-commons/pom.xml +++ b/spring-cloud-commons/pom.xml @@ -7,7 +7,7 @@ org.springframework.cloud spring-cloud-commons-parent - 2.1.2.BUILD-SNAPSHOT + 2.1.3.BUILD-SNAPSHOT .. spring-cloud-commons diff --git a/spring-cloud-context-integration-tests/pom.xml b/spring-cloud-context-integration-tests/pom.xml index de29c8ed..c7b62813 100644 --- a/spring-cloud-context-integration-tests/pom.xml +++ b/spring-cloud-context-integration-tests/pom.xml @@ -7,7 +7,7 @@ org.springframework.cloud spring-cloud-commons-parent - 2.1.2.BUILD-SNAPSHOT + 2.1.3.BUILD-SNAPSHOT .. spring-cloud-context-integration-tests diff --git a/spring-cloud-context/pom.xml b/spring-cloud-context/pom.xml index 063f5b88..4ed22eb7 100644 --- a/spring-cloud-context/pom.xml +++ b/spring-cloud-context/pom.xml @@ -7,7 +7,7 @@ org.springframework.cloud spring-cloud-commons-parent - 2.1.2.BUILD-SNAPSHOT + 2.1.3.BUILD-SNAPSHOT .. spring-cloud-context diff --git a/spring-cloud-loadbalancer/pom.xml b/spring-cloud-loadbalancer/pom.xml index 852a296d..cebcfe7a 100644 --- a/spring-cloud-loadbalancer/pom.xml +++ b/spring-cloud-loadbalancer/pom.xml @@ -7,7 +7,7 @@ org.springframework.cloud spring-cloud-commons-parent - 2.1.2.BUILD-SNAPSHOT + 2.1.3.BUILD-SNAPSHOT .. spring-cloud-loadbalancer diff --git a/spring-cloud-starter/pom.xml b/spring-cloud-starter/pom.xml index 64c17ed9..83a5cf87 100644 --- a/spring-cloud-starter/pom.xml +++ b/spring-cloud-starter/pom.xml @@ -6,7 +6,7 @@ org.springframework.cloud spring-cloud-commons-parent - 2.1.2.BUILD-SNAPSHOT + 2.1.3.BUILD-SNAPSHOT spring-cloud-starter spring-cloud-starter diff --git a/spring-cloud-test-support/pom.xml b/spring-cloud-test-support/pom.xml index 7a2036f3..f86e440b 100644 --- a/spring-cloud-test-support/pom.xml +++ b/spring-cloud-test-support/pom.xml @@ -7,7 +7,7 @@ org.springframework.cloud spring-cloud-commons-parent - 2.1.2.BUILD-SNAPSHOT + 2.1.3.BUILD-SNAPSHOT .. spring-cloud-test-support From 24514b085c2d2d2a24ce77234b05353dfc586f2c Mon Sep 17 00:00:00 2001 From: buildmaster Date: Thu, 20 Jun 2019 16:23:22 +0000 Subject: [PATCH 04/12] Update SNAPSHOT to 2.1.2.RELEASE --- README.adoc | 6 +++++- docs/pom.xml | 2 +- pom.xml | 4 ++-- spring-cloud-commons-dependencies/pom.xml | 4 ++-- spring-cloud-commons/pom.xml | 2 +- spring-cloud-context-integration-tests/pom.xml | 2 +- spring-cloud-context/pom.xml | 2 +- spring-cloud-loadbalancer/pom.xml | 2 +- spring-cloud-starter/pom.xml | 2 +- spring-cloud-test-support/pom.xml | 2 +- 10 files changed, 16 insertions(+), 12 deletions(-) diff --git a/README.adoc b/README.adoc index 9dccadc0..45159743 100644 --- a/README.adoc +++ b/README.adoc @@ -1,4 +1,8 @@ -// Do not edit this file (e.g. go instead to src/main/asciidoc) +//// +DO NOT EDIT THIS FILE. IT WAS GENERATED. +Manual changes to this file will be lost when it is generated again. +Edit the files in the src/main/asciidoc/ directory instead. +//// image::https://circleci.com/gh/spring-cloud/spring-cloud-commons.svg?style=svg[Build Status, link=https://circleci.com/gh/spring-cloud/spring-cloud-commons] diff --git a/docs/pom.xml b/docs/pom.xml index 991cd090..2281b586 100644 --- a/docs/pom.xml +++ b/docs/pom.xml @@ -8,7 +8,7 @@ org.springframework.cloud spring-cloud-commons-parent - 2.1.3.BUILD-SNAPSHOT + 2.1.2.RELEASE pom Spring Cloud Commons Docs diff --git a/pom.xml b/pom.xml index dc54b920..680092c0 100644 --- a/pom.xml +++ b/pom.xml @@ -5,7 +5,7 @@ 4.0.0 org.springframework.cloud spring-cloud-commons-parent - 2.1.3.BUILD-SNAPSHOT + 2.1.2.RELEASE pom Spring Cloud Commons Parent Spring Cloud Commons Parent @@ -13,7 +13,7 @@ org.springframework.cloud spring-cloud-build - 2.1.4.BUILD-SNAPSHOT + 2.1.5.RELEASE diff --git a/spring-cloud-commons-dependencies/pom.xml b/spring-cloud-commons-dependencies/pom.xml index 00fffb29..685709f7 100644 --- a/spring-cloud-commons-dependencies/pom.xml +++ b/spring-cloud-commons-dependencies/pom.xml @@ -6,11 +6,11 @@ spring-cloud-dependencies-parent org.springframework.cloud - 2.1.4.BUILD-SNAPSHOT + 2.1.5.RELEASE spring-cloud-commons-dependencies - 2.1.3.BUILD-SNAPSHOT + 2.1.2.RELEASE pom spring-cloud-commons-dependencies Spring Cloud Commons Dependencies diff --git a/spring-cloud-commons/pom.xml b/spring-cloud-commons/pom.xml index 368deda5..80c4875e 100644 --- a/spring-cloud-commons/pom.xml +++ b/spring-cloud-commons/pom.xml @@ -7,7 +7,7 @@ org.springframework.cloud spring-cloud-commons-parent - 2.1.3.BUILD-SNAPSHOT + 2.1.2.RELEASE .. spring-cloud-commons diff --git a/spring-cloud-context-integration-tests/pom.xml b/spring-cloud-context-integration-tests/pom.xml index c7b62813..00748f53 100644 --- a/spring-cloud-context-integration-tests/pom.xml +++ b/spring-cloud-context-integration-tests/pom.xml @@ -7,7 +7,7 @@ org.springframework.cloud spring-cloud-commons-parent - 2.1.3.BUILD-SNAPSHOT + 2.1.2.RELEASE .. spring-cloud-context-integration-tests diff --git a/spring-cloud-context/pom.xml b/spring-cloud-context/pom.xml index 4ed22eb7..35160693 100644 --- a/spring-cloud-context/pom.xml +++ b/spring-cloud-context/pom.xml @@ -7,7 +7,7 @@ org.springframework.cloud spring-cloud-commons-parent - 2.1.3.BUILD-SNAPSHOT + 2.1.2.RELEASE .. spring-cloud-context diff --git a/spring-cloud-loadbalancer/pom.xml b/spring-cloud-loadbalancer/pom.xml index cebcfe7a..30e0a1e2 100644 --- a/spring-cloud-loadbalancer/pom.xml +++ b/spring-cloud-loadbalancer/pom.xml @@ -7,7 +7,7 @@ org.springframework.cloud spring-cloud-commons-parent - 2.1.3.BUILD-SNAPSHOT + 2.1.2.RELEASE .. spring-cloud-loadbalancer diff --git a/spring-cloud-starter/pom.xml b/spring-cloud-starter/pom.xml index 83a5cf87..e96b9e85 100644 --- a/spring-cloud-starter/pom.xml +++ b/spring-cloud-starter/pom.xml @@ -6,7 +6,7 @@ org.springframework.cloud spring-cloud-commons-parent - 2.1.3.BUILD-SNAPSHOT + 2.1.2.RELEASE spring-cloud-starter spring-cloud-starter diff --git a/spring-cloud-test-support/pom.xml b/spring-cloud-test-support/pom.xml index f86e440b..02b13a3a 100644 --- a/spring-cloud-test-support/pom.xml +++ b/spring-cloud-test-support/pom.xml @@ -7,7 +7,7 @@ org.springframework.cloud spring-cloud-commons-parent - 2.1.3.BUILD-SNAPSHOT + 2.1.2.RELEASE .. spring-cloud-test-support From 3d3673ade78858e1c533dd7e5c4965f096ac156a Mon Sep 17 00:00:00 2001 From: buildmaster Date: Thu, 20 Jun 2019 16:26:17 +0000 Subject: [PATCH 05/12] Going back to snapshots --- README.adoc | 6 +----- docs/pom.xml | 2 +- pom.xml | 4 ++-- spring-cloud-commons-dependencies/pom.xml | 4 ++-- spring-cloud-commons/pom.xml | 2 +- spring-cloud-context-integration-tests/pom.xml | 2 +- spring-cloud-context/pom.xml | 2 +- spring-cloud-loadbalancer/pom.xml | 2 +- spring-cloud-starter/pom.xml | 2 +- spring-cloud-test-support/pom.xml | 2 +- 10 files changed, 12 insertions(+), 16 deletions(-) diff --git a/README.adoc b/README.adoc index 45159743..9dccadc0 100644 --- a/README.adoc +++ b/README.adoc @@ -1,8 +1,4 @@ -//// -DO NOT EDIT THIS FILE. IT WAS GENERATED. -Manual changes to this file will be lost when it is generated again. -Edit the files in the src/main/asciidoc/ directory instead. -//// +// Do not edit this file (e.g. go instead to src/main/asciidoc) image::https://circleci.com/gh/spring-cloud/spring-cloud-commons.svg?style=svg[Build Status, link=https://circleci.com/gh/spring-cloud/spring-cloud-commons] diff --git a/docs/pom.xml b/docs/pom.xml index 2281b586..991cd090 100644 --- a/docs/pom.xml +++ b/docs/pom.xml @@ -8,7 +8,7 @@ org.springframework.cloud spring-cloud-commons-parent - 2.1.2.RELEASE + 2.1.3.BUILD-SNAPSHOT pom Spring Cloud Commons Docs diff --git a/pom.xml b/pom.xml index 680092c0..dc54b920 100644 --- a/pom.xml +++ b/pom.xml @@ -5,7 +5,7 @@ 4.0.0 org.springframework.cloud spring-cloud-commons-parent - 2.1.2.RELEASE + 2.1.3.BUILD-SNAPSHOT pom Spring Cloud Commons Parent Spring Cloud Commons Parent @@ -13,7 +13,7 @@ org.springframework.cloud spring-cloud-build - 2.1.5.RELEASE + 2.1.4.BUILD-SNAPSHOT diff --git a/spring-cloud-commons-dependencies/pom.xml b/spring-cloud-commons-dependencies/pom.xml index 685709f7..00fffb29 100644 --- a/spring-cloud-commons-dependencies/pom.xml +++ b/spring-cloud-commons-dependencies/pom.xml @@ -6,11 +6,11 @@ spring-cloud-dependencies-parent org.springframework.cloud - 2.1.5.RELEASE + 2.1.4.BUILD-SNAPSHOT spring-cloud-commons-dependencies - 2.1.2.RELEASE + 2.1.3.BUILD-SNAPSHOT pom spring-cloud-commons-dependencies Spring Cloud Commons Dependencies diff --git a/spring-cloud-commons/pom.xml b/spring-cloud-commons/pom.xml index 80c4875e..368deda5 100644 --- a/spring-cloud-commons/pom.xml +++ b/spring-cloud-commons/pom.xml @@ -7,7 +7,7 @@ org.springframework.cloud spring-cloud-commons-parent - 2.1.2.RELEASE + 2.1.3.BUILD-SNAPSHOT .. spring-cloud-commons diff --git a/spring-cloud-context-integration-tests/pom.xml b/spring-cloud-context-integration-tests/pom.xml index 00748f53..c7b62813 100644 --- a/spring-cloud-context-integration-tests/pom.xml +++ b/spring-cloud-context-integration-tests/pom.xml @@ -7,7 +7,7 @@ org.springframework.cloud spring-cloud-commons-parent - 2.1.2.RELEASE + 2.1.3.BUILD-SNAPSHOT .. spring-cloud-context-integration-tests diff --git a/spring-cloud-context/pom.xml b/spring-cloud-context/pom.xml index 35160693..4ed22eb7 100644 --- a/spring-cloud-context/pom.xml +++ b/spring-cloud-context/pom.xml @@ -7,7 +7,7 @@ org.springframework.cloud spring-cloud-commons-parent - 2.1.2.RELEASE + 2.1.3.BUILD-SNAPSHOT .. spring-cloud-context diff --git a/spring-cloud-loadbalancer/pom.xml b/spring-cloud-loadbalancer/pom.xml index 30e0a1e2..cebcfe7a 100644 --- a/spring-cloud-loadbalancer/pom.xml +++ b/spring-cloud-loadbalancer/pom.xml @@ -7,7 +7,7 @@ org.springframework.cloud spring-cloud-commons-parent - 2.1.2.RELEASE + 2.1.3.BUILD-SNAPSHOT .. spring-cloud-loadbalancer diff --git a/spring-cloud-starter/pom.xml b/spring-cloud-starter/pom.xml index e96b9e85..83a5cf87 100644 --- a/spring-cloud-starter/pom.xml +++ b/spring-cloud-starter/pom.xml @@ -6,7 +6,7 @@ org.springframework.cloud spring-cloud-commons-parent - 2.1.2.RELEASE + 2.1.3.BUILD-SNAPSHOT spring-cloud-starter spring-cloud-starter diff --git a/spring-cloud-test-support/pom.xml b/spring-cloud-test-support/pom.xml index 02b13a3a..f86e440b 100644 --- a/spring-cloud-test-support/pom.xml +++ b/spring-cloud-test-support/pom.xml @@ -7,7 +7,7 @@ org.springframework.cloud spring-cloud-commons-parent - 2.1.2.RELEASE + 2.1.3.BUILD-SNAPSHOT .. spring-cloud-test-support From ad4e8d6a2997ac6c8907477ece5e9007eac0a1f6 Mon Sep 17 00:00:00 2001 From: buildmaster Date: Thu, 20 Jun 2019 16:26:17 +0000 Subject: [PATCH 06/12] Bumping versions to 2.1.4.BUILD-SNAPSHOT after release --- docs/pom.xml | 2 +- pom.xml | 2 +- spring-cloud-commons-dependencies/pom.xml | 2 +- spring-cloud-commons/pom.xml | 2 +- spring-cloud-context-integration-tests/pom.xml | 2 +- spring-cloud-context/pom.xml | 2 +- spring-cloud-loadbalancer/pom.xml | 2 +- spring-cloud-starter/pom.xml | 2 +- spring-cloud-test-support/pom.xml | 2 +- 9 files changed, 9 insertions(+), 9 deletions(-) diff --git a/docs/pom.xml b/docs/pom.xml index 991cd090..d22c32bb 100644 --- a/docs/pom.xml +++ b/docs/pom.xml @@ -8,7 +8,7 @@ org.springframework.cloud spring-cloud-commons-parent - 2.1.3.BUILD-SNAPSHOT + 2.1.4.BUILD-SNAPSHOT pom Spring Cloud Commons Docs diff --git a/pom.xml b/pom.xml index dc54b920..db04080e 100644 --- a/pom.xml +++ b/pom.xml @@ -5,7 +5,7 @@ 4.0.0 org.springframework.cloud spring-cloud-commons-parent - 2.1.3.BUILD-SNAPSHOT + 2.1.4.BUILD-SNAPSHOT pom Spring Cloud Commons Parent Spring Cloud Commons Parent diff --git a/spring-cloud-commons-dependencies/pom.xml b/spring-cloud-commons-dependencies/pom.xml index 00fffb29..8a62ca07 100644 --- a/spring-cloud-commons-dependencies/pom.xml +++ b/spring-cloud-commons-dependencies/pom.xml @@ -10,7 +10,7 @@ spring-cloud-commons-dependencies - 2.1.3.BUILD-SNAPSHOT + 2.1.4.BUILD-SNAPSHOT pom spring-cloud-commons-dependencies Spring Cloud Commons Dependencies diff --git a/spring-cloud-commons/pom.xml b/spring-cloud-commons/pom.xml index 368deda5..1c478a18 100644 --- a/spring-cloud-commons/pom.xml +++ b/spring-cloud-commons/pom.xml @@ -7,7 +7,7 @@ org.springframework.cloud spring-cloud-commons-parent - 2.1.3.BUILD-SNAPSHOT + 2.1.4.BUILD-SNAPSHOT .. spring-cloud-commons diff --git a/spring-cloud-context-integration-tests/pom.xml b/spring-cloud-context-integration-tests/pom.xml index c7b62813..aa02ae13 100644 --- a/spring-cloud-context-integration-tests/pom.xml +++ b/spring-cloud-context-integration-tests/pom.xml @@ -7,7 +7,7 @@ org.springframework.cloud spring-cloud-commons-parent - 2.1.3.BUILD-SNAPSHOT + 2.1.4.BUILD-SNAPSHOT .. spring-cloud-context-integration-tests diff --git a/spring-cloud-context/pom.xml b/spring-cloud-context/pom.xml index 4ed22eb7..a9e339e3 100644 --- a/spring-cloud-context/pom.xml +++ b/spring-cloud-context/pom.xml @@ -7,7 +7,7 @@ org.springframework.cloud spring-cloud-commons-parent - 2.1.3.BUILD-SNAPSHOT + 2.1.4.BUILD-SNAPSHOT .. spring-cloud-context diff --git a/spring-cloud-loadbalancer/pom.xml b/spring-cloud-loadbalancer/pom.xml index cebcfe7a..7ae53830 100644 --- a/spring-cloud-loadbalancer/pom.xml +++ b/spring-cloud-loadbalancer/pom.xml @@ -7,7 +7,7 @@ org.springframework.cloud spring-cloud-commons-parent - 2.1.3.BUILD-SNAPSHOT + 2.1.4.BUILD-SNAPSHOT .. spring-cloud-loadbalancer diff --git a/spring-cloud-starter/pom.xml b/spring-cloud-starter/pom.xml index 83a5cf87..11b5a685 100644 --- a/spring-cloud-starter/pom.xml +++ b/spring-cloud-starter/pom.xml @@ -6,7 +6,7 @@ org.springframework.cloud spring-cloud-commons-parent - 2.1.3.BUILD-SNAPSHOT + 2.1.4.BUILD-SNAPSHOT spring-cloud-starter spring-cloud-starter diff --git a/spring-cloud-test-support/pom.xml b/spring-cloud-test-support/pom.xml index f86e440b..7c3d754b 100644 --- a/spring-cloud-test-support/pom.xml +++ b/spring-cloud-test-support/pom.xml @@ -7,7 +7,7 @@ org.springframework.cloud spring-cloud-commons-parent - 2.1.3.BUILD-SNAPSHOT + 2.1.4.BUILD-SNAPSHOT .. spring-cloud-test-support From 23ac3caabedf8f4ca79d633b6f6b96358d6af701 Mon Sep 17 00:00:00 2001 From: Spencer Gibb Date: Fri, 21 Jun 2019 11:44:03 -0400 Subject: [PATCH 07/12] Updates version to 2.1.3 bumps build to 2.1.7 --- docs/pom.xml | 4 ++-- pom.xml | 4 ++-- spring-cloud-commons-dependencies/pom.xml | 4 ++-- spring-cloud-commons/pom.xml | 2 +- spring-cloud-context-integration-tests/pom.xml | 2 +- spring-cloud-context/pom.xml | 2 +- spring-cloud-loadbalancer/pom.xml | 2 +- spring-cloud-starter/pom.xml | 2 +- spring-cloud-test-support/pom.xml | 2 +- 9 files changed, 12 insertions(+), 12 deletions(-) diff --git a/docs/pom.xml b/docs/pom.xml index d22c32bb..c4ce40d6 100644 --- a/docs/pom.xml +++ b/docs/pom.xml @@ -8,7 +8,7 @@ org.springframework.cloud spring-cloud-commons-parent - 2.1.4.BUILD-SNAPSHOT + 2.1.3.BUILD-SNAPSHOT pom Spring Cloud Commons Docs @@ -16,7 +16,7 @@ spring-cloud-commons ${basedir}/.. - 1.2.x,1.3.x,2.0.x,2.1.x + 1.3.x,2.0.x,2.1.x diff --git a/pom.xml b/pom.xml index db04080e..2fe15e00 100644 --- a/pom.xml +++ b/pom.xml @@ -5,7 +5,7 @@ 4.0.0 org.springframework.cloud spring-cloud-commons-parent - 2.1.4.BUILD-SNAPSHOT + 2.1.3.BUILD-SNAPSHOT pom Spring Cloud Commons Parent Spring Cloud Commons Parent @@ -13,7 +13,7 @@ org.springframework.cloud spring-cloud-build - 2.1.4.BUILD-SNAPSHOT + 2.1.7.BUILD-SNAPSHOT diff --git a/spring-cloud-commons-dependencies/pom.xml b/spring-cloud-commons-dependencies/pom.xml index 8a62ca07..e1abc087 100644 --- a/spring-cloud-commons-dependencies/pom.xml +++ b/spring-cloud-commons-dependencies/pom.xml @@ -6,11 +6,11 @@ spring-cloud-dependencies-parent org.springframework.cloud - 2.1.4.BUILD-SNAPSHOT + 2.1.7.BUILD-SNAPSHOT spring-cloud-commons-dependencies - 2.1.4.BUILD-SNAPSHOT + 2.1.3.BUILD-SNAPSHOT pom spring-cloud-commons-dependencies Spring Cloud Commons Dependencies diff --git a/spring-cloud-commons/pom.xml b/spring-cloud-commons/pom.xml index 1c478a18..368deda5 100644 --- a/spring-cloud-commons/pom.xml +++ b/spring-cloud-commons/pom.xml @@ -7,7 +7,7 @@ org.springframework.cloud spring-cloud-commons-parent - 2.1.4.BUILD-SNAPSHOT + 2.1.3.BUILD-SNAPSHOT .. spring-cloud-commons diff --git a/spring-cloud-context-integration-tests/pom.xml b/spring-cloud-context-integration-tests/pom.xml index aa02ae13..c7b62813 100644 --- a/spring-cloud-context-integration-tests/pom.xml +++ b/spring-cloud-context-integration-tests/pom.xml @@ -7,7 +7,7 @@ org.springframework.cloud spring-cloud-commons-parent - 2.1.4.BUILD-SNAPSHOT + 2.1.3.BUILD-SNAPSHOT .. spring-cloud-context-integration-tests diff --git a/spring-cloud-context/pom.xml b/spring-cloud-context/pom.xml index a9e339e3..4ed22eb7 100644 --- a/spring-cloud-context/pom.xml +++ b/spring-cloud-context/pom.xml @@ -7,7 +7,7 @@ org.springframework.cloud spring-cloud-commons-parent - 2.1.4.BUILD-SNAPSHOT + 2.1.3.BUILD-SNAPSHOT .. spring-cloud-context diff --git a/spring-cloud-loadbalancer/pom.xml b/spring-cloud-loadbalancer/pom.xml index 7ae53830..cebcfe7a 100644 --- a/spring-cloud-loadbalancer/pom.xml +++ b/spring-cloud-loadbalancer/pom.xml @@ -7,7 +7,7 @@ org.springframework.cloud spring-cloud-commons-parent - 2.1.4.BUILD-SNAPSHOT + 2.1.3.BUILD-SNAPSHOT .. spring-cloud-loadbalancer diff --git a/spring-cloud-starter/pom.xml b/spring-cloud-starter/pom.xml index 11b5a685..83a5cf87 100644 --- a/spring-cloud-starter/pom.xml +++ b/spring-cloud-starter/pom.xml @@ -6,7 +6,7 @@ org.springframework.cloud spring-cloud-commons-parent - 2.1.4.BUILD-SNAPSHOT + 2.1.3.BUILD-SNAPSHOT spring-cloud-starter spring-cloud-starter diff --git a/spring-cloud-test-support/pom.xml b/spring-cloud-test-support/pom.xml index 7c3d754b..f86e440b 100644 --- a/spring-cloud-test-support/pom.xml +++ b/spring-cloud-test-support/pom.xml @@ -7,7 +7,7 @@ org.springframework.cloud spring-cloud-commons-parent - 2.1.4.BUILD-SNAPSHOT + 2.1.3.BUILD-SNAPSHOT .. spring-cloud-test-support From 3eeee62272fa829f8bf9ad29d4e02e0662fc1c8b Mon Sep 17 00:00:00 2001 From: Olga Maciaszek-Sharma Date: Mon, 1 Jul 2019 14:01:39 +0200 Subject: [PATCH 08/12] Apply Spring Boot's solution of coercing `false` LogLevel to `OFF`. (#575) Fixes gh-556. --- .../cloud/logging/LoggingRebinder.java | 12 +++++++++++- .../cloud/logging/LoggingRebinderTests.java | 15 +++++++++++++++ 2 files changed, 26 insertions(+), 1 deletion(-) diff --git a/spring-cloud-context/src/main/java/org/springframework/cloud/logging/LoggingRebinder.java b/spring-cloud-context/src/main/java/org/springframework/cloud/logging/LoggingRebinder.java index a1f8a77d..132f1afa 100644 --- a/spring-cloud-context/src/main/java/org/springframework/cloud/logging/LoggingRebinder.java +++ b/spring-cloud-context/src/main/java/org/springframework/cloud/logging/LoggingRebinder.java @@ -17,6 +17,7 @@ package org.springframework.cloud.logging; import java.util.Collections; +import java.util.Locale; import java.util.Map; import java.util.Map.Entry; @@ -37,6 +38,7 @@ import org.springframework.core.env.Environment; * changed. * * @author Dave Syer + * @author Olga Maciaszek-Sharma * */ public class LoggingRebinder @@ -79,11 +81,19 @@ public class LoggingRebinder name = null; } level = environment.resolvePlaceholders(level); - system.setLogLevel(name, LogLevel.valueOf(level.toUpperCase())); + system.setLogLevel(name, resolveLogLevel(level)); } catch (RuntimeException ex) { this.logger.error("Cannot set level: " + level + " for '" + name + "'"); } } + private LogLevel resolveLogLevel(String level) { + String trimmedLevel = level.trim(); + if ("false".equalsIgnoreCase(trimmedLevel)) { + return LogLevel.OFF; + } + return LogLevel.valueOf(trimmedLevel.toUpperCase(Locale.ENGLISH)); + } + } diff --git a/spring-cloud-context/src/test/java/org/springframework/cloud/logging/LoggingRebinderTests.java b/spring-cloud-context/src/test/java/org/springframework/cloud/logging/LoggingRebinderTests.java index 9a626783..4b97c0bc 100644 --- a/spring-cloud-context/src/test/java/org/springframework/cloud/logging/LoggingRebinderTests.java +++ b/spring-cloud-context/src/test/java/org/springframework/cloud/logging/LoggingRebinderTests.java @@ -18,6 +18,7 @@ package org.springframework.cloud.logging; import java.util.Collections; +import ch.qos.logback.classic.Level; import org.junit.After; import org.junit.Test; import org.slf4j.Logger; @@ -33,6 +34,7 @@ import static org.assertj.core.api.BDDAssertions.then; /** * @author Dave Syer + * @author Olga Maciaszek-Sharma * */ public class LoggingRebinderTests { @@ -71,4 +73,17 @@ public class LoggingRebinderTests { then(this.logger.isTraceEnabled()).isTrue(); } + @Test + public void logLevelFalseResolvedToOff() { + ch.qos.logback.classic.Logger logger = (ch.qos.logback.classic.Logger) LoggerFactory + .getLogger("org.springframework.cloud"); + StandardEnvironment environment = new StandardEnvironment(); + TestPropertyValues.of("logging.level.org.springframework.cloud=false") + .applyTo(environment); + rebinder.setEnvironment(environment); + rebinder.onApplicationEvent(new EnvironmentChangeEvent(environment, + Collections.singleton("logging.level.org.springframework.cloud"))); + then(Level.OFF).isEqualTo((logger.getLevel())); + } + } From 28b972725072361a3e155715cb107fca99b84a73 Mon Sep 17 00:00:00 2001 From: Tim Ysewyn Date: Tue, 16 Jul 2019 18:28:26 +0200 Subject: [PATCH 09/12] Changed order of decrypting values. (#581) Reversed order of sources in CompositePropertySource to adhere to the order of sources returned by the Spring Cloud Config server. --- ...ironmentDecryptApplicationInitializer.java | 8 +++++-- ...entDecryptApplicationInitializerTests.java | 22 ++++++++++++------- 2 files changed, 20 insertions(+), 10 deletions(-) diff --git a/spring-cloud-context/src/main/java/org/springframework/cloud/bootstrap/encrypt/EnvironmentDecryptApplicationInitializer.java b/spring-cloud-context/src/main/java/org/springframework/cloud/bootstrap/encrypt/EnvironmentDecryptApplicationInitializer.java index c851f38d..430bedab 100644 --- a/spring-cloud-context/src/main/java/org/springframework/cloud/bootstrap/encrypt/EnvironmentDecryptApplicationInitializer.java +++ b/spring-cloud-context/src/main/java/org/springframework/cloud/bootstrap/encrypt/EnvironmentDecryptApplicationInitializer.java @@ -17,6 +17,7 @@ package org.springframework.cloud.bootstrap.encrypt; import java.util.ArrayList; +import java.util.Collections; import java.util.LinkedHashMap; import java.util.LinkedHashSet; import java.util.List; @@ -187,8 +188,11 @@ public class EnvironmentDecryptApplicationInitializer implements if (source instanceof CompositePropertySource) { - for (PropertySource nested : ((CompositePropertySource) source) - .getPropertySources()) { + List> sources = new ArrayList<>( + ((CompositePropertySource) source).getPropertySources()); + Collections.reverse(sources); + + for (PropertySource nested : sources) { decrypt(nested, overrides); } diff --git a/spring-cloud-context/src/test/java/org/springframework/cloud/bootstrap/encrypt/EnvironmentDecryptApplicationInitializerTests.java b/spring-cloud-context/src/test/java/org/springframework/cloud/bootstrap/encrypt/EnvironmentDecryptApplicationInitializerTests.java index 8e1decee..07137f9c 100644 --- a/spring-cloud-context/src/test/java/org/springframework/cloud/bootstrap/encrypt/EnvironmentDecryptApplicationInitializerTests.java +++ b/spring-cloud-context/src/test/java/org/springframework/cloud/bootstrap/encrypt/EnvironmentDecryptApplicationInitializerTests.java @@ -16,6 +16,7 @@ package org.springframework.cloud.bootstrap.encrypt; +import java.util.Arrays; import java.util.Collections; import java.util.Map; @@ -35,7 +36,7 @@ import org.springframework.security.crypto.encrypt.Encryptors; import org.springframework.security.crypto.encrypt.TextEncryptor; import static org.assertj.core.api.BDDAssertions.then; -import static org.mockito.ArgumentMatchers.anyString; +import static org.mockito.ArgumentMatchers.eq; import static org.mockito.Mockito.mock; import static org.mockito.Mockito.when; import static org.springframework.cloud.bootstrap.encrypt.EnvironmentDecryptApplicationInitializer.DECRYPTED_PROPERTY_SOURCE_NAME; @@ -154,23 +155,28 @@ public class EnvironmentDecryptApplicationInitializerTests { @Test public void testDecryptCompositePropertySource() { - String expected = "always"; TextEncryptor textEncryptor = mock(TextEncryptor.class); - when(textEncryptor.decrypt(anyString())).thenReturn(expected); + when(textEncryptor.decrypt(eq("value1"))).thenReturn("always1"); + when(textEncryptor.decrypt(eq("value2"))).thenReturn("always2"); ConfigurableApplicationContext ctx = new AnnotationConfigApplicationContext(); EnvironmentDecryptApplicationInitializer initializer = new EnvironmentDecryptApplicationInitializer( textEncryptor); - MapPropertySource source = new MapPropertySource("nobody", - Collections.singletonMap("key", "{cipher}value")); + MapPropertySource devProfile = new MapPropertySource("dev-profile", + Collections.singletonMap("key", "{cipher}value1")); + + MapPropertySource defaultProfile = new MapPropertySource("default-profile", + Collections.singletonMap("key", "{cipher}value2")); + CompositePropertySource cps = mock(CompositePropertySource.class); - when(cps.getPropertyNames()).thenReturn(source.getPropertyNames()); - when(cps.getPropertySources()).thenReturn(Collections.singleton(source)); + when(cps.getPropertyNames()).thenReturn(devProfile.getPropertyNames()); + when(cps.getPropertySources()) + .thenReturn(Arrays.asList(devProfile, defaultProfile)); ctx.getEnvironment().getPropertySources().addLast(cps); initializer.initialize(ctx); - then(ctx.getEnvironment().getProperty("key")).isEqualTo(expected); + then(ctx.getEnvironment().getProperty("key")).isEqualTo("always1"); } } From c19575b978010c8e8cf8dcf67c75559b04e036ad Mon Sep 17 00:00:00 2001 From: Tim Ysewyn Date: Fri, 26 Jul 2019 12:13:51 +0200 Subject: [PATCH 10/12] Added optimizations to EnvironmentDecryptApplicationInitializer --- ...ironmentDecryptApplicationInitializer.java | 140 ++++++++++++------ ...entDecryptApplicationInitializerTests.java | 96 ++++++++++-- 2 files changed, 179 insertions(+), 57 deletions(-) diff --git a/spring-cloud-context/src/main/java/org/springframework/cloud/bootstrap/encrypt/EnvironmentDecryptApplicationInitializer.java b/spring-cloud-context/src/main/java/org/springframework/cloud/bootstrap/encrypt/EnvironmentDecryptApplicationInitializer.java index 430bedab..100b7c44 100644 --- a/spring-cloud-context/src/main/java/org/springframework/cloud/bootstrap/encrypt/EnvironmentDecryptApplicationInitializer.java +++ b/spring-cloud-context/src/main/java/org/springframework/cloud/bootstrap/encrypt/EnvironmentDecryptApplicationInitializer.java @@ -63,6 +63,11 @@ public class EnvironmentDecryptApplicationInitializer implements */ public static final String DECRYPTED_BOOTSTRAP_PROPERTY_SOURCE_NAME = "decryptedBootstrap"; + /** + * Prefix indicating an encrypted value. + */ + public static final String ENCRYPTED_PROPERTY_PREFIX = "{cipher}"; + private static final Pattern COLLECTION_PROPERTY = Pattern .compile("(\\S+)?\\[(\\d+)\\](\\.\\S+)?"); @@ -98,11 +103,24 @@ public class EnvironmentDecryptApplicationInitializer implements @Override public void initialize(ConfigurableApplicationContext applicationContext) { - ConfigurableEnvironment environment = applicationContext.getEnvironment(); MutablePropertySources propertySources = environment.getPropertySources(); Set found = new LinkedHashSet<>(); + if (!propertySources.contains(DECRYPTED_BOOTSTRAP_PROPERTY_SOURCE_NAME)) { + // No reason to decrypt bootstrap twice + PropertySource bootstrap = propertySources + .get(BootstrapApplicationListener.BOOTSTRAP_PROPERTY_SOURCE_NAME); + if (bootstrap != null) { + Map map = decrypt(bootstrap); + if (!map.isEmpty()) { + found.addAll(map.keySet()); + insert(applicationContext, new SystemEnvironmentPropertySource( + DECRYPTED_BOOTSTRAP_PROPERTY_SOURCE_NAME, map)); + } + } + } + removeDecryptedProperties(applicationContext); Map map = decrypt(propertySources); if (!map.isEmpty()) { // We have some decrypted properties @@ -110,16 +128,6 @@ public class EnvironmentDecryptApplicationInitializer implements insert(applicationContext, new SystemEnvironmentPropertySource( DECRYPTED_PROPERTY_SOURCE_NAME, map)); } - PropertySource bootstrap = propertySources - .get(BootstrapApplicationListener.BOOTSTRAP_PROPERTY_SOURCE_NAME); - if (bootstrap != null) { - map = decrypt(bootstrap); - if (!map.isEmpty()) { - found.addAll(map.keySet()); - insert(applicationContext, new SystemEnvironmentPropertySource( - DECRYPTED_BOOTSTRAP_PROPERTY_SOURCE_NAME, map)); - } - } if (!found.isEmpty()) { ApplicationContext parent = applicationContext.getParent(); if (parent != null) { @@ -166,26 +174,48 @@ public class EnvironmentDecryptApplicationInitializer implements } } + private void removeDecryptedProperties(ApplicationContext applicationContext) { + ApplicationContext parent = applicationContext; + while (parent != null) { + if (parent.getEnvironment() instanceof ConfigurableEnvironment) { + ((ConfigurableEnvironment) parent.getEnvironment()).getPropertySources() + .remove(DECRYPTED_PROPERTY_SOURCE_NAME); + } + parent = parent.getParent(); + } + } + public Map decrypt(PropertySources propertySources) { - Map overrides = new LinkedHashMap<>(); + Map properties = merge(propertySources); + decrypt(properties); + return properties; + } + + private Map decrypt(PropertySource source) { + Map properties = merge(source); + decrypt(properties); + return properties; + } + + private Map merge(PropertySources propertySources) { + Map properties = new LinkedHashMap<>(); List> sources = new ArrayList<>(); for (PropertySource source : propertySources) { sources.add(0, source); } for (PropertySource source : sources) { - decrypt(source, overrides); + merge(source, properties); } - return overrides; + return properties; } - private Map decrypt(PropertySource source) { - Map overrides = new LinkedHashMap<>(); - decrypt(source, overrides); - return overrides; + private Map merge(PropertySource source) { + Map properties = new LinkedHashMap<>(); + merge(source, properties); + return properties; } - private void decrypt(PropertySource source, Map overrides) { - + private void merge(PropertySource source, Map properties) { if (source instanceof CompositePropertySource) { List> sources = new ArrayList<>( @@ -193,7 +223,7 @@ public class EnvironmentDecryptApplicationInitializer implements Collections.reverse(sources); for (PropertySource nested : sources) { - decrypt(nested, overrides); + merge(nested, properties); } } @@ -206,47 +236,63 @@ public class EnvironmentDecryptApplicationInitializer implements Object property = source.getProperty(key); if (property != null) { String value = property.toString(); - if (value.startsWith("{cipher}")) { - value = value.substring("{cipher}".length()); - try { - value = this.encryptor.decrypt(value); - if (logger.isDebugEnabled()) { - logger.debug("Decrypted: key=" + key); - } - } - catch (Exception e) { - String message = "Cannot decrypt: key=" + key; - if (this.failOnError) { - throw new IllegalStateException(message, e); - } - if (logger.isDebugEnabled()) { - logger.warn(message, e); - } - else { - logger.warn(message); - } - // Set value to empty to avoid making a password out of the - // cipher text - value = ""; - } - overrides.put(key, value); + if (value.startsWith(ENCRYPTED_PROPERTY_PREFIX)) { + properties.put(key, value); if (COLLECTION_PROPERTY.matcher(key).matches()) { sourceHasDecryptedCollection = true; } } else if (COLLECTION_PROPERTY.matcher(key).matches()) { - // put non-ecrypted properties so merging of index properties + // put non-encrypted properties so merging of index properties // happens correctly otherCollectionProperties.put(key, value); } + else { + // override previously encrypted with non-encrypted property + properties.remove(key); + } } } // copy all indexed properties even if not encrypted if (sourceHasDecryptedCollection && !otherCollectionProperties.isEmpty()) { - overrides.putAll(otherCollectionProperties); + properties.putAll(otherCollectionProperties); } } } + private void decrypt(Map properties) { + properties.replaceAll((key, value) -> { + String valueString = value.toString(); + if (!valueString.startsWith(ENCRYPTED_PROPERTY_PREFIX)) { + return value; + } + return decrypt(key, valueString); + }); + } + + private String decrypt(String key, String original) { + String value = original.substring(ENCRYPTED_PROPERTY_PREFIX.length()); + try { + value = this.encryptor.decrypt(value); + if (logger.isDebugEnabled()) { + logger.debug("Decrypted: key=" + key); + } + return value; + } + catch (Exception e) { + String message = "Cannot decrypt: key=" + key; + if (this.failOnError) { + throw new IllegalStateException(message, e); + } + if (logger.isDebugEnabled()) { + logger.warn(message, e); + } + else { + logger.warn(message); + } + return ""; + } + } + } diff --git a/spring-cloud-context/src/test/java/org/springframework/cloud/bootstrap/encrypt/EnvironmentDecryptApplicationInitializerTests.java b/spring-cloud-context/src/test/java/org/springframework/cloud/bootstrap/encrypt/EnvironmentDecryptApplicationInitializerTests.java index 07137f9c..268c38d6 100644 --- a/spring-cloud-context/src/test/java/org/springframework/cloud/bootstrap/encrypt/EnvironmentDecryptApplicationInitializerTests.java +++ b/spring-cloud-context/src/test/java/org/springframework/cloud/bootstrap/encrypt/EnvironmentDecryptApplicationInitializerTests.java @@ -18,6 +18,7 @@ package org.springframework.cloud.bootstrap.encrypt; import java.util.Arrays; import java.util.Collections; +import java.util.HashMap; import java.util.Map; import org.junit.Test; @@ -36,9 +37,13 @@ import org.springframework.security.crypto.encrypt.Encryptors; import org.springframework.security.crypto.encrypt.TextEncryptor; import static org.assertj.core.api.BDDAssertions.then; -import static org.mockito.ArgumentMatchers.eq; import static org.mockito.Mockito.mock; +import static org.mockito.Mockito.times; +import static org.mockito.Mockito.verify; +import static org.mockito.Mockito.verifyNoMoreInteractions; import static org.mockito.Mockito.when; +import static org.springframework.cloud.bootstrap.BootstrapApplicationListener.BOOTSTRAP_PROPERTY_SOURCE_NAME; +import static org.springframework.cloud.bootstrap.encrypt.EnvironmentDecryptApplicationInitializer.DECRYPTED_BOOTSTRAP_PROPERTY_SOURCE_NAME; import static org.springframework.cloud.bootstrap.encrypt.EnvironmentDecryptApplicationInitializer.DECRYPTED_PROPERTY_SOURCE_NAME; /** @@ -71,9 +76,8 @@ public class EnvironmentDecryptApplicationInitializerTests { public void propertySourcesOrderedCorrectly() { ConfigurableApplicationContext context = new AnnotationConfigApplicationContext(); TestPropertyValues.of("foo: {cipher}bar").applyTo(context); - context.getEnvironment().getPropertySources() - .addFirst(new MapPropertySource("test_override", - Collections.singletonMap("foo", "{cipher}spam"))); + context.getEnvironment().getPropertySources().addFirst(new MapPropertySource( + "test_override", Collections.singletonMap("foo", "{cipher}spam"))); this.listener.initialize(context); then(context.getEnvironment().getProperty("foo")).isEqualTo("spam"); } @@ -155,13 +159,9 @@ public class EnvironmentDecryptApplicationInitializerTests { @Test public void testDecryptCompositePropertySource() { - TextEncryptor textEncryptor = mock(TextEncryptor.class); - when(textEncryptor.decrypt(eq("value1"))).thenReturn("always1"); - when(textEncryptor.decrypt(eq("value2"))).thenReturn("always2"); - ConfigurableApplicationContext ctx = new AnnotationConfigApplicationContext(); EnvironmentDecryptApplicationInitializer initializer = new EnvironmentDecryptApplicationInitializer( - textEncryptor); + Encryptors.noOpText()); MapPropertySource devProfile = new MapPropertySource("dev-profile", Collections.singletonMap("key", "{cipher}value1")); @@ -176,7 +176,83 @@ public class EnvironmentDecryptApplicationInitializerTests { ctx.getEnvironment().getPropertySources().addLast(cps); initializer.initialize(ctx); - then(ctx.getEnvironment().getProperty("key")).isEqualTo("always1"); + then(ctx.getEnvironment().getProperty("key")).isEqualTo("value1"); + } + + @Test + public void propertySourcesOrderedCorrectlyWithUnencryptedOverrides() { + ConfigurableApplicationContext context = new AnnotationConfigApplicationContext(); + TestPropertyValues.of("foo: {cipher}bar").applyTo(context); + context.getEnvironment().getPropertySources().addFirst(new MapPropertySource( + "test_override", Collections.singletonMap("foo", "spam"))); + this.listener.initialize(context); + then(context.getEnvironment().getProperty("foo")).isEqualTo("spam"); + } + + @Test + public void doNotDecryptBootstrapTwice() { + TextEncryptor encryptor = mock(TextEncryptor.class); + when(encryptor.decrypt("bar")).thenReturn("bar"); + when(encryptor.decrypt("bar2")).thenReturn("bar2"); + when(encryptor.decrypt("bar3")).thenReturn("bar3"); + when(encryptor.decrypt("baz")).thenReturn("baz"); + + EnvironmentDecryptApplicationInitializer initializer = new EnvironmentDecryptApplicationInitializer( + encryptor); + + ConfigurableApplicationContext context = new AnnotationConfigApplicationContext(); + CompositePropertySource bootstrap = new CompositePropertySource( + BOOTSTRAP_PROPERTY_SOURCE_NAME); + bootstrap.addPropertySource(new MapPropertySource("configService", + Collections.singletonMap("foo", "{cipher}bar"))); + context.getEnvironment().getPropertySources().addFirst(bootstrap); + + Map props = new HashMap<>(); + props.put("foo2", "{cipher}bar2"); + props.put("bar", "{cipher}baz"); + context.getEnvironment().getPropertySources().addAfter( + BOOTSTRAP_PROPERTY_SOURCE_NAME, new MapPropertySource("remote", props)); + + initializer.initialize(context); + + // Simulate retrieval of new properties via Spring Cloud Config + props.put("foo2", "{cipher}bar3"); + context.getEnvironment().getPropertySources().replace("remote", + new MapPropertySource("remote", props)); + + initializer.initialize(context); + + verify(encryptor).decrypt("bar"); + verify(encryptor).decrypt("bar2"); + verify(encryptor).decrypt("bar3"); + verify(encryptor, times(2)).decrypt("baz"); + + // Check if all encrypted properties are still decrypted + PropertySource decryptedBootstrap = context.getEnvironment() + .getPropertySources().get(DECRYPTED_BOOTSTRAP_PROPERTY_SOURCE_NAME); + then(decryptedBootstrap.getProperty("foo")).isEqualTo("bar"); + + PropertySource decrypted = context.getEnvironment().getPropertySources() + .get(DECRYPTED_PROPERTY_SOURCE_NAME); + then(decrypted.getProperty("foo2")).isEqualTo("bar3"); + then(decrypted.getProperty("bar")).isEqualTo("baz"); + } + + @Test + public void testOnlyDecryptIfNotOverridden() { + ConfigurableApplicationContext context = new AnnotationConfigApplicationContext(); + TextEncryptor encryptor = mock(TextEncryptor.class); + when(encryptor.decrypt("bar2")).thenReturn("bar2"); + EnvironmentDecryptApplicationInitializer initializer = new EnvironmentDecryptApplicationInitializer( + encryptor); + TestPropertyValues.of("foo: {cipher}bar", "foo2: {cipher}bar2").applyTo(context); + context.getEnvironment().getPropertySources().addFirst(new MapPropertySource( + "test_override", Collections.singletonMap("foo", "spam"))); + initializer.initialize(context); + then(context.getEnvironment().getProperty("foo")).isEqualTo("spam"); + then(context.getEnvironment().getProperty("foo2")).isEqualTo("bar2"); + verify(encryptor).decrypt("bar2"); + verifyNoMoreInteractions(encryptor); } } From 86650dc215018ed087f4ac20536a142915f79a43 Mon Sep 17 00:00:00 2001 From: Tim Ysewyn Date: Fri, 26 Jul 2019 15:58:13 +0200 Subject: [PATCH 11/12] Updated license header --- .../encrypt/EnvironmentDecryptApplicationInitializer.java | 4 ++-- .../EnvironmentDecryptApplicationInitializerTests.java | 3 ++- 2 files changed, 4 insertions(+), 3 deletions(-) diff --git a/spring-cloud-context/src/main/java/org/springframework/cloud/bootstrap/encrypt/EnvironmentDecryptApplicationInitializer.java b/spring-cloud-context/src/main/java/org/springframework/cloud/bootstrap/encrypt/EnvironmentDecryptApplicationInitializer.java index 100b7c44..327fd86c 100644 --- a/spring-cloud-context/src/main/java/org/springframework/cloud/bootstrap/encrypt/EnvironmentDecryptApplicationInitializer.java +++ b/spring-cloud-context/src/main/java/org/springframework/cloud/bootstrap/encrypt/EnvironmentDecryptApplicationInitializer.java @@ -1,5 +1,5 @@ /* - * Copyright 2013-2017 the original author or authors. + * Copyright 2013-2019 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -48,7 +48,7 @@ import org.springframework.security.crypto.encrypt.TextEncryptor; * override the encrypted values. * * @author Dave Syer - * + * @author Tim Ysewyn */ public class EnvironmentDecryptApplicationInitializer implements ApplicationContextInitializer, Ordered { diff --git a/spring-cloud-context/src/test/java/org/springframework/cloud/bootstrap/encrypt/EnvironmentDecryptApplicationInitializerTests.java b/spring-cloud-context/src/test/java/org/springframework/cloud/bootstrap/encrypt/EnvironmentDecryptApplicationInitializerTests.java index 268c38d6..a6ab2e6f 100644 --- a/spring-cloud-context/src/test/java/org/springframework/cloud/bootstrap/encrypt/EnvironmentDecryptApplicationInitializerTests.java +++ b/spring-cloud-context/src/test/java/org/springframework/cloud/bootstrap/encrypt/EnvironmentDecryptApplicationInitializerTests.java @@ -1,5 +1,5 @@ /* - * Copyright 2012-2019 the original author or authors. + * Copyright 2013-2019 the original author or authors. * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -49,6 +49,7 @@ import static org.springframework.cloud.bootstrap.encrypt.EnvironmentDecryptAppl /** * @author Dave Syer * @author Biju Kunjummen + * @author Tim Ysewyn */ public class EnvironmentDecryptApplicationInitializerTests { From 36355c025d9878acb2f93adc05267ded08c40bf9 Mon Sep 17 00:00:00 2001 From: Marcin Grzejszczak Date: Mon, 29 Jul 2019 14:28:21 +0200 Subject: [PATCH 12/12] Added symbolic link of index.adoc --- docs/src/main/asciidoc/index.adoc | 1 + 1 file changed, 1 insertion(+) create mode 120000 docs/src/main/asciidoc/index.adoc diff --git a/docs/src/main/asciidoc/index.adoc b/docs/src/main/asciidoc/index.adoc new file mode 120000 index 00000000..583ddeb0 --- /dev/null +++ b/docs/src/main/asciidoc/index.adoc @@ -0,0 +1 @@ +spring-cloud-commons.adoc \ No newline at end of file