Polish contribution

Along with surrounding map keys with dot from VCAP_SERVICES with `[ ]`,
this commit also does that for non-alphanumeric and `-` characters so that
they are not stripped off later.

See gh-18915

Author: mbhave

spring-boot-project/spring-boot/src/main/java/org/springframework/boot/cloud/CloudFoundryVcapEnvironmentPostProcessor.java

@@ -87,6 +87,7 @@
  *
  * @author Dave Syer
  * @author Andy Wilkinson
+ * @author Madhura Bhave
  * @since 1.3.0
  */
 public class CloudFoundryVcapEnvironmentPostProcessor
@@ -230,10 +231,19 @@ private String getPropertyName(String path, String key) {
 		if (key.startsWith("[")) {
 			return path + key;
 		}
-		if (key.contains(".")) {
+		if (shouldWrap(key)) {
 			return path + "[" + key + "]";
 		}
 		return path + "." + key;
 	}
 
+	private boolean shouldWrap(String key) {
+		for (char ch : key.toCharArray()) {
+			if (!Character.isLowerCase(ch) && !Character.isDigit(ch) && ch != '-') {
+				return true;
+			}
+		}
+		return false;
+	}
+
 }

spring-boot-project/spring-boot/src/test/java/org/springframework/boot/cloud/cloudfoundry/CloudFoundryVcapEnvironmentPostProcessorTests.java

@@ -30,6 +30,8 @@
  *
  * @author Dave Syer
  * @author Andy Wilkinson
+ * @author Hans Schulz
+ * @author Madhura Bhave
  */
 public class CloudFoundryVcapEnvironmentPostProcessorTests {
 
@@ -118,7 +120,7 @@ public void testServicePropertiesWithoutNA() {
 	}
 
 	@Test
-	void testServicePropertiesContainingKeysWithDot() {
+	public void testServicePropertiesContainingKeysWithDot() {
 		TestPropertySourceUtils.addInlinedPropertiesToEnvironment(this.context,
 				"VCAP_SERVICES={\"user-provided\":[{\"name\":\"test\",\"label\":\"test-label\","
 						+ "\"credentials\":{\"key.with.dots\":\"some-value\"}}]}");
@@ -127,6 +129,16 @@ void testServicePropertiesContainingKeysWithDot() {
 		assertThat(getProperty("vcap.services.test.credentials[key.with.dots]")).isEqualTo("some-value");
 	}
 
+	@Test
+	public void testServicePropertiesContainingKeysWithUpperCaseAndNonAlphaNumericCharacters() {
+		TestPropertySourceUtils.addInlinedPropertiesToEnvironment(this.context,
+				"VCAP_SERVICES={\"user-provided\":[{\"name\":\"test\",\"label\":\"test-label\","
+						+ "\"credentials\":{\"My-Key\":\"some-value\", \"foo@\":\"bar\"}}]}");
+		this.initializer.postProcessEnvironment(this.context.getEnvironment(), null);
+		assertThat(getProperty("vcap.services.test.credentials[My-Key]")).isEqualTo("some-value");
+		assertThat(getProperty("vcap.services.test.credentials[foo@]")).isEqualTo("bar");
+	}
+
 	private String getProperty(String key) {
 		return this.context.getEnvironment().getProperty(key);
 	}