Fix the order in which the postprocessors run. (#1333)

Author: OlgaMaciaszek

spring-cloud-commons/src/main/java/org/springframework/cloud/client/HostInfoEnvironmentPostProcessor.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2012-2020 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.
@@ -19,6 +19,7 @@
 import java.util.LinkedHashMap;
 
 import org.springframework.boot.SpringApplication;
+import org.springframework.boot.context.config.ConfigDataEnvironmentPostProcessor;
 import org.springframework.boot.context.properties.bind.Bindable;
 import org.springframework.boot.context.properties.bind.Binder;
 import org.springframework.boot.context.properties.source.ConfigurationPropertySources;
@@ -32,15 +33,17 @@
 
 /**
  * @author Spencer Gibb
+ * @author Olga Maciaszek-Sharma
  */
 public class HostInfoEnvironmentPostProcessor implements EnvironmentPostProcessor, Ordered {
 
-	// Before BootstrapConfigFileApplicationListener
-	private int order = Ordered.HIGHEST_PRECEDENCE + 9;
+	// Before BootstrapConfigFileApplicationListener, but after
+	// ConfigDataEnvironmentPostProcessor
+	private static final int ORDER = Math.addExact(ConfigDataEnvironmentPostProcessor.ORDER, 1);
 
 	@Override
 	public int getOrder() {
-		return this.order;
+		return ORDER;
 	}
 
 	@Override

spring-cloud-context/src/main/java/org/springframework/cloud/bootstrap/BootstrapConfigFileApplicationListener.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2013-2020 the original author or authors.
+ * Copyright 2013-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.
@@ -85,7 +85,7 @@
 
 /**
  * {@link EnvironmentPostProcessor} that configures the context environment by loading
- * properties from well known file locations. By default properties will be loaded from
+ * properties from well known file locations. By default, properties will be loaded from
  * 'application.properties' and/or 'application.yml' files in the following locations:
  * <ul>
  * <li>file:./config/</li>
@@ -116,6 +116,7 @@
  * @author Eddú Meléndez
  * @author Madhura Bhave
  * @author Scott Frederick
+ * @author Olga Maciaszek-Sharma
  * @since 1.0.0 {@link ConfigDataEnvironmentPostProcessor}
  */
 public class BootstrapConfigFileApplicationListener
@@ -168,9 +169,9 @@ public class BootstrapConfigFileApplicationListener
 	 */
 	public static final int DEFAULT_ORDER =
 			// This listener needs to run after the `ConfigDataEnvironmentPostProcessor`
-			// to
-			// make sure the `Environment.activeProfiles` are correctly set
-			Math.addExact(ConfigDataEnvironmentPostProcessor.ORDER, 1);
+			// and `HostInfoEnvironmentPostProcessor`
+			// to make sure the `Environment.activeProfiles` are correctly set
+			Math.addExact(ConfigDataEnvironmentPostProcessor.ORDER, 2);
 
 	private final Log logger;
 

spring-cloud-loadbalancer/src/test/java/org/springframework/cloud/loadbalancer/core/SubsetServiceInstanceListSupplierTest.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2012-2023 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.
@@ -40,7 +40,7 @@
 import static org.springframework.cloud.loadbalancer.core.LoadBalancerTestUtils.buildLoadBalancerClientFactory;
 
 /**
- * Tests for {@link SubsetServiceInstanceListSupplier}
+ * Tests for {@link SubsetServiceInstanceListSupplier}.
  *
  * @author Zhuozhi Ji
  */