On Thu, 16 Sep 2021 15:28:36 GMT, Julia Boes <jb...@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

Reply via email to