From: Andre Malo <nd@apache.org>
Date: Thu, 6 Mar 2003 23:12:29 +0000 (+0000)
Subject: mod_usertrack: don't set the cookie in subrequests. This works
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=6993016e2a7aefead198dd1c750dc413f0b8ad75;p=apache

mod_usertrack: don't set the cookie in subrequests. This works
around the problem that cookies were set twice during fast internal
redirects.

PR: 13211


git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@98910 13f79535-47bb-0310-9956-ffa450edef68
---

diff --git a/CHANGES b/CHANGES
index 033d962f57..ae1ed2fb7a 100644
--- a/CHANGES
+++ b/CHANGES
@@ -2,6 +2,10 @@ Changes with Apache 2.1.0-dev
 
   [Remove entries to the current 2.0 section below, when backported]
 
+  *) mod_usertrack: don't set the cookie in subrequests. This works
+     around the problem that cookies were set twice during fast internal
+     redirects. PR 13211.  [André Malo]
+
   *) Linux 2.4+: enable coredumps when Apache is started as root
      if CoreDumpDir is configured [Greg Ames]
 
diff --git a/modules/metadata/mod_usertrack.c b/modules/metadata/mod_usertrack.c
index f9c4e1b1ca..ad144ea9e6 100644
--- a/modules/metadata/mod_usertrack.c
+++ b/modules/metadata/mod_usertrack.c
@@ -200,7 +200,8 @@ static int spot_cookie(request_rec *r)
     const char *cookie;
     const char *value;
 
-    if (!dcfg->enabled) {
+    /* Do not run in subrequests */
+    if (!dcfg->enabled || r->main) {
         return DECLINED;
     }