RFR 8038079: Re-examine integration of SPNEGO authentication

classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|

RFR 8038079: Re-examine integration of SPNEGO authentication

Pavel Rappo
Hello,

Could you please review the following change for [1]?

    http://cr.openjdk.java.net/~prappo/8038079/webrev.00/

sun.net.www.protocol.http.HttpURLConnection defined in the java.base module uses
different authentication schemes. Among others there's a scheme called SPNEGO,
which is defined in a different module, java.security.jgss

Thanks,
-Pavel

--------------------------------------------------------------------------------
[1] https://bugs.openjdk.java.net/browse/JDK-8038079

Reply | Threaded
Open this post in threaded view
|

Re: RFR 8038079: Re-examine integration of SPNEGO authentication

Chris Hegarty
On 13 Dec 2016, at 12:28, Pavel Rappo <[hidden email]> wrote:
>
> Hello,
>
> Could you please review the following change for [1]?
>
>    http://cr.openjdk.java.net/~prappo/8038079/webrev.00/

Looks good. Maybe just a short comment in the module-info.class?

   // to support SPNEGO from HttpURLConnection

-Chris.
Reply | Threaded
Open this post in threaded view
|

Re: RFR 8038079: Re-examine integration of SPNEGO authentication

Daniel Fuchs
In reply to this post by Pavel Rappo
Hi Pavel,

This looks good to me.

best regards,

-- daniel

On 13/12/16 12:28, Pavel Rappo wrote:

> Hello,
>
> Could you please review the following change for [1]?
>
>     http://cr.openjdk.java.net/~prappo/8038079/webrev.00/
>
> sun.net.www.protocol.http.HttpURLConnection defined in the java.base module uses
> different authentication schemes. Among others there's a scheme called SPNEGO,
> which is defined in a different module, java.security.jgss
>
> Thanks,
> -Pavel
>
> --------------------------------------------------------------------------------
> [1] https://bugs.openjdk.java.net/browse/JDK-8038079
>