Adopted comment and checked in:
commit dc13bd1a2366c71880a59586ebe70f40683afa3c
thanks!
Christina
On 05/17/2016 06:25 PM, John Magne wrote:
Looks good.
If tested to work conditional ACK.
Just one thing, when throwing a TPSException at the end of the patch,
please give it the error code, TPSStatus.STATUS_ERROR_CONTACT_ADMIN
----- Original Message -----
From: "Christina Fu" <[email protected]>
To: "pki-devel" <[email protected]>
Sent: Tuesday, May 17, 2016 6:13:01 PM
Subject: [Pki-devel] [PATCH]
pki-cfu-0122-Ticket-1527-reopened-retrieved-wrong-ca-connector-co.patch
Ticket #1527 (reopened) retrieved wrong ca connector config parameter
This ticket was reopened due to retrieving wrong ca connector
config param under the situation when format is performed during enrollment.
The following is attempted:
op.enroll.userKey.ca.conn
while the following is intended:
op.format.userKey.ca.conn
In addition, this patch also fixes the following issues;
a. reason param name is not conforming: "reason" instead of
"revokeReason"
b. adding default reason to format TPS profiles
c. by default mappingResolver.formatProfileMappingResolver resolves
to tokenKey, while enroll resolves to userKey.
-> now changed the userKey
d. if revocation fails during format, it was forgiving.
-> now changed so that error is logged in activity log and exception
thrown and bail out
Tested to work.
thanks,
Christina
_______________________________________________
Pki-devel mailing list
[email protected]
https://www.redhat.com/mailman/listinfo/pki-devel
_______________________________________________
Pki-devel mailing list
[email protected]
https://www.redhat.com/mailman/listinfo/pki-devel
