Skip to content

Commit

Permalink
Closes #854.
Browse files Browse the repository at this point in the history
- Replaced ByteArrayInputStream reflection with new BytesInputStream implementation. The reflection is what required --add-opens java.base/java.io=jjwt.api on JDK 17+.
- Refactored KeysBridge to perform our own key length logic instead of delegating to sun.security.util.KeyUtil.  The reflection is what required --add-opens java.base/sun.security.util=jjwt.api on JDK 17+
- Removed AddOpens.java due to above refactoring (no longer needed).
- Returned a test-only --add-opens for sun.security.util for 3 test cases (added to test.addOpens maven property)
  • Loading branch information
lhazlewood committed Oct 6, 2023
1 parent d886054 commit 3a30ea1
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -133,7 +133,10 @@
<test.addOpens>
--add-opens java.base/java.lang=ALL-UNNAMED, <!-- Needed by EasyMock/cglib -->
--add-opens java.desktop/java.beans=ALL-UNNAMED, <!-- Needed by EasyMock/cglib -->
--add-opens java.base/java.lang.ref=ALL-UNNAMED <!-- Needed by PowerMock -->
--add-opens java.base/java.lang.ref=ALL-UNNAMED, <!-- Needed by PowerMock -->
<!-- needed by KeysImplTest.testKeyPairFor, KeysTest.testDeprecatedKeyPairFor, and
KeysTest.testKeyPairBuilder: -->
--add-opens java.base/sun.security.util=ALL-UNNAMED
</test.addOpens>

</properties>
Expand Down

0 comments on commit 3a30ea1

Please sign in to comment.