-
Notifications
You must be signed in to change notification settings - Fork 764
[RUN-3090] Document changes for Amazon S3 SDK upgrade in migration guide #10440
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: development
Are you sure you want to change the base?
[RUN-3090] Document changes for Amazon S3 SDK upgrade in migration guide #10440
Conversation
|
Waiting for team review... |
|
Hi @emirmx ! Can you give us a bit of context behind this change? The migration docs are heavily used. |
Hi @ConnorLand. We upgraded the S3 SDK that we use to implement S3 file storage backend. The new version has different behavior causing breaking changes. |
|
|
||
| ### Amazon S3 SDK Upgrade | ||
|
|
||
| With Mendix 10.6 we started using new version of AWS SDK for accessing S3 storage. New SDK has some differences which affects our S3 storage implementation. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Which subversion of SP 10.6? Should this change get a SP 10.6.x release note? @emirmx
|
|
||
| ### Amazon S3 SDK Upgrade | ||
|
|
||
| With Mendix 10.6, we started using a new version of the AWS SDK for accessing S3 storage. The new SDK version has some differences which affects our S3 storage implementation. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Instead of "a new version", which AWS version is it exactly?
|
|
||
| #### com.mendix.storage.s3.Region / com.mendix.storage.s3.EndPoint Settings | ||
|
|
||
| SDK version ___ is stricter with these settings. `com.mendix.storage.s3.Region` setting should be always set to the region matching the region of the bucket. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We need to put the exact version here, instead of referring to it as "the newer SDK version".
|
|
||
| ### Amazon S3 SDK Upgrade | ||
|
|
||
| With Mendix 10.6, we started using a new version of the AWS SDK for accessing S3 storage. The new SDK version has some differences which affects our S3 storage implementation. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
10.6.x? Subversion? And should that get a retroactive release note?
|
|
||
| #### AWS Signature V2 support (com.mendix.storage.s3.UseV2Auth Setting) | ||
|
|
||
| SDK version ___ does not support AWS Signature v2 which is enabled by `UseV2Auth` setting. This signature type is deprecated, and is not supported by new regions. For more information, see AWS's ____ Documentation. We do not expect this have any effect when Amazon S3 is used. This will prevent use of S3 compatible solutions that only supports v2 signature type. For these cases you need to switch to either Amazon S3 or a compatible solution that supports newer signature types. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@emirmx -- AWS' documentation link, please.
No description provided.