On Mon, 20 Sep 2021 15:28:05 GMT, Julia Boes <jb...@openjdk.org> wrote:

>> This change implements a simple web server that can be run on the 
>> command-line with `java -m jdk.httpserver`.
>> 
>> This is facilitated by adding an entry point for the `jdk.httpserver` 
>> module, an implementation class whose main method is run when the above 
>> command is executed. This is the first such module entry point in the JDK.
>> 
>> The server is a minimal HTTP server that serves the static files of a given 
>> directory, similar to existing alternatives on other platforms and 
>> convenient for testing, development, and debugging.
>> 
>> Additionally, a small API is introduced for programmatic creation and 
>> customization.
>> 
>> Testing: tier1-3.
>
> Julia Boes has updated the pull request with a new target base due to a merge 
> or a rebase. The pull request now contains 12 commits:
> 
>  - Merge branch 'master' into simpleserver
>  - check isHidden, isSymlink, isReadable for all path segments 
>  - add checks for all path segments
>  - Merge branch 'master' into componentcheck
>  - Merge branch 'master' into simpleserver
>  - improve output on startup
>  - correct path handling
>  - small spec rewording
>  - add module main class to symbolgenerator
>  - remove UnmodifiableHeaders constant
>  - ... and 2 more: 
> https://git.openjdk.java.net/jdk/compare/4d95a5d6...10523290

src/jdk.httpserver/share/classes/sun/net/httpserver/simpleserver/FileServerHandler.java
 line 340:

> 338:                 }
> 339:             }
> 340:             return false;

This will start checking from the root of the file system. I believe we want to 
start checking from the root of the FileServerHandler, root excluded.

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

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

Reply via email to