On Fri, 15 Nov 2024 10:25:18 GMT, Daniel Fuchs <dfu...@openjdk.org> wrote:

>> Please find here a patch that cleans up the java.net.http module code to 
>> remove permission checks and doPriviliged calls.
>> This was mostly mechanical.
>
> Daniel Fuchs 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 three additional 
> commits since the last revision:
> 
>  - Merge branch 'master' into HttpClient-SM-8344228
>  - Review feedback
>  - 8344228: Revisit SecurityManager usage in java.net.http after JEP 486 
> integration

These changes look good to me. I had a couple of trivial comments which I've 
added inline.

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

Marked as reviewed by jpai (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/22118#pullrequestreview-2438562003

Reply via email to