Re: [Pki-devel] [PATCH] 547 Refactored CRMFPopClient.

Looks nice. Thinks are much simpler. Quick Comments/Questions: 1. Looks like we've added a command line param for the transport cert. I notice there is still a check to see if the number of params is 4 or more. Is this still valid, or is the transport cert param optional? 2. Down in the code where we are generating keys either ec or rsa, we still check to see if the algorithm is valid. Was this check not already done at the top of the method? 3. Inside "submitRequest" you have some System.out.println's lingering. Was the idea to get rid of all of those? 4. Does the usage string give an idea what params are optional and such? If not would this be helpful? Is it the "default" designation used for this purpose? Anyway, ACK on the functionality if its been of course tested to work. ----- Original Message -----