From 0c411859eb702dbe55bd81a1778cc0f8caa57281 Mon Sep 17 00:00:00 2001
From: Norman Maurer <norman_maurer@apple.com>
Date: Wed, 14 Jul 2021 07:46:17 +0200
Subject: [PATCH] SelfSignedCertificate should work in http2 tests (#11486)

Motivation:

We need to add `--add-exports java.base/sun.security.x509=ALL-UNNAMED` when running the tests for codec-http2 as some of the tests use SelfSignedCertificate.

Modifications:

- Add `--add-exports java.base/sun.security.x509=ALL-UNNAMED` when running the tests for codec-http2
- Ensure we export correct when running with JDK12, 13, 14 and 15 as well

Result:

No more tests failure due not be able to access classes
---
 codec-http2/pom.xml | 2 ++
 pom.xml             | 9 +++++++++
 2 files changed, 11 insertions(+)

diff --git a/codec-http2/pom.xml b/codec-http2/pom.xml
index c787a53c62..e9d74896ef 100644
--- a/codec-http2/pom.xml
+++ b/codec-http2/pom.xml
@@ -30,6 +30,8 @@
 
   <properties>
     <javaModuleName>io.netty.codec.http2</javaModuleName>
+    <!-- Needed for SSL tests as these use the SelfSignedCertificate -->
+    <argLine.java9.extras>--add-exports java.base/sun.security.x509=ALL-UNNAMED</argLine.java9.extras>
   </properties>
 
   <dependencies>
diff --git a/pom.xml b/pom.xml
index 1cb438dee7..0a9b73e486 100644
--- a/pom.xml
+++ b/pom.xml
@@ -216,6 +216,9 @@
         <jdk>15</jdk>
       </activation>
       <properties>
+        <argLine.java9.extras />
+        <!-- Export some stuff which is used during our tests -->
+        <argLine.java9>--illegal-access=deny ${argLine.java9.extras}</argLine.java9>
         <!-- Not use alpn agent as Java11+ supports alpn out of the box -->
         <argLine.alpnAgent />
         <forbiddenapis.skip>true</forbiddenapis.skip>
@@ -237,6 +240,9 @@
         <jdk>14</jdk>
       </activation>
       <properties>
+        <argLine.java9.extras />
+        <!-- Export some stuff which is used during our tests -->
+        <argLine.java9>--illegal-access=deny ${argLine.java9.extras}</argLine.java9>
         <!-- Not use alpn agent as Java11+ supports alpn out of the box -->
         <argLine.alpnAgent />
         <forbiddenapis.skip>true</forbiddenapis.skip>
@@ -255,6 +261,9 @@
         <jdk>13</jdk>
       </activation>
       <properties>
+        <argLine.java9.extras />
+        <!-- Export some stuff which is used during our tests -->
+        <argLine.java9>--illegal-access=deny ${argLine.java9.extras}</argLine.java9>
         <!-- Not use alpn agent as Java11+ supports alpn out of the box -->
         <argLine.alpnAgent />
         <forbiddenapis.skip>true</forbiddenapis.skip>