On Wed, 10 Feb 2021 07:46:02 GMT, Michael McMahon <micha...@openjdk.org> wrote:

>> Could I get the following change reviewed please? It fixes a problem (in 
>> JEP380) on Windows where some file operations on Unix domain sockets were 
>> not working and led to the feature being disabled on Windows 2019 Server in 
>> JDK 16. So, the fix re-enables the feature on all versions of Windows that 
>> support it.
>> 
>> The test checks all the file APIs that were affected by the problem. The 
>> change touches the code that handles symbolic links in Windows (since they 
>> are implemented as NTFS reparse points, like Unix sockets), but I didn't add 
>> any specific testing in this area, as I assume the existing unit tests for 
>> NIO symbolic links should cover that. If I should add more tests here, then 
>> I can do that.
>> 
>> Thanks,
>> Michael
>
> Michael McMahon has updated the pull request with a new target base due to a 
> merge or a rebase. The incremental webrev excludes the unrelated changes 
> brought in by the merge/rebase. The pull request contains 16 additional 
> commits since the last revision:
> 
>  - Rearrange test in WindowsPath. Change to Security.java test to explicitly 
> delete socket files
>  - Merge branch 'master' into 8252971-socket-attributes
>  - update
>  - fix mistake in last push
>  - update
>  - Merge branch 'master' into 8252971-socket-attributes
>  - add specific check for unix domain socket
>  - Merge branch 'master' into 8252971-socket-attributes
>  - update
>  - update
>  - ... and 6 more: 
> https://git.openjdk.java.net/jdk/compare/f61a6ff5...70832057

Changes LGTM.

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

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

Reply via email to