fix(laravel-soap-126): 400 Bad Request, XML not as expected (#138)

* fix(laravel-soap-126): 400 Bad Request, XML not as expected
Added mustUnderstand option for Wsse Header

* Apply fixes from StyleCI (#139)

Co-authored-by: Gregor Becker <[email protected]>

Co-authored-by: Gregor Becker <[email protected]>
Co-authored-by: Gregor Becker <[email protected]>

Author: CodeDredd

src/Middleware/WsseMiddleware.php

@@ -62,6 +62,11 @@ class WsseMiddleware extends Middleware
      */
     private $encrypt = false;
 
+    /**
+     * @var bool
+     */
+    private $mustUnderstand = true;
+
     /**
      * @var bool
      */
@@ -127,7 +132,7 @@ public function withServerCertificateHasSubjectKeyIdentifier(bool $hasSubjectKey
     public function beforeRequest(callable $handler, RequestInterface $request): Promise
     {
         $xml = SoapXml::fromStream($request->getBody());
-        $wsse = new WSSESoap($xml->getXmlDocument());
+        $wsse = new WSSESoap($xml->getXmlDocument(), $this->mustUnderstand);
 
         // Prepare the WSSE soap class:
         $wsse->signAllHeaders = $this->signAllHeaders;

tests/Unit/SoapClientTest.php

@@ -40,6 +40,21 @@ public function testMagicCallByConfig()
         self::assertTrue($response->ok());
     }
 
+    public function testWsseWithWsaCall()
+    {
+        Soap::fake();
+        $client = Soap::baseWsdl('https://laravel-soap.wsdl')->withWsse([
+            'userTokenName' => 'Test',
+            'userTokenPassword' => 'passwordTest',
+            'mustUnderstand' => false,
+        ])->withWsa();
+        $response = $client->Get_User();
+        $lastRequestInfo = $client->getEngine()->collectLastRequestInfo();
+        self::assertStringNotContainsString('mustUnderstand', $lastRequestInfo->getLastRequest());
+//        dd($client->debugLastSoapRequest());
+        self::assertTrue($response->ok());
+    }
+
     public function testArrayAccessResponse()
     {
         Soap::fakeSequence()->push('test');