#2337 TKS secure channel protocol and JSS refinements
Closed: migrated 3 years ago by dmoluguw. Opened 7 years ago by jmagne.

Ticket #801 resulted in moving much of the jni symkey functionality into either JSS itself or Java code within the TKS subsystem.

The review for that resulted in some potential refinements, which will be listed in this ticket.

JSS: Provide a "unwrapSymmetricTemporary" variant to the new call I created called "unwrapSymmetricPerm". This way the user will know which one to use more easily. Currently, the stock "unwrapSymmetric" call assumes a temporary key.

JSS: Make sure the license text on newly created files is up to date.

JSS and TKS: Make sure the code works on luna hsm. Adjust either or both pieces if needed.

TKS: Improve the efficiency of the computeSessionKey_SCP01, call right now it kind of generates all 3 keys needed and returns only one, which is called 3 time . Just call it once and return all 3 keys at once.

TKS: Rework the methods that deal with the shared secret key. Right now the use is a bit confusing.


One more refinenemt:

TKS right now when we try to do a symettric key derivation by encryption, if this fails on the hsm, we catch the exception and try a fallback (which works). For efficiency reason, the reviewer would like to ahve this feature conrolled by a config parameter to avoid the initial failed try, (if we know it will fail on a certain hsm).

Per discussion with jmagne on 06/03/2016: 10.4

Metadata Update from @jmagne:
- Issue set to the milestone: UNTRIAGED

7 years ago

Dogtag PKI is moving from Pagure issues to GitHub issues. This means that existing or new
issues will be reported and tracked through Dogtag PKI's GitHub Issue tracker.

This issue has been cloned to GitHub and is available here:
https://github.com/dogtagpki/pki/issues/2457

If you want to receive further updates on the issue, please navigate to the
GitHub issue and click on Subscribe button.

Thank you for understanding, and we apologize for any inconvenience.

Metadata Update from @dmoluguw:
- Issue close_status updated to: migrated
- Issue status updated to: Closed (was: Open)

3 years ago

Login to comment on this ticket.

Metadata