RFR: 8344855: Remove calls to SecurityManager and doPrivileged in HTTP related implementation classes in the sun.net and sun.net.www.http packages after JEP 486 integration [v2]

Jaikiran Pai jpai at openjdk.org
Mon Nov 25 04:37:13 UTC 2024


On Fri, 22 Nov 2024 15:08:33 GMT, Daniel Fuchs <dfuchs at openjdk.org> wrote:

>> Some further cleaning in the legacy HTTP implementation. 
>> Usual removal of doPrivileged, GetPropertyAction, checkPermission, etc...
>> 
>> I also took the opportunity to also remove some constructors that were never called in the legacy HttpClient, and to fix some throws signatures.
>
> Daniel Fuchs has updated the pull request incrementally with one additional commit since the last revision:
> 
>   MAX_DATA_REMAINING should be long; review feedback

This looks good to me.

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

Marked as reviewed by jpai (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/22321#pullrequestreview-2457134776


More information about the net-dev mailing list