Skip to content

Conversation

@moresandeep
Copy link
Contributor

What changes were proposed in this pull request?

This PR adds builds support for JDK 17 and removes support of JDK 8.

How was this patch tested?

This patch was locally tested

  1. Ran a full build with unit tests on JDK 17
  2. Installed Knox runtime on JDK 17 and tested basic functionality (SSO)

@moresandeep moresandeep requested a review from pzampino July 30, 2025 11:36
@moresandeep moresandeep force-pushed the feature/jdk17-upgrade branch from a147bc6 to a10af14 Compare July 30, 2025 13:40
@moresandeep moresandeep changed the title KNOX-3162 - Build support for JDK 17 [WIP] KNOX-3162 - Build support for JDK 17 Jul 30, 2025
moresandeep and others added 10 commits July 30, 2025 15:04
…uncycastle exception, set max connections for PoolingHttpClientConnectionManager (apache#1065)
…y getParameter in UrlEncodedFormRequest (apache#1074)

* KNOX-3179: Moved client id validation into client secret parse, Modify getParameter in UrlEncodedFormRequest

* KNOX-3179: New test for UrlEncodedFormRequest getParameter

* KNOX-3179: Use existing mismatch message
Bumps [org.bouncycastle:bcpkix-jdk18on](https://github.com/bcgit/bc-java) from 1.78 to 1.78.1.
- [Changelog](https://github.com/bcgit/bc-java/blob/main/docs/releasenotes.html)
- [Commits](https://github.com/bcgit/bc-java/commits)

---
updated-dependencies:
- dependency-name: org.bouncycastle:bcpkix-jdk18on
  dependency-version: 1.78.1
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@moresandeep moresandeep changed the title [WIP] KNOX-3162 - Build support for JDK 17 KNOX-3162 - Build support for JDK 17 Aug 22, 2025
@moresandeep moresandeep changed the title KNOX-3162 - Build support for JDK 17 [WIP] KNOX-3162 - Build support for JDK 17 Aug 22, 2025
@moresandeep
Copy link
Contributor Author

Moving to a different branch https://github.com/apache/knox/tree/feature/jdk17_upgrade

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants