Merge branch 'async-aws:master' into master

Author: victor-upmeet

manifest.json

@@ -1,6 +1,6 @@
 {
     "variables": {
-        "${LATEST}": "3.340.1"
+        "${LATEST}": "3.340.4"
     },
     "endpoints": "https://raw.githubusercontent.com/aws/aws-sdk-php/${LATEST}/src/data/endpoints.json",
     "services": {

psalm.baseline.xml

@@ -246,11 +246,6 @@
       <code><![CDATA[list<Reason::*>]]></code>
     </MoreSpecificReturnType>
   </file>
-  <file src="src/Service/Route53/src/Result/CreateHostedZoneResponse.php">
-    <PossiblyNullPropertyAssignmentValue>
-      <code><![CDATA[$headers['location'][0] ?? null]]></code>
-    </PossiblyNullPropertyAssignmentValue>
-  </file>
   <file src="src/Service/S3/src/Result/ListObjectsV2Output.php">
     <LessSpecificReturnStatement>
       <code>$items</code>
@@ -331,5 +326,5 @@
       <code><![CDATA[list<KeyAgreementAlgorithmSpec::*>]]></code>
       <code><![CDATA[list<SigningAlgorithmSpec::*>]]></code>
     </MoreSpecificReturnType>
-  </file>  
+  </file>
 </files>

src/CodeGenerator/src/Generator/CodeGenerator/PopulatorGenerator.php

@@ -190,39 +190,39 @@ private function generatePopulator(Operation $operation, StructureShape $shape,
             $memberShape = $member->getShape();
             switch ($memberShape->getType()) {
                 case 'timestamp':
-                    $body .= strtr('$this->PROPERTY_NAME = isset($headers["LOCATION_NAME"][0]) ? new \DateTimeImmutable($headers["LOCATION_NAME"][0]) : null;' . "\n", [
-                        'PROPERTY_NAME' => $propertyName,
-                        'LOCATION_NAME' => $locationName,
-                    ]);
+                    $propertyValue = 'new \DateTimeImmutable($headers["LOCATION_NAME"][0])';
 
                     break;
                 case 'integer':
                 case 'long':
-                    $body .= strtr('$this->PROPERTY_NAME = isset($headers["LOCATION_NAME"][0]) ? (int) $headers["LOCATION_NAME"][0] : null;' . "\n", [
-                        'PROPERTY_NAME' => $propertyName,
-                        'LOCATION_NAME' => $locationName,
-                    ]);
+                    $propertyValue = '(int) $headers["LOCATION_NAME"][0]';
 
                     break;
                 case 'boolean':
                     $this->requirementsRegistry->addRequirement('ext-filter');
 
-                    $body .= strtr('$this->PROPERTY_NAME = isset($headers["LOCATION_NAME"][0]) ? filter_var($headers["LOCATION_NAME"][0], FILTER_VALIDATE_BOOLEAN) : null;' . "\n", [
-                        'PROPERTY_NAME' => $propertyName,
-                        'LOCATION_NAME' => $locationName,
-                    ]);
+                    $propertyValue = 'filter_var($headers["LOCATION_NAME"][0], FILTER_VALIDATE_BOOLEAN)';
 
                     break;
                 case 'string':
-                    $body .= strtr('$this->PROPERTY_NAME = $headers["LOCATION_NAME"][0] ?? null;' . "\n", [
-                        'PROPERTY_NAME' => $propertyName,
-                        'LOCATION_NAME' => $locationName,
-                    ]);
+                    $propertyValue = '$headers["LOCATION_NAME"][0]';
 
                     break;
                 default:
                     throw new \RuntimeException(\sprintf('Type %s is not yet implemented', $memberShape->getType()));
             }
+            if (!$member->isRequired()) {
+                if ('$headers["LOCATION_NAME"][0]' === $propertyValue) {
+                    $propertyValue .= '?? null';
+                } else {
+                    $propertyValue = 'isset($headers["LOCATION_NAME"][0]) ? ' . $propertyValue . ' : null';
+                }
+            }
+
+            $body .= strtr('$this->PROPERTY_NAME = ' . $propertyValue . ';' . "\n", [
+                'PROPERTY_NAME' => $propertyName,
+                'LOCATION_NAME' => $locationName,
+            ]);
         }
 
         // This will catch arbitrary values that exists in undefined "headers"

src/Service/Route53/CHANGELOG.md

@@ -6,6 +6,10 @@
 
 - AWS api-change: Added `us-isof-east-1`  and `us-isof-south-1` regions
 
+### Changed
+
+- trust AWS's API response: And not check if required headers are present
+
 ## 2.6.0
 
 ### Added

src/Service/Route53/src/Result/CreateHostedZoneResponse.php

@@ -90,7 +90,7 @@ protected function populateResult(Response $response): void
     {
         $headers = $response->getHeaders();
 
-        $this->location = $headers['location'][0] ?? null;
+        $this->location = $headers['location'][0];
 
         $data = new \SimpleXMLElement($response->getContent());
         $this->hostedZone = $this->populateResultHostedZone($data->HostedZone);

src/Service/Ssm/src/Exception/ParameterNotFoundException.php

@@ -6,6 +6,9 @@
 
 /**
  * The parameter couldn't be found. Verify the name and try again.
+ *
+ * > For the `DeleteParameter` and `GetParameter` actions, if the specified parameter doesn't exist, the
+ * > `ParameterNotFound` exception is *not* recorded in CloudTrail event logs.
  */
 final class ParameterNotFoundException extends ClientException
 {

src/Service/Ssm/src/Input/PutParameterRequest.php

@@ -13,7 +13,7 @@
 final class PutParameterRequest extends Input
 {
     /**
-     * The fully qualified name of the parameter that you want to add to the system.
+     * The fully qualified name of the parameter that you want to create or update.
      *
      * > You can't enter the Amazon Resource Name (ARN) for a parameter, only the parameter name itself.
      *
@@ -71,7 +71,7 @@ final class PutParameterRequest extends Input
     private $value;
 
     /**
-     * The type of parameter that you want to add to the system.
+     * The type of parameter that you want to create.
      *
      * > `SecureString` isn't currently supported for CloudFormation templates.
      *
@@ -89,8 +89,8 @@ final class PutParameterRequest extends Input
      * The Key Management Service (KMS) ID that you want to use to encrypt a parameter. Use a custom key for better
      * security. Required for parameters that use the `SecureString` data type.
      *
-     * If you don't specify a key ID, the system uses the default key associated with your Amazon Web Services account which
-     * is not as secure as using a custom key.
+     * If you don't specify a key ID, the system uses the default key associated with your Amazon Web Services account,
+     * which is not as secure as using a custom key.
      *
      * - To use a custom KMS key, choose the `SecureString` data type with the `Key ID` parameter.
      *

src/Service/Ssm/src/SsmClient.php

@@ -203,7 +203,7 @@ public function getParametersByPath($input): GetParametersByPathResult
     }
 
     /**
-     * Add a parameter to the system.
+     * Create or update a parameter in Parameter Store.
      *
      * @see https://docs.aws.amazon.com/systems-manager/latest/APIReference/API_PutParameter.html
      * @see https://docs.aws.amazon.com/aws-sdk-php/v3/api/api-ssm-2014-11-06.html#putparameter