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

List:       kde-commits
Subject:    =?utf-8?q?=5Bkdelibs=5D_kioslave/http=3A_Remove_the_sending_of_D?=
From:       Dawit Alemayehu <adawit () kde ! org>
Date:       2011-04-17 16:33:03
Message-ID: 20110417163303.933B2A609B () git ! kde ! org
[Download RAW message or body]

Git commit 8cbae684f47f12f5f5177d0e438a0c3f0ea9a7e5 by Dawit Alemayehu.
Committed on 16/04/2011 at 21:07.
Pushed by adawit into branch 'master'.

Remove the sending of DNT:0 since it is redundant with the spec.
If the header is missing, it is assumed you want to OPT-IN to be tracked.

Also do not send the DNT:1 header by default. It must be explicitly enabled
by the user from the configuration dialog.

M  +1    -3    kioslave/http/http.cpp     

http://commits.kde.org/kdelibs/8cbae684f47f12f5f5177d0e438a0c3f0ea9a7e5

diff --git a/kioslave/http/http.cpp b/kioslave/http/http.cpp
index f7a5ea8..58d730a 100644
--- a/kioslave/http/http.cpp
+++ b/kioslave/http/http.cpp
@@ -2480,10 +2480,8 @@ bool HTTPProtocol::sendQuery()
     }
 
     // DoNotTrack feature...
-    if (config()->readEntry("DoNotTrack", true))
+    if (config()->readEntry("DoNotTrack", false))
       header += QLatin1String("DNT: 1\r\n");
-    else
-      header += QLatin1String("DNT: 0\r\n");
 
     // Remember that at least one failed (with 401 or 407) request/response
     // roundtrip is necessary for the server to tell us that it requires

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