First of all, I think it's a nice framework that lays the basis for supporting multiple DRM transport keys. Thanks for taking care of the encrypt/decrypt case as well, which is essential in DRM for supporting HSM's that do not support wrapping/unwrapping.

A couple observations/questions:

* in base/kra/src/com/netscape/kra/EnrollmentService.java, transportCert is specifically deleted from the requests after extraction.
We might want to consider making it optional. I understand that some customer in the past has utilized DRM requests for their own purposes. If space is a concern, one idea is to store the nickname instead. Just something to think about.

* Another thing, perhaps as a phase 2, is to think about how to get the exact transport cert that the client is using into the request to the DRM. The primary scenario that we wish to cover, I think, is the case when the transport keys are in transition. The scenario in my mind would be someone getting to the enrollment page (thus a transport key is already in the browser), then taking his/her time to fill out the form, meanwhile, the CA's transport cert changed. However, in this patch, CA is getting the transport cert from it's CS.cfg and stuffing it into the request, which means that in this scenario, CA is stuffing the new transport cert into the request instead of the old one that the client is using.
Again, I understand that it is not an easy one to resolve, but it is essential to this feature so we need to solve eventually, perhaps at the next phase. We can discuss more about this.

Christina

On 09/25/2013 04:59 PM, Andrew Wnuk wrote:

    This patch provides basic support for DRM transport key rotation described
    in http://pki.fedoraproject.org/wiki/DRM_Transport_Key_Rotation

    This patch provides implementation for tickets:
     - 729 - CA to include transport certificate when submitting archival request to DRM
     - 730 - DRM to detect presence of transport certificate attribute in submitted archival
             request and validate transport certificate against DRM's transport key list
     - 731 - DRM to provide handling for alternative transport key based on detected
             and validated transport certificate arriving as a part of extended archival request
_______________________________________________
Pki-devel mailing list
Pki-devel@redhat.com
https://www.redhat.com/mailman/listinfo/pki-devel