summaryrefslogtreecommitdiffstats
path: root/base/server/cms/src/com/netscape
diff options
context:
space:
mode:
authorAde Lee <alee@redhat.com>2017-05-23 10:01:47 -0400
committerAde Lee <alee@redhat.com>2017-05-23 15:26:16 -0400
commitf40d0aaf446b162994e9c8598a7b00a6d4c906f2 (patch)
treec88b74ad137b2cecdec48f176e0f6fc7e683c3ef /base/server/cms/src/com/netscape
parent3027b565320c96857b7f7fdffed9a5fbec084bab (diff)
downloadpki-f40d0aaf446b162994e9c8598a7b00a6d4c906f2.tar.gz
pki-f40d0aaf446b162994e9c8598a7b00a6d4c906f2.tar.xz
pki-f40d0aaf446b162994e9c8598a7b00a6d4c906f2.zip
Encapsulate recovery request approval audit logs
The audit logs where an agent grants an asynchronous recovery request and the case where recovery request is appproved from the REST API are consolidated and encapsulated in a class. Change-Id: I237c1dcfc413012d421f3ccc64e21c7caf5a7701
Diffstat (limited to 'base/server/cms/src/com/netscape')
-rw-r--r--base/server/cms/src/com/netscape/cms/servlet/key/GrantAsyncRecovery.java65
1 files changed, 11 insertions, 54 deletions
diff --git a/base/server/cms/src/com/netscape/cms/servlet/key/GrantAsyncRecovery.java b/base/server/cms/src/com/netscape/cms/servlet/key/GrantAsyncRecovery.java
index c41052554..2a5006787 100644
--- a/base/server/cms/src/com/netscape/cms/servlet/key/GrantAsyncRecovery.java
+++ b/base/server/cms/src/com/netscape/cms/servlet/key/GrantAsyncRecovery.java
@@ -34,8 +34,9 @@ import com.netscape.certsrv.base.EBaseException;
import com.netscape.certsrv.base.IArgBlock;
import com.netscape.certsrv.common.ICMSRequest;
import com.netscape.certsrv.kra.IKeyService;
-import com.netscape.certsrv.logging.AuditEvent;
import com.netscape.certsrv.logging.ILogger;
+import com.netscape.certsrv.logging.event.SecurityDataRecoveryStateChangeEvent;
+import com.netscape.certsrv.request.RequestId;
import com.netscape.cms.servlet.base.CMSServlet;
import com.netscape.cms.servlet.common.CMSRequest;
import com.netscape.cms.servlet.common.CMSTemplate;
@@ -194,32 +195,7 @@ public class GrantAsyncRecovery extends CMSServlet {
String agentID,
HttpServletRequest req, HttpServletResponse resp,
Locale locale) {
- String auditMessage = null;
String auditSubjectID = auditSubjectID();
- String auditRequestID = reqID;
- String auditAgentID = agentID;
-
- // "normalize" the "reqID"
- if (auditRequestID != null) {
- auditRequestID = auditRequestID.trim();
-
- if (auditRequestID.equals("")) {
- auditRequestID = ILogger.UNIDENTIFIED;
- }
- } else {
- auditRequestID = ILogger.UNIDENTIFIED;
- }
-
- // "normalize" the "auditAgentID"
- if (auditAgentID != null) {
- auditAgentID = auditAgentID.trim();
-
- if (auditAgentID.equals("")) {
- auditAgentID = ILogger.UNIDENTIFIED;
- }
- } else {
- auditAgentID = ILogger.UNIDENTIFIED;
- }
try {
header.addStringValue(OUT_OP,
@@ -233,40 +209,21 @@ public class GrantAsyncRecovery extends CMSServlet {
header.addStringValue("requestID", reqID);
header.addStringValue("agentID", agentID);
- // store a message in the signed audit log file
- auditMessage = CMS.getLogMessage(
- AuditEvent.KEY_RECOVERY_AGENT_LOGIN,
- auditSubjectID,
- ILogger.SUCCESS,
- auditRequestID,
- auditAgentID);
- audit(auditMessage);
-
- } catch (EBaseException e) {
- header.addStringValue(OUT_ERROR, e.toString(locale));
-
- // store a message in the signed audit log file
- auditMessage = CMS.getLogMessage(
- AuditEvent.KEY_RECOVERY_AGENT_LOGIN,
+ audit(new SecurityDataRecoveryStateChangeEvent(
auditSubjectID,
- ILogger.FAILURE,
- auditRequestID,
- auditAgentID);
+ ILogger.SUCCESS,
+ new RequestId(reqID),
+ "approve"));
- audit(auditMessage);
} catch (Exception e) {
header.addStringValue(OUT_ERROR, e.toString());
- // store a message in the signed audit log file
- auditMessage = CMS.getLogMessage(
- AuditEvent.KEY_RECOVERY_AGENT_LOGIN,
- auditSubjectID,
- ILogger.FAILURE,
- auditRequestID,
- auditAgentID);
-
- audit(auditMessage);
+ audit(new SecurityDataRecoveryStateChangeEvent(
+ auditSubjectID,
+ ILogger.FAILURE,
+ new RequestId(reqID),
+ "approve"));
}
}
}