From 4326ca420842f975a7178b87fc7be21e540b1ca9 Mon Sep 17 00:00:00 2001 From: Guiqiang Zhang Date: Wed, 3 May 2017 22:13:45 +0800 Subject: [PATCH] remove requestTracker logic --- src/main/com/topcoder/dde/servlet/TCSServlet.java | 1 - src/main/com/topcoder/web/common/BaseServlet.java | 3 --- .../com/topcoder/web/corp/controller/GraphServlet.java | 2 -- .../com/topcoder/web/forums/controller/ForumsServlet.java | 3 --- .../topcoder/web/privatelabel/controller/MainServlet.java | 3 --- .../com/topcoder/web/tc/controller/legacy/MainServlet.java | 4 ---- .../legacy/pacts/servlet/PactsInternalServlet.java | 4 ---- .../legacy/pacts/servlet/PactsMemberServlet.java | 7 ------- .../legacy/stat/servlet/StatisticsHttpServlet.java | 4 ---- 9 files changed, 31 deletions(-) diff --git a/src/main/com/topcoder/dde/servlet/TCSServlet.java b/src/main/com/topcoder/dde/servlet/TCSServlet.java index 206c02cfffc..288bbbbdefa 100644 --- a/src/main/com/topcoder/dde/servlet/TCSServlet.java +++ b/src/main/com/topcoder/dde/servlet/TCSServlet.java @@ -55,7 +55,6 @@ protected void process(HttpServletRequest request, HttpServletResponse response) info = createSessionInfo(tcRequest, authentication, user.getPrincipals()); tcRequest.setAttribute(SESSION_INFO_KEY, info); - //RequestTracker.trackRequest(authentication.getActiveUser(), tcRequest); StringBuffer loginfo = new StringBuffer(100); loginfo.append("[* "); diff --git a/src/main/com/topcoder/web/common/BaseServlet.java b/src/main/com/topcoder/web/common/BaseServlet.java index 064291e7173..8ef391c5107 100755 --- a/src/main/com/topcoder/web/common/BaseServlet.java +++ b/src/main/com/topcoder/web/common/BaseServlet.java @@ -232,9 +232,6 @@ protected void process(HttpServletRequest request, HttpServletResponse response) tcResponse = HttpObjectFactory.createUnCachedResponse(response); } tcRequest.setAttribute(SESSION_INFO_KEY, info); - //todo perhaps this should be configurable...so implementing classes - //todo don't have to do it if they don't want to - RequestTracker.trackRequest(authentication.getActiveUser(), tcRequest); StringBuffer loginfo = new StringBuffer(100); loginfo.append("[* "); diff --git a/src/main/com/topcoder/web/corp/controller/GraphServlet.java b/src/main/com/topcoder/web/corp/controller/GraphServlet.java index 9b15adaeb40..c7e8ec36626 100755 --- a/src/main/com/topcoder/web/corp/controller/GraphServlet.java +++ b/src/main/com/topcoder/web/corp/controller/GraphServlet.java @@ -11,7 +11,6 @@ import com.topcoder.web.common.CachedDataAccess; import com.topcoder.web.common.HttpObjectFactory; import com.topcoder.web.common.NavigationException; -import com.topcoder.web.common.RequestTracker; import com.topcoder.web.common.TCRequest; import com.topcoder.web.common.TCResponse; import com.topcoder.web.common.cache.CacheClient; @@ -98,7 +97,6 @@ public void process(HttpServletRequest request, HttpServletResponse response) { TCResponse tcResponse = HttpObjectFactory.createUnCachedResponse(response); WebAuthentication authentication = new BasicAuthentication( new SessionPersistor(tcRequest.getSession()), tcRequest, tcResponse, BasicAuthentication.MAIN_SITE); - RequestTracker.trackRequest(authentication.getActiveUser(), tcRequest); log.info("[*** graph *** " + dataRequest.getContentHandle() + " *** " + authentication.getActiveUser().getUserName() + " ***]"); diff --git a/src/main/com/topcoder/web/forums/controller/ForumsServlet.java b/src/main/com/topcoder/web/forums/controller/ForumsServlet.java index b831b9465c3..d3c986ed05e 100644 --- a/src/main/com/topcoder/web/forums/controller/ForumsServlet.java +++ b/src/main/com/topcoder/web/forums/controller/ForumsServlet.java @@ -165,9 +165,6 @@ protected void process(HttpServletRequest request, HttpServletResponse response) log.debug("using an uncached response"); tcResponse = HttpObjectFactory.createUnCachedResponse(response); } - //todo perhaps this should be configurable...so implementing classes - //todo don't have to do it if they don't want to - RequestTracker.trackRequest(authentication.getActiveUser(), tcRequest); com.jivesoftware.base.User forumUser = null; ForumFactory forumFactory = ForumFactory.getInstance(authToken); diff --git a/src/main/com/topcoder/web/privatelabel/controller/MainServlet.java b/src/main/com/topcoder/web/privatelabel/controller/MainServlet.java index 91962b3b0dc..a688e483abb 100755 --- a/src/main/com/topcoder/web/privatelabel/controller/MainServlet.java +++ b/src/main/com/topcoder/web/privatelabel/controller/MainServlet.java @@ -68,9 +68,6 @@ protected void process(HttpServletRequest request, HttpServletResponse response) TCSubject user = getUser(authentication.getActiveUser().getId(), tcRequest); info = createSessionInfo(tcRequest, authentication, user.getPrincipals()); tcRequest.setAttribute(SESSION_INFO_KEY, info); - //todo perhaps this should be configurable...so implementing classes - //todo don't have to do it if they don't want to - RequestTracker.trackRequest(authentication.getActiveUser(), tcRequest); StringBuffer loginfo = new StringBuffer(100); loginfo.append("[* "); diff --git a/src/main/com/topcoder/web/tc/controller/legacy/MainServlet.java b/src/main/com/topcoder/web/tc/controller/legacy/MainServlet.java index 33aa9e54839..967b251e599 100755 --- a/src/main/com/topcoder/web/tc/controller/legacy/MainServlet.java +++ b/src/main/com/topcoder/web/tc/controller/legacy/MainServlet.java @@ -120,10 +120,6 @@ private void processCommands(HttpServletRequest request, HttpServletResponse res requestTask = request.getParameter("t"); requestCommand = request.getParameter("c"); - if (!"sponsor_image".equals(requestCommand)) { - RequestTracker.trackRequest(authentication.getActiveUser(), tcRequest); - } - if (requestTask == null) requestTask = Conversion.checkNull((String) request.getAttribute("t")); if (requestCommand == null) diff --git a/src/main/com/topcoder/web/tc/controller/legacy/pacts/servlet/PactsInternalServlet.java b/src/main/com/topcoder/web/tc/controller/legacy/pacts/servlet/PactsInternalServlet.java index 2218c3f97d5..1597ad4ce5b 100755 --- a/src/main/com/topcoder/web/tc/controller/legacy/pacts/servlet/PactsInternalServlet.java +++ b/src/main/com/topcoder/web/tc/controller/legacy/pacts/servlet/PactsInternalServlet.java @@ -21,7 +21,6 @@ import com.topcoder.web.common.HttpObjectFactory; import com.topcoder.web.common.NavigationException; import com.topcoder.web.common.PermissionException; -import com.topcoder.web.common.RequestTracker; import com.topcoder.web.common.SessionInfo; import com.topcoder.web.common.StringUtils; import com.topcoder.web.common.TCRequest; @@ -109,9 +108,6 @@ public void trace(HttpServletRequest request, HttpServletResponse response) thro TCSubject user = getUser(authentication.getActiveUser().getId()); SessionInfo info = createSessionInfo(tcRequest, authentication, user.getPrincipals()); tcRequest.setAttribute(SESSION_INFO_KEY, info); - //todo perhaps this should be configuraable...so implementing classes - //todo don't have to do it if they don't want to - RequestTracker.trackRequest(authentication.getActiveUser(), tcRequest); StringBuffer loginfo = new StringBuffer(100); loginfo.append("[* "); diff --git a/src/main/com/topcoder/web/tc/controller/legacy/pacts/servlet/PactsMemberServlet.java b/src/main/com/topcoder/web/tc/controller/legacy/pacts/servlet/PactsMemberServlet.java index c92e0e227b3..302e8204832 100755 --- a/src/main/com/topcoder/web/tc/controller/legacy/pacts/servlet/PactsMemberServlet.java +++ b/src/main/com/topcoder/web/tc/controller/legacy/pacts/servlet/PactsMemberServlet.java @@ -21,7 +21,6 @@ import com.topcoder.web.common.BaseServlet; import com.topcoder.web.common.HttpObjectFactory; import com.topcoder.web.common.NavigationException; -import com.topcoder.web.common.RequestTracker; import com.topcoder.web.common.SessionInfo; import com.topcoder.web.common.StringUtils; import com.topcoder.web.common.TCRequest; @@ -135,9 +134,6 @@ protected void doGet(HttpServletRequest request, HttpServletResponse response) TCSubject user = getUser(authentication.getActiveUser().getId()); SessionInfo info = createSessionInfo(tcRequest, authentication, user.getPrincipals()); tcRequest.setAttribute(SESSION_INFO_KEY, info); - //todo perhaps this should be configuraable...so implementing classes - //todo don't have to do it if they don't want to - RequestTracker.trackRequest(authentication.getActiveUser(), tcRequest); // check to see if the user has not logged in if (info.isAnonymous()) { @@ -267,9 +263,6 @@ protected void doPost(HttpServletRequest request, HttpServletResponse response) TCSubject user = getUser(authentication.getActiveUser().getId()); SessionInfo info = createSessionInfo(tcRequest, authentication, user.getPrincipals()); tcRequest.setAttribute(SESSION_INFO_KEY, info); - //todo perhaps this should be configuraable...so implementing classes - //todo don't have to do it if they don't want to - RequestTracker.trackRequest(authentication.getActiveUser(), tcRequest); // check to see if the user has not logged in if (info.isAnonymous()) { diff --git a/src/main/com/topcoder/web/tc/controller/legacy/stat/servlet/StatisticsHttpServlet.java b/src/main/com/topcoder/web/tc/controller/legacy/stat/servlet/StatisticsHttpServlet.java index 18eed5c4596..1a5ec576d86 100755 --- a/src/main/com/topcoder/web/tc/controller/legacy/stat/servlet/StatisticsHttpServlet.java +++ b/src/main/com/topcoder/web/tc/controller/legacy/stat/servlet/StatisticsHttpServlet.java @@ -9,7 +9,6 @@ import com.topcoder.web.common.BaseServlet; import com.topcoder.web.common.CachedDataAccess; import com.topcoder.web.common.HttpObjectFactory; -import com.topcoder.web.common.RequestTracker; import com.topcoder.web.common.SessionInfo; import com.topcoder.web.common.TCRequest; import com.topcoder.web.common.TCResponse; @@ -129,9 +128,6 @@ protected void process(HttpServletRequest request, HttpServletResponse response) TCSubject user = getUser(authentication.getActiveUser().getId()); info = createSessionInfo(tcRequest, authentication, user.getPrincipals()); tcRequest.setAttribute(SESSION_INFO_KEY, info); - //todo perhaps this should be configurable...so implementing classes - //todo don't have to do it if they don't want to - RequestTracker.trackRequest(authentication.getActiveUser(), tcRequest); StringBuffer loginfo = new StringBuffer(100);