Browse Source

Switch default in id from 0 to 8080 if server.port is not set. Fixes #1154 (#1155)

Co-authored-by: Ryan Baxter <524254+ryanjbaxter@users.noreply.github.com>
pull/1167/head
Ryan Baxter 2 years ago committed by GitHub
parent
commit
12beb8cc08
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
  1. 4
      spring-cloud-commons/src/main/java/org/springframework/cloud/commons/util/IdUtils.java
  2. 2
      spring-cloud-commons/src/test/java/org/springframework/cloud/commons/util/IdUtilsTests.java

4
spring-cloud-commons/src/main/java/org/springframework/cloud/commons/util/IdUtils.java

@ -27,9 +27,9 @@ public final class IdUtils { @@ -27,9 +27,9 @@ public final class IdUtils {
private static final String SEPARATOR = ":";
// @checkstyle:off
public static final String DEFAULT_SERVICE_ID_STRING = "${vcap.application.name:${spring.application.name:application}}:${vcap.application.instance_index:${spring.application.index:${local.server.port:${server.port:0}}}}:${vcap.application.instance_id:${cachedrandom.${vcap.application.name:${spring.application.name:application}}.value}}";
public static final String DEFAULT_SERVICE_ID_STRING = "${vcap.application.name:${spring.application.name:application}}:${vcap.application.instance_index:${spring.application.index:${local.server.port:${server.port:8080}}}}:${vcap.application.instance_id:${cachedrandom.${vcap.application.name:${spring.application.name:application}}.value}}";
public static final String DEFAULT_SERVICE_ID_WITH_ACTIVE_PROFILES_STRING = "${vcap.application.name:${spring.application.name:application}:${spring.profiles.active}}:${vcap.application.instance_index:${spring.application.index:${local.server.port:${server.port:0}}}}:${vcap.application.instance_id:${cachedrandom.${vcap.application.name:${spring.application.name:application}}.value}}";
public static final String DEFAULT_SERVICE_ID_WITH_ACTIVE_PROFILES_STRING = "${vcap.application.name:${spring.application.name:application}:${spring.profiles.active}}:${vcap.application.instance_index:${spring.application.index:${local.server.port:${server.port:8080}}}}:${vcap.application.instance_id:${cachedrandom.${vcap.application.name:${spring.application.name:application}}.value}}";
// @checkstyle:on

2
spring-cloud-commons/src/test/java/org/springframework/cloud/commons/util/IdUtilsTests.java

@ -125,7 +125,7 @@ public class IdUtilsTests { @@ -125,7 +125,7 @@ public class IdUtilsTests {
@Test
public void testServiceIdDefaults() {
this.env.setProperty("cachedrandom.application.value", "123abc");
then("application:0:123abc").isEqualTo(IdUtils.getResolvedServiceId(this.env));
then("application:8080:123abc").isEqualTo(IdUtils.getResolvedServiceId(this.env));
}
@Test

Loading…
Cancel
Save