Skip to content
This repository has been archived by the owner on Jan 23, 2025. It is now read-only.

remove requestTracker logic #7

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 0 additions & 1 deletion src/main/com/topcoder/dde/servlet/TCSServlet.java
Original file line number Diff line number Diff line change
Expand Up @@ -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("[* ");
Expand Down
3 changes: 0 additions & 3 deletions src/main/com/topcoder/web/common/BaseServlet.java
Original file line number Diff line number Diff line change
Expand Up @@ -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("[* ");
Expand Down
2 changes: 0 additions & 2 deletions src/main/com/topcoder/web/corp/controller/GraphServlet.java
Original file line number Diff line number Diff line change
Expand Up @@ -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;
Expand Down Expand Up @@ -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() + " ***]");

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -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);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -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("[* ");
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -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)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -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;
Expand Down Expand Up @@ -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("[* ");
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -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;
Expand Down Expand Up @@ -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()) {
Expand Down Expand Up @@ -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()) {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -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;
Expand Down Expand Up @@ -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);
Expand Down