X-Git-Url: https://gerrit.onap.org/r/gitweb?p=aaf%2Fauthz.git;a=blobdiff_plain;f=cadi%2Fcore%2Fsrc%2Fmain%2Fjava%2Forg%2Fonap%2Faaf%2Fcadi%2Ffilter%2FCadiFilter.java;h=31785795d7105050598f4c3a9349050464dc18ab;hp=a2f168a1d5cfab0015370a45b4023894679fba20;hb=861b557b294032f9443ee1fca6b42d7252e92481;hpb=d2f8381eeef23535e17b770022060a5ba06eedba diff --git a/cadi/core/src/main/java/org/onap/aaf/cadi/filter/CadiFilter.java b/cadi/core/src/main/java/org/onap/aaf/cadi/filter/CadiFilter.java index a2f168a1..31785795 100644 --- a/cadi/core/src/main/java/org/onap/aaf/cadi/filter/CadiFilter.java +++ b/cadi/core/src/main/java/org/onap/aaf/cadi/filter/CadiFilter.java @@ -263,6 +263,7 @@ public class CadiFilter implements Filter { float code=0f, validate=0f; String user = "n/a"; String tag = ""; + TafResp tresp = null; try { HttpServletRequest hreq = (HttpServletRequest)request; if (noAuthn(hreq)) { @@ -272,7 +273,7 @@ public class CadiFilter implements Filter { } else { HttpServletResponse hresp = (HttpServletResponse)response; startValidate=System.nanoTime(); - TafResp tresp = httpChecker.validate(hreq, hresp, hreq); + tresp = httpChecker.validate(hreq, hresp, hreq); validate = Timing.millis(startValidate); if (tresp.isAuthenticated()==RESP.IS_AUTHENTICATED) { user = tresp.getPrincipal().personalName(); @@ -288,9 +289,15 @@ public class CadiFilter implements Filter { } catch (ClassCastException e) { throw new ServletException("CadiFilter expects Servlet to be an HTTP Servlet",e); } finally { - access.printf(Level.WARN, "Trans: user=%s[%s],ip=%s,ms=%f,validate=%f,code=%f", - user,tag,request.getRemoteAddr(), - Timing.millis(startAll),validate,code); + if (tresp != null) { + access.printf(Level.INFO, "Trans: user=%s[%s],ip=%s,ms=%f,validate=%f,code=%f,result=%s", + user,tag,request.getRemoteAddr(), + Timing.millis(startAll),validate,code,tresp.isAuthenticated().toString()); + } else { + access.printf(Level.INFO, "Trans: user=%s[%s],ip=%s,ms=%f,validate=%f,code=%f,result=FAIL", + user,tag,request.getRemoteAddr(), + Timing.millis(startAll),validate,code); + } } }