RFR: 8273655: content-types.properties files are missing some common types [v2]

Julia Boes jboes at openjdk.java.net
Thu Sep 16 15:40:50 UTC 2021


On Thu, 16 Sep 2021 15:28:36 GMT, Julia Boes <jboes at openjdk.org> wrote:

>> This change adds some common types to the content-type.properties files, notably .js, .css, and .jar, as well as some others. 
>> 
>> The duplicated entry for .zip is removed from the Windows properties file.
>
> Julia Boes has updated the pull request incrementally with one additional commit since the last revision:
> 
>   address PR comments and refactor test

@anthonyvdotbe Thanks for the comments, I incorporated them with one exception: I didn't add typescript (.ts) as it conflicts with MPEG transport stream on several platforms. 

A note on the test: I refactored it slightly to account for platform-specific type detection, for example in the case of .js, which is mapped to `application/javascript` on linux-aarch64 rather than `text/javascript`. The reason for this deviation is that the test uses `Files.probeContentType`, which only falls back to the content-types.properties file if no file type detector is installed.

Testing: tier1-3 all clear.

-------------

PR: https://git.openjdk.java.net/jdk/pull/5506


More information about the nio-dev mailing list