Skip to content

KNOX-3162 - Build support for JDK 17#1114

Merged
smolnar82 merged 4 commits intoapache:masterfrom
smolnar82:KNOX-3162
Nov 26, 2025
Merged

KNOX-3162 - Build support for JDK 17#1114
smolnar82 merged 4 commits intoapache:masterfrom
smolnar82:KNOX-3162

Conversation

@smolnar82
Copy link
Contributor

KNOX-3162 - Build support for JDK 17

What changes were proposed in this pull request?

This is the "clone" of #1108 with the fix for the failing tests from @bonampak .

How was this patch tested?

@moresandeep indicated in #1108 that he tested these changes locally.

moresandeep and others added 4 commits November 26, 2025 16:48
…vault library needs to be upgraded to 3.2.0 that is compatible with Spring 6.2). Removed spring-vault-dependencies as it is unavailable since 3.0+. Added httpclient and httpcore libraries to managed dependencies as these were taken from spring-vault-dependencies now. Removed duplicated dependencies from gateway pom.xml
Copy link
Contributor

@pzampino pzampino left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@smolnar82 smolnar82 merged commit a0073c9 into apache:master Nov 26, 2025
1 check passed
moresandeep added a commit to moresandeep/knox that referenced this pull request Dec 1, 2025
* Update Knox to JDK 17

* Fix issues with SecureKnoxShellTest

* KNOX-3216: Upgrade Spring to 6.x (apache#1109)

* KNOX-3217: fix TestHashicorpVaultAliasService test failures. (spring vault library needs to be upgraded to 3.2.0 that is compatible with Spring 6.2). Removed spring-vault-dependencies as it is unavailable since 3.0+. Added httpclient and httpcore libraries to managed dependencies as these were taken from spring-vault-dependencies now. Removed duplicated dependencies from gateway pom.xml

---------

Co-authored-by: Sandeep Moré <moresandeep@gmail.com>
Co-authored-by: Phil Zampino <pzampino@apache.org>
Co-authored-by: bonampak <14160522+bonampak@users.noreply.github.com>
@smolnar82 smolnar82 deleted the KNOX-3162 branch January 7, 2026 11:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants