Re: RFR: 8250968: Symlinks attributes not preserved when using jarsigner on zip files

2020-08-26 Thread Weijun Wang
Are you going to update the warning text or create a new one? Thanks, Max > On Aug 26, 2020, at 2:26 PM, Seán Coffey wrote: > > This is a follow on from the recent 8218021 fix. The jarsigner tool removes > symlink attribute data from zipfiles when signing them. jarsigner should > preserve

RFR: 8250968: Symlinks attributes not preserved when using jarsigner on zip files

2020-08-26 Thread Seán Coffey
This is a follow on from the recent 8218021 fix. The jarsigner tool removes symlink attribute data from zipfiles when signing them. jarsigner should preserve this data. The fix involves preserving the 16 bits associated with the file attributes (instead of the current 12). That's done in

Re: RFR: 8238157: Remove intermittent key from AmazonCA.java

2020-08-26 Thread Hai-May Chao
Looks good. Thanks, Hai-May > On Aug 26, 2020, at 10:13 AM, Rajan Halade wrote: > > Please review this update to remove key intermittent from AmazonCA test. This > test no longer fails intermittently. > > @@ -24,7 +24,6 @@ > /* > * @test > * @bug 8233223 > - * @key intermittent > *

Re: RFR: 8238157: Remove intermittent key from AmazonCA.java

2020-08-26 Thread Xuelei Fan
It's good. Thanks! Xuelei On 8/26/2020 10:13 AM, Rajan Halade wrote: Please review this update to remove key intermittent from AmazonCA test. This test no longer fails intermittently. @@ -24,7 +24,6 @@  /*   * @test   * @bug 8233223 *- * @key intermittent*   * @summary Interoperability

RFR: 8238157: Remove intermittent key from AmazonCA.java

2020-08-26 Thread Rajan Halade
Please review this update to remove key intermittent from AmazonCA test. This test no longer fails intermittently. @@ -24,7 +24,6 @@ /* * @test * @bug 8233223 - * @key intermittent * @summary Interoperability tests with Amazon's CA1, CA2, CA3, and CA4 * @build ValidatePathWithParams *

Re: RFR (16): 8241003: Deprecate "denigrated" java.security.cert APIs that represent DNs as Principal or String objects

2020-08-26 Thread Sean Mullan
On 8/21/20 2:14 PM, Valerie Peng wrote: Is it intentional to use toString() in src/jdk.jartool/share/classes/sun/security/tools/jarsigner/Main.java? Normally, toString() is kind of loose, not as strict as getName(). In this case I think toString() is ok, since jarsigner is printing

Re: RFR (16): 8241003: Deprecate "denigrated" java.security.cert APIs that represent DNs as Principal or String objects

2020-08-26 Thread Sean Mullan
On 8/21/20 2:01 PM, Xuelei Fan wrote: Looks fine to me. Just a trivial format comment.  Some use link for "RFC 2253", some do not. It's OK.  And it's good as well if you want to use a uniform style. Good point. Several methods in X509CertSelector and X509CRLSelector reference RFC 2253