Skip to content

Commit

Permalink
Enable SHA1 jar signing for jdk-18 api/java_util (#3496)
Browse files Browse the repository at this point in the history
Signed-off-by: Andrew Leonard <anleonar@redhat.com>
  • Loading branch information
andrew-m-leonard committed Mar 24, 2022
1 parent 46131c1 commit e0e394f
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions jck/jtrunner/JavaTestRunner.java
Original file line number Diff line number Diff line change
Expand Up @@ -329,7 +329,7 @@ public static boolean generateJTB() throws Exception {
bw.close();
}

if ( getJckVersionInt(jckVersionNo) >= 18 && tests.contains("api/java_net") ) {
if ( getJckVersionInt(jckVersionNo) >= 18 && (tests.contains("api/java_net") || tests.contains("api/java_util")) ) {
// Requires SHA1 enabling for jar signers in jdk-18+
secPropsFile = resultDir + File.separator + "security.properties";
System.out.println("Custom security properties to be stored in: " + secPropsFile);
Expand Down Expand Up @@ -1159,7 +1159,7 @@ private static boolean testsRequireDisplay (String tests) {
private static String getTestSpecificJvmOptions(String jckVersion, String tests) {
String testSpecificJvmOptions = "";

if ( tests.contains("api/javax_net") || tests.contains("api/javax_xml") || (getJckVersionInt(jckVersionNo) >= 18 && tests.contains("api/java_net")) ) {
if ( tests.contains("api/javax_net") || tests.contains("api/javax_xml") || (getJckVersionInt(jckVersionNo) >= 18 && (tests.contains("api/java_net") || tests.contains("api/java_util"))) ) {
// Needs extra security.properties
testSpecificJvmOptions += " -Djava.security.properties=" + secPropsFile;
}
Expand Down

0 comments on commit e0e394f

Please sign in to comment.