RFR(S): 8224727: Problem list 2 tests in security/infra/java/security/cert/CertPathValidator/certification

[Rajan Halade]

Looks good, thanks!

- Rajan

On 5/24/19 10:50 PM, Langer, Christoph wrote:
> Hi Rajan,
>
> thanks for this. Problem list update would be then:
> http://cr.openjdk.java.net/~clanger/webrevs/8224727.1/
>
> Please review.
>
> Thanks
> Christoph
>
>
>> -----Original Message-----
>> From: Rajan Halade <rajan.halade at oracle.com>
>> Sent: Freitag, 24. Mai 2019 18:52
>> To: Langer, Christoph <christoph.langer at sap.com>
>> Cc: Sean Mullan <sean.mullan at oracle.com>; security-dev <security-
>> dev at openjdk.java.net>
>> Subject: Re: RFR(S): 8224727: Problem list 2 tests in
>> security/infra/java/security/cert/CertPathValidator/certification
>>
>> I have pushed fix for ComodoCA with JDK-8202651 and have filed
>> JDK-8224768 to address ActalisCA issue. You can problem list ActalisCA
>> for now with JDK-8224727.
>>
>> Thanks,
>> Rajan
>>
>> On 5/24/19 6:43 AM, Sean Mullan wrote:
>>> On 5/24/19 4:00 AM, Langer, Christoph wrote:
>>>> Hi,
>>>>
>>>> please review the problem listing of
>>>>
>>>>
>> security/infra/java/security/cert/CertPathValidator/certification/ActalisCA.ja
>> va
>>>> and
>>>>
>>>>
>> security/infra/java/security/cert/CertPathValidator/certification/ComodoCA.
>> java
>>>>
>>>> until JDK-8202651 and JDK-8215546 are resolved.
>>>>
>>>> Bug: https://bugs.openjdk.java.net/browse/JDK-8224727
>>>>
>>>> Webrev: http://cr.openjdk.java.net/~clanger/webrevs/8224727.0/
>>> I would prefer to just problem list ActalisCA.java. Rajan has a fix
>>> for ComodoCA which he posted the other day as part of
>>> https://bugs.openjdk.java.net/browse/JDK-8202651, so if he just splits
>>> that off into a separate bug then he can push that fix. Can you
>>> coordinate with Rajan the best way to handle this?
>>>
>>> There seems to be overlap in these various bugs for the failures in
>>> Actalis, Buypass and Comodo - I think we should clean this up so that
>>> there is one unique bug per failing test.
>>>
>>> --Sean