On Mon, 11 May 2026 19:02:55 GMT, Sean Mullan <[email protected]> wrote:

>> Mikhail Yankelevich has updated the pull request incrementally with one 
>> additional commit since the last revision:
>> 
>>   comments
>
> src/java.base/share/classes/java/security/cert/X509Certificate.java line 128:
> 
>> 126:     /**
>> 127:      * Checks that the certificate is currently valid. It is if
>> 128:      * the current time are within the validity period given in the
> 
> s/are/is/

n/a since rolled back to the previous wording

> src/java.base/share/classes/java/security/cert/X509Certificate.java line 131:
> 
>> 129:      * certificate.
>> 130:      * <p>
>> 131:      * The validity period consists of two time values:
> 
> I prefer if we use the term "date" to be consistent with RFC 5280, but we can 
> reword this: "The validity period consists of two dates, defined as two time 
> values:"

Changed

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

PR Review Comment: https://git.openjdk.org/jdk/pull/30047#discussion_r3275688010
PR Review Comment: https://git.openjdk.org/jdk/pull/30047#discussion_r3275685318

Reply via email to