Skip to content

Commit

Permalink
Fix: update CloudFrontKeyValueStore client to use AWS4aSignerCRTWrapper
Browse files Browse the repository at this point in the history
  • Loading branch information
muhammad-othman committed Jan 12, 2024
1 parent 459d2e5 commit f273701
Show file tree
Hide file tree
Showing 5 changed files with 16 additions and 3 deletions.
11 changes: 11 additions & 0 deletions generator/.DevConfigs/9dba8c79-b860-4427-9ea4-f8625d376658.json
Original file line number Diff line number Diff line change
@@ -0,0 +1,11 @@
{
"services": [
{
"serviceName": "CloudFrontKeyValueStore",
"type": "patch",
"changeLogMessages": [
"Fixed CloudFrontKeyValueStore to use Sigv4a instead of Sigv4."
]
}
]
}
2 changes: 2 additions & 0 deletions generator/ServiceClientGeneratorLib/GeneratorHelpers.cs
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,8 @@ public static string DetermineSigner(string signatureVersion, string serviceBase
// we should not continue to add new hardcoded service specific signers
// and instead implement a solution based on a signer selection specification
return "EventBridgeSigner";
case "CloudFrontKeyValueStore":
return "AWS4aSignerCRTWrapper";
}

switch (signatureVersion)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -225,7 +225,7 @@ public AmazonCloudFrontKeyValueStoreClient(string awsAccessKeyId, string awsSecr
/// </summary>
protected override AbstractAWSSigner CreateSigner()
{
return new AWS4Signer();
return new AWS4aSignerCRTWrapper();
}

/// <summary>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -225,7 +225,7 @@ public AmazonCloudFrontKeyValueStoreClient(string awsAccessKeyId, string awsSecr
/// </summary>
protected override AbstractAWSSigner CreateSigner()
{
return new AWS4Signer();
return new AWS4aSignerCRTWrapper();
}

/// <summary>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -229,7 +229,7 @@ public ICloudFrontKeyValueStorePaginatorFactory Paginators
/// </summary>
protected override AbstractAWSSigner CreateSigner()
{
return new AWS4Signer();
return new AWS4aSignerCRTWrapper();
}

/// <summary>
Expand Down

0 comments on commit f273701

Please sign in to comment.