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

List:       kde-commits
Subject:    KDE/kdelibs/kioslave/http
From:       Dawit Alemayehu <adawit () kde ! org>
Date:       2007-10-17 23:06:15
Message-ID: 1192662375.755138.5187.nullmailer () svn ! kde ! org
[Download RAW message or body]

SVN commit 726539 by adawit:

- Forward port the fix for BR 150904 and 150070 from KDE 3.5 branch.

BUG:150904
BUG:150070


 M  +4 -6      http.cpp  


--- trunk/KDE/kdelibs/kioslave/http/http.cpp #726538:726539
@@ -3290,12 +3290,6 @@
 
   } while (!m_bEOF && (len || noHeader) && (headerSize < maxHeaderSize) && \
(gets(buffer, sizeof(buffer)-1)));  
-  // Send the current response before processing starts or it
-  // might never get sent...
-  // Except if this is just a validation, then the cached request sends them
-  if (!cacheValidated)
-    forwardHttpResponseHeader();
-
   // Now process the HTTP/1.1 upgrade
   QStringList::Iterator opt = upgradeOffers.begin();
   for( ; opt != upgradeOffers.end(); ++opt) {
@@ -3647,6 +3641,10 @@
     mimeType( m_strMimeType );
   }
 
+  // Do not move send response header before any redirection as it seems
+  // to screw up some sites. See BR# 150904.
+  forwardHttpResponseHeader();
+
   if (m_request.method == HTTP_HEAD)
      return true;
 


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