Integrated: 8327738: Remove unused internal method sun.n.w.p.h.HttpURLConnection.setDefaultAuthenticator

Eirik Bjørsnøs eirbjo at openjdk.org
Tue Mar 12 17:41:18 UTC 2024


On Sun, 10 Mar 2024 20:12:10 GMT, Eirik Bjørsnøs <eirbjo at openjdk.org> wrote:

> Please review this cleanup PR which removes the internal, unused method `sun.n.w.p.h.HttpURLConnection.setDefaultAuthenticator` with associated code.
> 
> This was discovered while investigating methods annotated with `@Deprecated`, but missing `@deprecated` javadoc tags.
> 
> The method sets a private static field which is read in various places. This field and its uses is also removed. After these changes, the associated interface `HttpAuthenticator` falls out of use and is removed as well.
> 
> Testing: No tests changes in this PR, this is purely a cleanup removal of unused code. A grep for `setDefaultAuthentication` across the JDK repo after this PR returns no matches.

This pull request has now been integrated.

Changeset: b9c3dc3b
Author:    Eirik Bjørsnøs <eirbjo at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/b9c3dc3b47459890f4abdea2f3367662d3467289
Stats:     123 lines in 2 files changed: 0 ins; 122 del; 1 mod

8327738: Remove unused internal method sun.n.w.p.h.HttpURLConnection.setDefaultAuthenticator

Reviewed-by: djelinski, dfuchs

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

PR: https://git.openjdk.org/jdk/pull/18178


More information about the net-dev mailing list