Browse Source

Merge pull request #12415 from wangkx/bad_unknown

HPCC-21870 Remove 'unknown' user from ESP log

Reviewed-By: Russ Whitehead <william.whitehead@lexisnexis.com>
Reviewed-By: Richard Chapman <rchapman@hpccsystems.com>
Richard Chapman 6 years ago
parent
commit
aae6b39a44
1 changed files with 4 additions and 1 deletions
  1. 4 1
      esp/bindings/http/platform/httpservice.cpp

+ 4 - 1
esp/bindings/http/platform/httpservice.cpp

@@ -237,7 +237,10 @@ int CEspHttpServer::processRequest()
 
         StringBuffer peerStr, pathStr;
         const char *userid=ctx->queryUserId();
-        ESPLOG(LogMin, "%s %s, from %s@%s", method.str(), m_request->getPath(pathStr).str(), (userid) ? userid : "unknown", m_request->getPeer(peerStr).str());
+        if (isEmptyString(userid))
+            ESPLOG(LogMin, "%s %s, from %s", method.str(), m_request->getPath(pathStr).str(), m_request->getPeer(peerStr).str());
+        else //user ID is in HTTP header
+            ESPLOG(LogMin, "%s %s, from %s@%s", method.str(), m_request->getPath(pathStr).str(), userid, m_request->getPeer(peerStr).str());
 
         authState = checkUserAuth();
         if ((authState == authTaskDone) || (authState == authFailed))