Skip to content

Commit 35d2fc9

Browse files
committed
fix: Update endpoint resolver to handle anonymous credentials when trying to retrieve account IDs
1 parent e42eb5c commit 35d2fc9

File tree

3 files changed

+13
-2
lines changed

3 files changed

+13
-2
lines changed
Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,11 @@
1+
{
2+
"services": [
3+
{
4+
"serviceName": "DynamoDBv2",
5+
"type": "patch",
6+
"changeLogMessages": [
7+
"Update endpoint resolver to handle anonymous credentials when trying to retrieve account IDs"
8+
]
9+
}
10+
]
11+
}

generator/ServiceClientGeneratorLib/Generators/Endpoints/EndpointResolver.partial.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ private string GetValueSource(Parameter parameter)
2525
case "AWS::S3::DisableMultiRegionAccessPoints": return "config.DisableMultiregionAccessPoints";
2626
case "AWS::S3::UseGlobalEndpoint": return "config.USEast1RegionalEndpointValue == S3UsEast1RegionalEndpointValue.Legacy";
2727
case "AWS::STS::UseGlobalEndpoint": return "false";
28-
case "AWS::Auth::AccountId": return "requestContext.Identity is AWSCredentials credentials ? credentials.GetCredentials().AccountId : null";
28+
case "AWS::Auth::AccountId": return "requestContext.Identity is AWSCredentials credentials ? credentials.GetCredentials()?.AccountId : null";
2929
case "AWS::Auth::AccountIdEndpointMode": return "config.AccountIdEndpointMode.ToString().ToLower()";
3030
default: throw new Exception("Unknown builtIn");
3131
}

sdk/src/Services/DynamoDBv2/Generated/Internal/AmazonDynamoDBEndpointResolver.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,7 @@ protected override EndpointParameters MapEndpointsParameters(IRequestContext req
5454
result.UseDualStack = config.UseDualstackEndpoint;
5555
result.UseFIPS = config.UseFIPSEndpoint;
5656
result.Endpoint = config.ServiceURL;
57-
result.AccountId = requestContext.Identity is AWSCredentials credentials ? credentials.GetCredentials().AccountId : null;
57+
result.AccountId = requestContext.Identity is AWSCredentials credentials ? credentials.GetCredentials()?.AccountId : null;
5858
result.AccountIdEndpointMode = config.AccountIdEndpointMode.ToString().ToLower();
5959

6060

0 commit comments

Comments
 (0)