Skip to content

Commit 5f895d7

Browse files
committed
Merge branch '6.2.x'
2 parents 5e213b2 + c0429db commit 5f895d7

File tree

2 files changed

+9
-1
lines changed

2 files changed

+9
-1
lines changed

spring-web/src/main/java/org/springframework/web/accept/MappingMediaTypeFileExtensionResolver.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -95,7 +95,7 @@ private void addFileExtension(MediaType mediaType, String extension) {
9595

9696
@Override
9797
public List<String> resolveFileExtensions(MediaType mediaType) {
98-
List<String> fileExtensions = this.fileExtensions.get(mediaType);
98+
List<String> fileExtensions = this.fileExtensions.get(mediaType.removeQualityValue());
9999
return (fileExtensions != null ? fileExtensions : Collections.emptyList());
100100
}
101101

spring-web/src/test/java/org/springframework/web/accept/MappingMediaTypeFileExtensionResolverTests.java

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -53,6 +53,14 @@ void resolveExtensionsNoMatch() {
5353
.resolveFileExtensions(MediaType.TEXT_HTML)).isEmpty();
5454
}
5555

56+
@Test
57+
void resolveExtensionsWithQualityParameter() {
58+
List<String> extensions = new MappingMediaTypeFileExtensionResolver(DEFAULT_MAPPINGS)
59+
.resolveFileExtensions(MediaType.parseMediaType("application/json;q=0.9"));
60+
61+
assertThat(extensions).containsExactly("json");
62+
}
63+
5664
@Test // SPR-13747
5765
public void lookupMediaTypeCaseInsensitive() {
5866
assertThat(new MappingMediaTypeFileExtensionResolver(DEFAULT_MAPPINGS).lookupMediaType("JSON"))

0 commit comments

Comments
 (0)