Skip to content

Commit b2decb4

Browse files
committed
Merge branch '6.1.x'
2 parents 39c0400 + 77bdbf7 commit b2decb4

File tree

2 files changed

+9
-2
lines changed

2 files changed

+9
-2
lines changed

spring-web/src/main/java/org/springframework/web/cors/CorsConfiguration.java

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -275,14 +275,16 @@ private static void parseCommaDelimitedOrigin(String rawValue, Consumer<String>
275275
case ']' -> withinPortRange = false;
276276
case ',' -> {
277277
if (!withinPortRange) {
278-
valueConsumer.accept(rawValue.substring(start, current).trim());
278+
String originValue = rawValue.substring(start, current).trim();
279+
valueConsumer.accept(originValue);
279280
start = current + 1;
280281
}
281282
}
282283
}
283284
}
284285
if (start < rawValue.length()) {
285-
valueConsumer.accept(rawValue.substring(start));
286+
String originValue = rawValue.substring(start).trim();
287+
valueConsumer.accept(originValue);
286288
}
287289
}
288290

spring-web/src/test/java/org/springframework/web/cors/CorsConfigurationTests.java

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -305,6 +305,11 @@ void checkOriginAllowed() {
305305
assertThat(config.checkOrigin("https://a1.com")).isEqualTo("https://a1.com");
306306
assertThat(config.checkOrigin("https://a2.com/")).isEqualTo("https://a2.com/");
307307

308+
// comma-delimited origins list with space
309+
config.setAllowedOrigins(Collections.singletonList("https://a1.com, https://a2.com"));
310+
assertThat(config.checkOrigin("https://a1.com")).isEqualTo("https://a1.com");
311+
assertThat(config.checkOrigin("https://a2.com/")).isEqualTo("https://a2.com/");
312+
308313
// specific origin matches Origin header with or without trailing "/"
309314
config.setAllowedOrigins(Collections.singletonList("https://domain.com"));
310315
assertThat(config.checkOrigin("https://domain.com")).isEqualTo("https://domain.com");

0 commit comments

Comments
 (0)