Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SMAdapter.translateKeyScheme and JCESecurityModule implementation #86

Merged
merged 6 commits into from Apr 27, 2015

Conversation

demsey
Copy link
Contributor

@demsey demsey commented Apr 27, 2015

HSM enables to convert double and triple length DES key encrypted
using X9.17 methods to more secure key using the variant method.
In theory is possible do it using export a key under a ZMK and
import it again under LMK with more secure encryption scheme,
but very often exporting and importing X9.17 keys is locked.
Translate Key Scheme to variant methods is enabled by default.

Also others cleaning patches;)

In addition:
* fix several PMD EmptyCatchBlock violations
* key scheme and variant checking and throwing IllegalArgumentException
* add lacking JUnit tests
In addition:
* fix several PMD EmptyCatchBlock violations
* key scheme and Variant checking and throwing IllegalArgumentException
HSM enables to convert double and triple length DES key encrypted
using X9.17 methods to more secure key using the variant method.
In theory is possible do it using export a key under a ZMK and
import it again under LMK with more secure encryption scheme,
but very often exporting and importing X9.17 keys is locked.
Translate Key Scheme to variant methods is enabled by default.
ar added a commit that referenced this pull request Apr 27, 2015
SMAdapter.translateKeyScheme and JCESecurityModule implementation
@ar ar merged commit 97db37f into jpos:master Apr 27, 2015
@yinheli
Copy link
Contributor

yinheli commented Apr 28, 2015

good job

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.

None yet

3 participants