[Pki-devel] [PATCH] 59 Refactored audit service.

[Ade Lee]

As mentioned on #irc, the audit() functions in Auditor are too specific
to auditing in the User / Group servlets.  It will be difficult to
provide any set of parameters that will cover all cases, so it may be
easiest for now to do what the AdminServlet audit(String msg) function
does -- which is take in a single String argument.

The location of the class is also not the best.  If nothing else, I
would put it under com.netscape.certsrv.logging as this is where all the
other classes handling auditing functions reside.

The javadoc comments on each of the methods also need to be changed --
they talk about extended admin servlets.

Also, it might be nice to go ahead and change admin servlet right now to
use the new class.  This will involve a little more testing -- using
some of the existing servlets and confirming that the audit log messages
are being written - but it will prevent code duplication.

We will need to use this Auditor class in the other servlets we are
replacing.

Ade
 
On Mon, 2012-05-21 at 10:25 -0500, Endi Sukma Dewata wrote:
> Previously the audit service was only available to subclasses of
> AdminServlet. This functionality has been moved into a more generic 
> Auditor service which can be used by others including REST services. In 
> the future the AdminServlet should be modified to use the Auditor service.
> 
> Ticket #160
> 
> _______________________________________________
> Pki-devel mailing list
> Pki-devel at redhat.com
> https://www.redhat.com/mailman/listinfo/pki-devel