-
-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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
Upgrade internal S3 library #2315
Comments
Is this something still possible with Nextcloud 11? @rullzer @icewind1991 |
Doubt it. 12 at earliest. (aka early next year) |
I wouldn't risk doing that for 11. But makes sense to do it early in the 12 cycle. |
Because this is blockig S3 for AWS in Frankfurt I think this is important 😁😅 |
Is v2 (the current implemenetation) still supported and "secure" if we are talking about authentification ;) |
Has the AWS PHP SDK changed that significantly over this time period? Or is this something that someone with only basic understanding of nexcloud app structuring can pick up? |
Updated the sdk in #4410 Please spend some time testing the various s3 integrations (not with production data ofc) |
Out of scope for 12 -> move to 13 |
MEH. Was hoping I could move my files into S3 now. |
~5-6 months after 12 I would say roughly. |
#4410 should be ready for review. Any help testing the changes would be greatly appreciated (and helps make sure they actually get merged 😄 ) |
one remark, even if maybe this is already present: it would be nice, at least in some future, that you could use AWS server side encryption for S3 objects as well. is that being done / considered or should I create another ticket? ref: http://docs.aws.amazon.com/AmazonS3/latest/dev/UsingServerSideEncryption.html |
Fixed with #4410 |
#60 is closed now, but the internal S3 lib is too old to support all AWS regions because of v2 vs. v4 authentication mechanisms.
So please upgrade the internal S3 connection lib to enable all regions (e.g. Frankfurt).
(ref: #768 (comment))
The text was updated successfully, but these errors were encountered: