Skip to content

Commit 722efc7

Browse files
authored
Fix Issues where a period is included in the URI (Azure#29249)
1 parent 38515f2 commit 722efc7

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

sdk/appconfiguration/azure-spring-cloud-appconfiguration-config-web/src/main/java/com/azure/spring/cloud/config/web/AppConfigurationEndpoint.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -79,7 +79,7 @@ public AppConfigurationEndpoint(HttpServletRequest request, List<ConfigStore> co
7979
String topic = requestTopic.asText();
8080
store = topic.substring(
8181
topic.toLowerCase(Locale.ROOT).indexOf(CONFIG_STORE_TOPIC) + CONFIG_STORE_TOPIC.length() + 1);
82-
endpoint = String.format("https://%s.", store);
82+
endpoint = String.format("https://%s", store);
8383
} else {
8484
throw new IllegalArgumentException("Refresh request missing topic field.");
8585
}

sdk/appconfiguration/azure-spring-cloud-appconfiguration-config-web/src/test/java/com/azure/spring/cloud/config/web/AppConfigurationEndpointTest.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -67,14 +67,14 @@ public void validationParsing() throws JsonGenerationException, JsonMappingExcep
6767
Map<String, String> allRequestParams = new HashMap<String, String>();
6868

6969
AppConfigurationEndpoint endpoint = new AppConfigurationEndpoint(request, configStores, allRequestParams);
70-
assertEquals("https://testConfig.", endpoint.getEndpoint());
70+
assertEquals("https://testConfig", endpoint.getEndpoint());
7171
assertEquals("testConfig", endpoint.getStore());
7272

7373
requestBody = mapper.readValue(new File(GET_TEST_REFRESH), JsonNode.class).toString();
7474
when(lines.collect(Mockito.any())).thenReturn(requestBody);
7575

7676
endpoint = new AppConfigurationEndpoint(request, configStores, allRequestParams);
77-
assertEquals("https://testConfig.", endpoint.getEndpoint());
77+
assertEquals("https://testConfig", endpoint.getEndpoint());
7878
assertEquals("testConfig", endpoint.getStore());
7979
}
8080

0 commit comments

Comments
 (0)