Re: RFR: 8301167: Update VerifySignedJar to actually exercise and test verification [v2]

2023-02-06 Thread Weijun Wang
On Mon, 6 Feb 2023 09:16:45 GMT, Eirik Bjorsnos wrote: >> test/jdk/java/util/jar/JarFile/VerifySignedJar.java line 76: >> >>> 74: // Read entry by name >>> 75: ZipEntry ze = >>> Objects.requireNonNull(jf.getEntry("getprop.class")); >>> 76: JarEntry je = >>>

Re: RFR: 8301167: Update VerifySignedJar to actually exercise and test verification [v3]

2023-02-06 Thread Weijun Wang
On Mon, 6 Feb 2023 09:21:29 GMT, Eirik Bjorsnos wrote: >> This PR resurrects VerifySignedJar which currently tests nothing. >> >> VerifySignedJar currently verifies a binary JAR which was signed with SHA-1 >> back in April 2000. Because SHA-1 signed JARs has been disabled for a while, >> the J

Re: RFR: 8301167: Update VerifySignedJar to actually exercise and test verification [v2]

2023-02-06 Thread Eirik Bjorsnos
On Mon, 6 Feb 2023 04:09:29 GMT, Weijun Wang wrote: >> Eirik Bjorsnos has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Add whitespace after "if" >> >> Co-authored-by: Andrey Turbanov > > test/jdk/java/util/jar/JarFile/VerifySignedJa

Re: RFR: 8301167: Update VerifySignedJar to actually exercise and test verification [v3]

2023-02-06 Thread Eirik Bjorsnos
> This PR resurrects VerifySignedJar which currently tests nothing. > > VerifySignedJar currently verifies a binary JAR which was signed with SHA-1 > back in April 2000. Because SHA-1 signed JARs has been disabled for a while, > the JAR is treated as unsigned so the test doesn't really test anyt

Re: RFR: 8301167: Update VerifySignedJar to actually exercise and test verification [v2]

2023-02-05 Thread Weijun Wang
On Fri, 3 Feb 2023 08:01:21 GMT, Eirik Bjorsnos wrote: >> This PR resurrects VerifySignedJar which currently tests nothing. >> >> VerifySignedJar currently verifies a binary JAR which was signed with SHA-1 >> back in April 2000. Because SHA-1 signed JARs has been disabled for a while, >> the J

Re: RFR: 8301167: Update VerifySignedJar to actually exercise and test verification [v2]

2023-02-03 Thread Eirik Bjorsnos
> This PR resurrects VerifySignedJar which currently tests nothing. > > VerifySignedJar currently verifies a binary JAR which was signed with SHA-1 > back in April 2000. Because SHA-1 signed JARs has been disabled for a while, > the JAR is treated as unsigned so the test doesn't really test anyt

Re: RFR: 8301167: Update VerifySignedJar to actually exercise and test verification

2023-02-02 Thread Andrey Turbanov
On Wed, 25 Jan 2023 17:38:13 GMT, Eirik Bjorsnos wrote: > This PR resurrects VerifySignedJar which currently tests nothing. > > VerifySignedJar currently verifies a binary JAR which was signed with SHA-1 > back in April 2000. Because SHA-1 signed JARs has been disabled for a while, > the JAR i

Re: RFR: 8301167: Update VerifySignedJar to actually exercise and test verification

2023-01-26 Thread Weijun Wang
On Wed, 25 Jan 2023 17:38:13 GMT, Eirik Bjorsnos wrote: > This PR resurrects VerifySignedJar which currently tests nothing. > > VerifySignedJar currently verifies a binary JAR which was signed with SHA-1 > back in April 2000. Because SHA-1 signed JARs has been disabled for a while, > the JAR i

RFR: 8301167: Update VerifySignedJar to actually exercise and test verification

2023-01-26 Thread Eirik Bjorsnos
This PR resurrects VerifySignedJar which currently tests nothing. VerifySignedJar currently verifies a binary JAR which was signed with SHA-1 back in April 2000. Because SHA-1 signed JARs has been disabled for a while, the JAR is treated as unsigned so the test doesn't really test anything as of