adds SOAPAction header for soap 1.1

Author: laland

src/VCR/LibraryHooks/SoapHook.php

@@ -69,8 +69,18 @@ public function doRequest($request, $location, $action, $version, $one_way = 0,
  }
 
  $vcrRequest = new Request('POST', $location);
- $contentType = ($version == SOAP_1_2) ? 'application/soap+xml' : 'text/xml';
- $vcrRequest->setHeader('Content-Type', $contentType . '; charset=utf-8; action="' . $action . '"');
+
+ switch ($version) {
+ case SOAP_1_1:
+ $vcrRequest->setHeader('Content-Type', 'text/xml; charset=utf-8;');
+ $vcrRequest->setHeader('SOAPAction', $action);
+ break;
+
+ case SOAP_1_2:
+ $vcrRequest->setHeader('Content-Type', sprintf('application/soap+xml; charset=utf-8; action="%s"', $action));
+ break;
+ }
+
  $vcrRequest->setBody($request);
 
  if (!empty($options['login'])) {

tests/VCR/LibraryHooks/SoapHookTest.php

@@ -42,8 +42,11 @@ public function testShouldInterceptCallWhenEnabled()
 
  public function testShouldHandleSOAPVersion11()
  {
- $expectedHeader = 'text/xml; charset=utf-8; action="http://ws.cdyne.com/WeatherWS/GetCityWeatherByZIP"';
- $this->soapHook->enable($this->getHeaderCheckCallback($expectedHeader));
+ $expectedHeaders = [
+ 'Content-Type' => 'text/xml; charset=utf-8;',
+ 'SOAPAction' => 'http://ws.cdyne.com/WeatherWS/GetCityWeatherByZIP',
+ ];
+ $this->soapHook->enable($this->getHeadersCheckCallback($expectedHeaders));
 
  $client = new \SoapClient(
  'http://wsf.cdyne.com/WeatherWS/Weather.asmx?WSDL',
@@ -89,4 +92,19 @@ protected function getHeaderCheckCallback($expectedHeader)
  return new Response(200, array(), '');
  };
  }
+
+ /**
+ * @param array $expectedHeaders
+ * @return \callable
+ */
+ protected function getHeadersCheckCallback(array $expectedHeaders)
+ {
+ $test = $this;
+ return function (Request $request) use ($test, $expectedHeaders) {
+ foreach ($expectedHeaders as $expectedHeaderName => $expectedHeader) {
+ $test->assertEquals($expectedHeader, $request->getHeader($expectedHeaderName));
+ }
+ return new Response(200, array(), '');
+ };
+ }
 }