Re: RFR: 8225433: Clarify behavior of PKIXParameters.setRevocationEnabled when PKIXRevocationChecker is used [v2]

2022-04-27 Thread Hai-May Chao
On Wed, 27 Apr 2022 12:48:29 GMT, Sean Mullan  wrote:

>> This change improves the specification for the case when a 
>> `PKIXRevocationChecker` is supplied as one of the `CertPathChecker` 
>> parameters. Specifically, it makes it more clear that a 
>> `PKIXRevocationChecker` overrides the default revocation checking mechanism 
>> of a PKIX service provider, and will be used to check revocation 
>> irrespective of the setting of the RevocationEnabled parameter.
>> 
>> Will also file a CSR.
>
> Sean Mullan has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Change wording of "However".

Marked as reviewed by hchao (Committer).

-

PR: https://git.openjdk.java.net/jdk/pull/8287


Re: RFR: 8225433: Clarify behavior of PKIXParameters.setRevocationEnabled when PKIXRevocationChecker is used [v2]

2022-04-27 Thread Sean Mullan
> This change improves the specification for the case when a 
> `PKIXRevocationChecker` is supplied as one of the `CertPathChecker` 
> parameters. Specifically, it makes it more clear that a 
> `PKIXRevocationChecker` overrides the default revocation checking mechanism 
> of a PKIX service provider, and will be used to check revocation irrespective 
> of the setting of the RevocationEnabled parameter.
> 
> Will also file a CSR.

Sean Mullan has updated the pull request incrementally with one additional 
commit since the last revision:

  Change wording of "However".

-

Changes:
  - all: https://git.openjdk.java.net/jdk/pull/8287/files
  - new: https://git.openjdk.java.net/jdk/pull/8287/files/36a0ff13..99ee66f3

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=jdk=8287=01
 - incr: https://webrevs.openjdk.java.net/?repo=jdk=8287=00-01

  Stats: 4 lines in 1 file changed: 0 ins; 0 del; 4 mod
  Patch: https://git.openjdk.java.net/jdk/pull/8287.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/8287/head:pull/8287

PR: https://git.openjdk.java.net/jdk/pull/8287