8002344 code review request: (was Re: [PATCH] for bug 2376501: Krb5LoginModule config class does not return proper KDC list from DNS)

[Severin Gehwolf]

On Tue, 2012-11-13 at 12:15 +0100, Severin Gehwolf wrote:
> On Tue, 2012-11-13 at 19:00 +0800, Weijun Wang wrote:
> > Where did you create this bug? Every new OpenJDK bug should starts with 
> > 7 or 8. I've been wondering what 2376501 means and thought it's 
> > something RedHat internal.
> 
> :) All I know is that it's nothing internal to Red Hat and I've created
> it via bugs.sun.com and clicking on "submit new bug"[1]. The bug ID came
> from the email I've received after it was submitted. Anywho, just wanted
> to let you know that there's another bug open *somewhere* within Oracle
> for the same issue and I can't access it.
> 
> Cheers,
> Severin
> 
> [1] Forwarded this bug email to you privately.
> 
> > On 11/13/2012 06:43 PM, Severin Gehwolf wrote:
> > > Hi Max,
> > >
> > > On Fri, 2012-11-09 at 08:38 +0800, Weijun Wang wrote:
> > >> Hi Severin
> > >>
> > >> I've created an OpenJDK bug and created a new webrev:
> > >>
> > >>      http://cr.openjdk.java.net/~weijun/8002344/webrev.00/
> > >>
> > >> The Config.java change is identical to yours, and I added a small tweak
> > >> in KrbServiceLocator, and a quite ugly regression test.
> > >
> > > Cool. Thanks! FWIW, I've created a bug with ID 2376501 for this (even
> > > before I submitted a patch). Unfortunately, this bug is not accessible
> > > to me. All it let me do was creating it :)

A fix for this has been pushed upstream:
http://hg.openjdk.java.net/jdk8/tl/jdk/rev/f740a9ac6eb6

Cheers,
Severin