Skip to content

Conversation

@ggallotti
Copy link
Member

@ggallotti ggallotti commented Jul 5, 2022

  • Bump AWSSDK.S3 to latest Version
  • Set forced use of Signature V4 for AWS for us-east-1. **

Only for Region 'us-east-1', the SDK does not use Signature V4, instead uses SigV2 (for compatibility reasons) that we don't need:

Configures if the S3 client should use Signature Version 4 signing with requests. By default, this setting is set to true which will use Signature Version 4 for all requests except presigned URL requests when the region is set to us-east-1. When UseSignatureVersion4 is explicitly set to true by directly setting this property or directly setting this property through configuration, Signature Version 4 will be used for all requests when able to do so. When this setting is false, Signature Version 2 will be used. Note that when the setting is false, Signature Version 4 may still be used by default in some cases or with some regions.

https://docs.aws.amazon.com/sdkfornet/v3/apidocs/items/Amazon/TAWSConfigsS3.html

- Bump S3 SDK version
@genexusbot
Copy link
Collaborator

Cherry pick to beta success

@ggallotti ggallotti had a problem deploying to external-storage-tests July 5, 2022 20:40 Failure
@ggallotti ggallotti temporarily deployed to external-storage-tests July 5, 2022 20:55 Inactive
Copy link
Collaborator

@claudiamurialdo claudiamurialdo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is the same version change required for .NET Framework?.
dotnet/src/dotnetframework/Providers/Storage/GXAmazonS3/GXAmazonS3.csproj also references AWSSDK.S3 version 3.7.3.15.

@genexusbot
Copy link
Collaborator

Cherry pick to beta success

@ggallotti ggallotti temporarily deployed to external-storage-tests July 15, 2022 19:27 Inactive
@ggallotti
Copy link
Member Author

ggallotti commented Jul 15, 2022

Is the same version change required for .NET Framework?. dotnet/src/dotnetframework/Providers/Storage/GXAmazonS3/GXAmazonS3.csproj also references AWSSDK.S3 version 3.7.3.15.

Right, fixed!

@ggallotti ggallotti merged commit 7274627 into master Jul 25, 2022
@ggallotti ggallotti deleted the s3-use-sigv4 branch July 25, 2022 23:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants