received verbal ack from jmagne.  Minor changes made to address some comments.
Pushed to master:
commit 91c77390474d67cfd0c15b8b3377997b3f0cd38a

Christina

On 01/30/2015 10:48 AM, Christina Fu wrote:
attached please find the Phase 1 code for
https://fedorahosted.org/pki/ticket/1028
 Ticket#1028 TPS rewrite: provide externalReg functionality

Due to a recent change of task priorities, I am to check in this feature in two phases (coming back to phase 2 at a later time).
In this Phase 1, the code for this feature is in place, but may not be in the most optimal condition.
The goal for Phase 1 is to:

1. not break existing functionality (non ExternalReg -- which is default)
2. when ExternalReg is turned on, it works for the basic case (didn't
test for more complicated cases)... e.g. I don't know if say an existing
cert on the token is missing from the externalReg user record, whether
it will be removed or not.
3. it is not guaranteed to work with more complicated scenario
4. code has been run through and cleaned up in Eclipse

Later, for Phase 2:

1. really clean up and add bells and whistles (more error checkings, more efficient code,
proper logging, etc.)
2. test/cover all cases

Jack has already helped testing it with a real token with ExternalReg turned off (didn't seem to break the existing functionalities);
Jack has also helped testing it with a real token with ExternalReg turned on for simple enrollment/recovery

and Jack is going to review this Phase 1 code.  Thank you Jack!

Christina




_______________________________________________
Pki-devel mailing list
Pki-devel@redhat.com
https://www.redhat.com/mailman/listinfo/pki-devel