testbot/web: Use the default Domain and Path of our cookies.

Francois Gouget fgouget at codeweavers.com
Mon Jun 9 05:43:28 CDT 2014


The Path defaults to '/' which is what we want.
The Domain is set to the most restrictive value which is good.
This also means we're no longer putting the port number in the Domain value which fixes our cookies when the web server is running on a non-standard port or the connection is going through port forwarding.
---
 testbot/lib/WineTestBot/CGI/PageBase.pm | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/testbot/lib/WineTestBot/CGI/PageBase.pm b/testbot/lib/WineTestBot/CGI/PageBase.pm
index bfa44c4..a724507 100644
--- a/testbot/lib/WineTestBot/CGI/PageBase.pm
+++ b/testbot/lib/WineTestBot/CGI/PageBase.pm
@@ -162,8 +162,6 @@ sub SetCookies($)
       $Cookie = CGI::Cookie->new(-Name    => "SessionId",
                                  -Value   => $Session->Id,
                                  -Expires => $Expire,
-                                 -Domain  => $ENV{"HTTP_HOST"},
-                                 -Path    => "/",
                                  -Secure  => $UseSSL);
       $Request->err_headers_out->add("Set-Cookie", $Cookie);
     }
@@ -184,8 +182,6 @@ sub SetCookies($)
     $Cookie = CGI::Cookie->new(-Name    => "SessionActive",
                                -Value   => $SessionPermanent,
                                -Expires => $Expire,
-                               -Domain  => $ENV{"HTTP_HOST"},
-                               -Path    => "/",
                                -Secure  => !1);
     $Request->err_headers_out->add("Set-Cookie", $Cookie);
   }
-- 
2.0.0.rc2



More information about the wine-patches mailing list