fix: fix x-amzn-query-mode header (#3123)

* fix: fix x-amzn-query-mode header

This change makes the `x-amzn-query-mode` header to be a string value instead of a boolean one.

* chore: remove dynamic property creation

Dynamic property creation is not supported in php 8.4+.

* chore: add entry log change

* Update fix-query-header-mode.json

---------

Co-authored-by: Sean O'Brien <60306702+stobrien89@users.noreply.github.com>

Author: yenfryherrerafeliz

.changes/nextrelease/fix-query-header-mode.json

@@ -0,0 +1,7 @@
+[
+  {
+    "type": "enhancement",
+    "category": "",
+    "description": "Sets the header `x-amzn-query-mode` to a string value instead of a boolean."
+  }
+]

src/AwsClient.php

@@ -540,7 +540,7 @@ private function addQueryModeHeader(): void
             Middleware::mapRequest(function (RequestInterface $r) {
                 return $r->withHeader(
                     'x-amzn-query-mode',
-                    true
+                    "true"
                 );
             }),
             'x-amzn-query-mode-header'

tests/AwsClientTest.php

@@ -994,7 +994,7 @@ public function testQueryModeHeaderAdded(): void
         $list->setHandler(new MockHandler([new Result()]));
         $list->appendSign(Middleware::tap(function ($cmd, $req) {
             $this->assertTrue($req->hasHeader('x-amzn-query-mode'));
-            $this->assertEquals(true, $req->getHeaderLine('x-amzn-query-mode'));
+            $this->assertEquals("true", $req->getHeaderLine('x-amzn-query-mode'));
         }));
         $client->TestOperation();
     }

tests/Integ/CredentialsContext.php

@@ -20,6 +20,8 @@ class CredentialsContext extends Assert implements
     private static $credentialsFile;
     private static $roleName;
     private static $roleArn;
+    private $credentials;
+    private $client;
 
     /**
      * @BeforeFeature @credentials