Adapt to new Placeholder resolution API

snicoll · May 2, 2024 · caa3f7e · caa3f7e
1 parent a4ba084
commit caa3f7e
Show file tree

Hide file tree

Showing 4 changed files with 10 additions and 4 deletions.
diff --git a/...rc/main/java/org/springframework/boot/autoconfigure/jdbc/DataSourceAutoConfiguration.java b/...rc/main/java/org/springframework/boot/autoconfigure/jdbc/DataSourceAutoConfiguration.java
@@ -157,6 +157,7 @@ private boolean hasDataSourceUrlProperty(ConditionContext context) {
 					return StringUtils.hasText(environment.getProperty(DATASOURCE_URL_PROPERTY));
 				}
 				catch (IllegalArgumentException ex) {
+					// NOTE: This should be PlaceholderResolutionException
 					// Ignore unresolvable placeholder errors
 				}
 			}

diff --git a/.../springframework/boot/test/web/SpringBootTestRandomPortEnvironmentPostProcessorTests.java b/.../springframework/boot/test/web/SpringBootTestRandomPortEnvironmentPostProcessorTests.java
@@ -169,6 +169,7 @@ void postProcessWhenManagementServerPortPlaceholderAbsentShouldFail() {
 		addTestPropertySource("0", null);
 		this.propertySources
 			.addLast(new MapPropertySource("other", Collections.singletonMap("management.server.port", "${port}")));
+		// NOTE: This should be PlaceholderResolutionException
 		assertThatIllegalArgumentException()
 			.isThrownBy(() -> this.postProcessor.postProcessEnvironment(this.environment, null))
 			.withMessage("Could not resolve placeholder 'port' in value \"${port}\"");
@@ -196,6 +197,7 @@ void postProcessWhenServerPortPlaceholderAbsentShouldFail() {
 		source.put("server.port", "${port}");
 		source.put("management.server.port", "9090");
 		this.propertySources.addLast(new MapPropertySource("other", source));
+		// NOTE: This should be PlaceholderResolutionException
 		assertThatIllegalArgumentException()
 			.isThrownBy(() -> this.postProcessor.postProcessEnvironment(this.environment, null))
 			.withMessage("Could not resolve placeholder 'port' in value \"${port}\"");

diff --git a/...ngframework/boot/context/config/ConfigDataEnvironmentContributorPlaceholdersResolver.java b/...ngframework/boot/context/config/ConfigDataEnvironmentContributorPlaceholdersResolver.java
@@ -56,7 +56,8 @@ class ConfigDataEnvironmentContributorPlaceholdersResolver implements Placeholde
 		this.failOnResolveFromInactiveContributor = failOnResolveFromInactiveContributor;
 		this.conversionService = conversionService;
 		this.helper = new PropertyPlaceholderHelper(SystemPropertyUtils.PLACEHOLDER_PREFIX,
-				SystemPropertyUtils.PLACEHOLDER_SUFFIX, SystemPropertyUtils.VALUE_SEPARATOR, true);
+				SystemPropertyUtils.PLACEHOLDER_SUFFIX, SystemPropertyUtils.VALUE_SEPARATOR,
+				SystemPropertyUtils.ESCAPE_CHARACTER, true);
 	}
 
 	@Override

diff --git a/...org/springframework/boot/context/properties/bind/PropertySourcesPlaceholdersResolver.java b/...org/springframework/boot/context/properties/bind/PropertySourcesPlaceholdersResolver.java
@@ -1,5 +1,5 @@
 /*
- * Copyright 2012-2022 the original author or authors.
+ * Copyright 2012-2024 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.
@@ -47,8 +47,10 @@ public PropertySourcesPlaceholdersResolver(Iterable<PropertySource<?>> sources)
 
 	public PropertySourcesPlaceholdersResolver(Iterable<PropertySource<?>> sources, PropertyPlaceholderHelper helper) {
 		this.sources = sources;
-		this.helper = (helper != null) ? helper : new PropertyPlaceholderHelper(SystemPropertyUtils.PLACEHOLDER_PREFIX,
-				SystemPropertyUtils.PLACEHOLDER_SUFFIX, SystemPropertyUtils.VALUE_SEPARATOR, true);
+		this.helper = (helper != null) ? helper
+				: new PropertyPlaceholderHelper(SystemPropertyUtils.PLACEHOLDER_PREFIX,
+						SystemPropertyUtils.PLACEHOLDER_SUFFIX, SystemPropertyUtils.VALUE_SEPARATOR,
+						SystemPropertyUtils.ESCAPE_CHARACTER, true);
 	}
 
 	@Override