You are viewing a plain text version of this content. The canonical link for it is here.
Posted to cvs@httpd.apache.org by jo...@apache.org on 2019/11/14 08:39:47 UTC

svn commit: r1869785 - /httpd/httpd/trunk/modules/session/mod_session_cookie.c

Author: jorton
Date: Thu Nov 14 08:39:46 2019
New Revision: 1869785

URL: http://svn.apache.org/viewvc?rev=1869785&view=rev
Log:
Also avoid adding the Set-Cookie header in both r->headers_out and
r->err_headers_out in ap_cookie_remove and ap_cookie_remove2 functions
to avoid duplication in HTTP response.  (Follow-up to r1843244)

Closes #73
PR: 60910
Submitted by: Lubos Uhliarik <luhliari redhat.com>

Modified:
    httpd/httpd/trunk/modules/session/mod_session_cookie.c

Modified: httpd/httpd/trunk/modules/session/mod_session_cookie.c
URL: http://svn.apache.org/viewvc/httpd/httpd/trunk/modules/session/mod_session_cookie.c?rev=1869785&r1=1869784&r2=1869785&view=diff
==============================================================================
--- httpd/httpd/trunk/modules/session/mod_session_cookie.c (original)
+++ httpd/httpd/trunk/modules/session/mod_session_cookie.c Thu Nov 14 08:39:46 2019
@@ -68,7 +68,7 @@ static apr_status_t session_cookie_save(
                             NULL);
         }
         else {
-            ap_cookie_remove(r, conf->name, conf->name_attrs, r->headers_out,
+            ap_cookie_remove(r, conf->name, conf->name_attrs,
                              r->err_headers_out, NULL);
         }
     }
@@ -82,7 +82,7 @@ static apr_status_t session_cookie_save(
         }
         else {
             ap_cookie_remove2(r, conf->name2, conf->name2_attrs,
-                              r->headers_out, r->err_headers_out, NULL);
+                              r->err_headers_out, NULL);
         }
     }