No problem. I made a slight modification to your fix to replace the fixed password with a randomly generated one. Here’s the webrev: http://cr.openjdk.java.net/~vinnie/8062264/webrev.00/
If you’re OK with that then I’ll push your fix along with two other fixes to KeychainStore. BTW OCA approval is required before your patch can be integrated and I don’t see your name listed at: http://www.oracle.com/technetwork/community/oca-486395.html Did you receive a confirmation email? On 28 Oct 2014, at 09:02, Florian Bruckner (3kraft) <florian.bruck...@3kraft.com> wrote: > Hi Vincent, > > don't want to be pushy - just a gentle reminder :-) > > regards, > > Florian > > On 14.10.14 15:13, Vincent Ryan wrote: >> I’m the blocker. I’ll try to get your changset pushed by the end of this >> week. >> >> >> On 14 Oct 2014, at 13:24, Florian Bruckner (3kraft) >> <florian.bruck...@3kraft.com> wrote: >> >>> Hi Vincent, >>> >>> it seems we've stalled on this somewhere. >>> >>> What do I need to do to have the patch accepted? >>> >>> regards, >>> >>> Florian >>> >>> >>> On 21.08.14 11:28, Vincent Ryan wrote: >>>> Hello Florian, >>>> >>>> I can sponsor this changeset and attribute the fix to you. >>>> >>>> >>>> On 21 Aug 2014, at 09:52, Florian Bruckner (3kraft) >>>> <florian.bruck...@3kraft.com> wrote: >>>> >>>>> Hi Sean, >>>>> >>>>> the OCA has been approved - what are the next steps? >>>>> >>>>> With best regards, >>>>> >>>>> Florian >>>>> >>>>> On 05.08.14 21:47, Sean Mullan wrote: >>>>>> This seems like a reasonable change to me. In order to proceed with >>>>>> accepting your patch, you will first need to sign an OCA. See step 0 of >>>>>> http://openjdk.java.net/contribute/ >>>>>> >>>>>> Thanks, >>>>>> Sean >>