Changeset 151668 in webkit

Timestamp:

Jun 17, 2013 11:09:40 PM (11 years ago)

Author:

commit-queue@webkit.org

Message:

[curl] Set the http response status text
​https://bugs.webkit.org/show_bug.cgi?id=117307

Patch by Peter Gal <​galpeter@inf.u-szeged.hu> on 2013-06-17
Reviewed by Brent Fulgham.

No new tests, covered by existing ones.

• platform/network/curl/ResourceHandleManager.cpp:

(WebCore::headerCallback):

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• platform/network/curl/ResourceHandleManager.cpp (modified) (1 diff)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2013-06-17  Peter Gal  <galpeter@inf.u-szeged.hu>
+
+        [curl] Set the http response status text
+        https://bugs.webkit.org/show_bug.cgi?id=117307
+
+        Reviewed by Brent Fulgham.
+
+        No new tests, covered by existing ones.
+
+        * platform/network/curl/ResourceHandleManager.cpp:
+        (WebCore::headerCallback):
+
 2013-06-14  Brent Fulgham  <bfulgham@apple.com>
 

trunk/Source/WebCore/platform/network/curl/ResourceHandleManager.cpp

@@ -367 +367 @@
             else
                 d->m_response.setHTTPHeaderField(key, value);
+        } else if (header.startsWith("HTTP", false)) {
+            // This is the first line of the response.
+            // Extract the http status text from this.
+            //
+            // If the FOLLOWLOCATION option is enabled for the curl handle then
+            // curl will follow the redirections internally. Thus this header callback
+            // will be called more than one time with the line starting "HTTP" for one job.
+            long httpCode = 0;
+            curl_easy_getinfo(d->m_handle, CURLINFO_RESPONSE_CODE, &httpCode);
+
+            String httpCodeString = String::number(httpCode);
+            int statusCodePos = header.find(httpCodeString);
+
+            if (statusCodePos != -1) {
+                // The status text is after the status code.
+                String status = header.substring(statusCodePos + httpCodeString.length());
+                d->m_response.setHTTPStatusText(status.stripWhiteSpace());
+            }
+
         }
     }