Skip to content

Conversation

@dev-koan
Copy link
Collaborator

@dev-koan dev-koan commented Nov 3, 2025

This update adds support for PBES2, PBEKeyFactory, PBEKey, and PBEParameters for PBE algorithms with Hmac and AES for OpenJCEPlus provider.

Back-ported from: #833

Signed-off-by: Dev Agarwal [email protected]

@dev-koan
Copy link
Collaborator Author

dev-koan commented Nov 3, 2025

FYI
-> Modified hashcode() to return the same value as jdk17.
-> Added PBEUtil.java file as it doesn't exist in jdk17 and back
-> PBES2Parameter.java was modified with a change in parseKDF() compared to main.
-> Slight changes in PBEKey.java as jdk11 is an older version and doesn't support some things
-> PBE algorithms with the hash function SHA512\224 or SHA512\256 were removed as they are not supported in the base Cipher.java in openjdk. In this function openjdk doesn't account for algorithms having \ in the name which can found in the newer jdk version (here)

@dev-koan dev-koan force-pushed the implement_pbe_11 branch 2 times, most recently from d61175c to 7016dd7 Compare November 3, 2025 10:50
@dev-koan dev-koan force-pushed the implement_pbe_11 branch 2 times, most recently from 37a2fc3 to b831d83 Compare November 6, 2025 14:56
Copy link
Member

@jasonkatonica jasonkatonica left a comment

Choose a reason for hiding this comment

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

LGTM

This update adds support for PBES2, PBEKeyFactory, PBEKey,
and PBEParameters for PBE algorithms with Hmac and AES for
OpenJCEPlus provider.

Signed-off-by: Dev Agarwal <[email protected]>
Copy link
Member

@KostasTsiounis KostasTsiounis left a comment

Choose a reason for hiding this comment

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

LGTM

@jasonkatonica jasonkatonica merged commit 589439b into IBM:java11 Nov 6, 2025
3 checks passed
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.

4 participants