RFR: 8344228: Revisit SecurityManager usage in java.net.http after JEP 486 integration [v3]

Jaikiran Pai jpai at openjdk.org
Fri Nov 15 13:25:53 UTC 2024


On Fri, 15 Nov 2024 10:25:18 GMT, Daniel Fuchs <dfuchs at 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


More information about the net-dev mailing list