[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    kdelibs/kioslave/http/kcookiejar
From:       Waldo Bastian <bastian () kde ! org>
Date:       2005-02-28 23:13:53
Message-ID: 20050228231353.5D1AF16F7D () office ! kde ! org
[Download RAW message or body]

CVS commit by waba: 

Use correct domain for storing cookie policy
BUG: 100465


  M +3 -10     kcookiejar.cpp   1.122
  M +3 -2      kcookieserver.cpp   1.58


--- kdelibs/kioslave/http/kcookiejar/kcookieserver.cpp  #1.57:1.58
@@ -560,5 +560,6 @@ KCookieServer::setDomainAdvice(QString u
       QStringList domains;
       mCookieJar->extractDomains(fqdn, domains);
-      mCookieJar->setDomainAdvice(domains[0],
+      
+      mCookieJar->setDomainAdvice(domains[domains.count() > 3 ? 3 : 0],
                                   KCookieJar::strToAdvice(advice));
       // Save the cookie config if it has changed

--- kdelibs/kioslave/http/kcookiejar/kcookiejar.cpp  #1.121:1.122
@@ -999,12 +999,4 @@ KCookieAdvice KCookieJar::cookieAdvice(K
        if (!valid)
        {
-          // Maybe the domain doesn't start with a "."
-          QString domain = '.' + cookiePtr->domain();
-          if (domains.contains(domain))
-             valid = true;
-       }
-       
-       if (!valid)
-       {
           // Maybe it points to a sub-domain
           if (cookiePtr->domain().endsWith("."+cookiePtr->host()))
@@ -1113,5 +1105,6 @@ void KCookieJar::setDomainAdvice(const Q
 void KCookieJar::setDomainAdvice(KHttpCookiePtr cookiePtr, KCookieAdvice _advice)
 {
-    QString domain = stripDomain(cookiePtr); // We file the cookie under this domain.
+    QString domain;
+    stripDomain(cookiePtr->host(), domain); // We file the cookie under this domain.
 
     setDomainAdvice(domain, _advice);


[prev in list] [next in list] [prev in thread] [next in thread]