don't let mod_usertrack overwrite other cookies

PR: 26002
Submitted by: Scott Moore <apache nopdesign.com>
Reviewed by: Brad Nicholes, Jeff Trawick

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

diff --git a/CHANGES b/CHANGES
index 7091fc192c9201b8971af9fc07542de337aaa056..fc326b9870163667689488cd2fe7ade0cc054865 100644 (file)
--- a/CHANGES
+++ b/CHANGES
@@ -1,5 +1,8 @@
 Changes with Apache 2.0.49
 
+  *) mod_usertrack no longer overwrites other cookies.
+     PR 26002.  [Scott Moore <apache nopdesign.com>]
+
   *) worker MPM: fix stack overlay bug that could cause the parent
      process to crash.  [Jeff Trawick]
 

diff --git a/STATUS b/STATUS
index 2f6b56ac3ca1c5e0e3341cb589338e5607a40255..e45aacab4000da1d08b57f4afd0967af94da498f 100644 (file)
--- a/STATUS
+++ b/STATUS
@@ -1,5 +1,5 @@
 APACHE 2.0 STATUS:                                              -*-text-*-
-Last modified at [$Date: 2004/01/29 21:15:39 $]
+Last modified at [$Date: 2004/01/29 21:30:48 $]
 
 Release:
 
@@ -305,11 +305,6 @@ PATCHES TO BACKPORT FROM 2.1
         modules/experimental/cache_cache.c r1.5
       +1: jwoolley, bnicholes, rederpj
 
-    * Don't let mod_usertrack overwrite other cookies. (2.0 + 1.3)
-      PR 26002.
-        modules/metadata/mod_usertrack.c: r1.44
-      +1: nd, bnicholes, trawick
-
     * mod_usertrack: Use always the Cookie header to determine
       the cookie name (2.0 + 1.3). PR 11475.
         modules/metadata/mod_usertrack.c: r1.45

diff --git a/modules/metadata/mod_usertrack.c b/modules/metadata/mod_usertrack.c
index f69712f616e9737ee7ee686ff079d01b63053a8b..4672639b13a0b3329ad3ea9f32957df0965d864c 100644 (file)
--- a/modules/metadata/mod_usertrack.c
+++ b/modules/metadata/mod_usertrack.c
@@ -188,7 +188,7 @@ static void make_cookie(request_rec *r)
                                  NULL);
     }
 
-    apr_table_setn(r->headers_out,
+    apr_table_addn(r->headers_out,
                    (dcfg->style == CT_COOKIE2 ? "Set-Cookie2" : "Set-Cookie"),
                    new_cookie);
     apr_table_setn(r->notes, "cookie", apr_pstrdup(r->pool, cookiebuf));   /* log first time */