Merge pull request #519 from janusman/fix-Http-adapter-extraction

Fixes bugs from PR 484; fix Http adapter for extraction requests

library/Solarium/Core/Client/Adapter/Http.php

@@ -111,13 +111,30 @@ class Http extends Configurable implements AdapterInterface
 
         if ($method == Request::METHOD_POST) {
             if ($request->getFileUpload()) {
+                $boundary = '----------' . md5(time());
+                $CRLF = "\r\n";
+                $file = $request->getFileUpload();
+                // Add the proper boundary to the Content-Type header
+                $headers = $request->getHeaders();
+                // Remove the Content-Type header, because we will replace it with something else.
+                if (($key = array_search("Content-Type: multipart/form-data", $headers)) !== false) {
+                  unset($headers[$key]);
+                }
+                $request->setHeaders($headers);
+                $request->addHeader("Content-Type: multipart/form-data; boundary={$boundary}");
+                $data =  "--{$boundary}" . $CRLF;
+                $data .= 'Content-Disposition: form-data; name="upload"; filename=' . $file . $CRLF;
+                $data .= 'Content-Type: application/octet-stream' . $CRLF . $CRLF;
+                $data .= file_get_contents($file) . $CRLF;
+                $data .= '--' . $boundary . '--';
+                $content_length = strlen($data);
+                $request->addHeader("Content-Length: $content_length\r\n");
                 stream_context_set_option(
                     $context,
                     'http',
                     'content',
-                    file_get_contents($request->getFileUpload())
+                    $data
                 );
-                $request->addHeader('Content-Type: multipart/form-data');
             } else {
                 $data = $request->getRawData();
                 if (null !== $data) {

tests/Solarium/Tests/Core/Client/Adapter/HttpTest.php

@@ -184,16 +184,20 @@ class HttpTest extends \PHPUnit_Framework_TestCase
 
         $context = $this->adapter->createContext($request, $endpoint);
 
+        // Remove content from comparison, since we can't determine the
+        // random boundary string.
+        $stream_context_get_options = stream_context_get_options($context);
+        unset($stream_context_get_options['http']['content']);
+        unset($stream_context_get_options['http']['header']);
+
         $this->assertEquals(
             array(
                 'http' => array(
                     'method' => $method,
                     'timeout' => $timeout,
-                    'content' => file_get_contents(__FILE__),
-                    'header' => 'Content-Type: multipart/form-data',
                 )
             ),
-            stream_context_get_options($context)
+            $stream_context_get_options
         );
     }