Skip to content

Commit 976619f

Browse files
committed
Fix: use application_id from VCAP
1 parent 80eaf61 commit 976619f

File tree

4 files changed

+6
-5
lines changed

4 files changed

+6
-5
lines changed

spring-boot-admin-client/src/main/java/de/codecentric/boot/admin/client/config/CloudFoundryApplicationProperties.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@
2323
@lombok.Data
2424
@ConfigurationProperties("vcap.application")
2525
public class CloudFoundryApplicationProperties {
26-
private String instanceId;
26+
private String applicationId;
2727
private String instanceIndex;
2828
private List<String> uris = new ArrayList<>();
2929
}

spring-boot-admin-client/src/main/java/de/codecentric/boot/admin/client/registration/metadata/CloudFoundryMetadataContributor.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -32,10 +32,10 @@ public CloudFoundryMetadataContributor(CloudFoundryApplicationProperties cfAppli
3232

3333
@Override
3434
public Map<String, String> getMetadata() {
35-
if (StringUtils.hasText(this.cfApplicationProperties.getInstanceId()) &&
35+
if (StringUtils.hasText(this.cfApplicationProperties.getApplicationId()) &&
3636
StringUtils.hasText(this.cfApplicationProperties.getInstanceIndex())) {
3737
Map<String, String> map = new HashMap<>();
38-
map.put("applicationId", this.cfApplicationProperties.getInstanceId());
38+
map.put("applicationId", this.cfApplicationProperties.getApplicationId());
3939
map.put("instanceId", this.cfApplicationProperties.getInstanceIndex());
4040
return map;
4141
}

spring-boot-admin-client/src/test/java/de/codecentric/boot/admin/client/config/SpringBootAdminClientCloudFoundryAutoConfigurationTest.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -55,6 +55,7 @@ public void non_cloud_platform() {
5555
@Test
5656
public void cloudfoundry() {
5757
String vcap = "VCAP_APPLICATION:{\"application_users\":[]," +
58+
"\"application_id\":\"9958288f-9842-4ddc-93dd-1ea3c90634cd\"," +
5859
"\"instance_id\":\"bb7935245adf3e650dfb7c58a06e9ece\"," +
5960
"\"instance_index\":0,\"version\":\"3464e092-1c13-462e-a47c-807c30318a50\"," +
6061
"\"name\":\"foo\",\"uris\":[\"foo.cfapps.io\"]," +
@@ -70,7 +71,7 @@ public void cloudfoundry() {
7071
load("spring.boot.admin.client.url:http://localhost:8081", vcap);
7172
ApplicationFactory factory = context.getBean(ApplicationFactory.class);
7273
CloudFoundryMetadataContributor contributor = context.getBean(CloudFoundryMetadataContributor.class);
73-
assertThat(contributor.getMetadata()).containsEntry("applicationId", "bb7935245adf3e650dfb7c58a06e9ece")
74+
assertThat(contributor.getMetadata()).containsEntry("applicationId", "9958288f-9842-4ddc-93dd-1ea3c90634cd")
7475
.containsEntry("instanceId", "0");
7576
assertThat(factory).isInstanceOf(CloudFoundryApplicationFactory.class);
7677
assertThat(factory.createApplication().getServiceUrl()).isEqualTo("http://dsyerenv.cfapps.io/");

spring-boot-admin-client/src/test/java/de/codecentric/boot/admin/client/registration/metadata/CloudFoundryMetadataContributorTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,7 @@ public void should_return_empty_metadata() {
3434
@Test
3535
public void should_return_metadata() {
3636
CloudFoundryApplicationProperties cfApplicationProperties = new CloudFoundryApplicationProperties();
37-
cfApplicationProperties.setInstanceId("appId");
37+
cfApplicationProperties.setApplicationId("appId");
3838
cfApplicationProperties.setInstanceIndex("1");
3939
CloudFoundryMetadataContributor contributor = new CloudFoundryMetadataContributor(cfApplicationProperties);
4040
assertThat(contributor.getMetadata()).containsEntry("applicationId", "appId").containsEntry("instanceId", "1");

0 commit comments

Comments
 (0)