Skip to content

Upgrade internal S3 library #2315

Description

@flypenguin

#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))

Activity

  1. MariusBluem commented on Nov 24, 2016

    @MariusBluem
    Member

    Is this something still possible with Nextcloud 11? @rullzer @icewind1991

  2. LukasReschke commented on Nov 24, 2016

    @LukasReschke
    Member

    Doubt it. 12 at earliest. (aka early next year)

  3. rullzer commented on Nov 24, 2016

    @rullzer
    Member

    I wouldn't risk doing that for 11. But makes sense to do it early in the 12 cycle.

  4. MariusBluem commented on Nov 24, 2016

    @MariusBluem
    Member

    Because this is blockig S3 for AWS in Frankfurt I think this is important 😁😅

  5. added this to the Nextcloud 12.0 milestone on Dec 8, 2016
  6. MariusBluem commented on Dec 10, 2016

    @MariusBluem
    Member

    Is v2 (the current implemenetation) still supported and "secure" if we are talking about authentification ;)

  7. 8BitAce commented on Apr 19, 2017

    @8BitAce

    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?

  8. icewind1991 commented on Apr 20, 2017

    @icewind1991
    Member

    Updated the sdk in #4410

    Please spend some time testing the various s3 integrations (not with production data ofc)

  9. MorrisJobke commented on May 4, 2017

    @MorrisJobke
    Member

    Out of scope for 12 -> move to 13

  10. flypenguin commented on May 4, 2017

    @flypenguin
    Author

    MEH. Was hoping I could move my files into S3 now.
    Any timeline for 13?

  11. MorrisJobke commented on May 4, 2017

    @MorrisJobke
    Member

    Any timeline for 13?

    ~5-6 months after 12 I would say roughly.

  12. icewind1991 commented on Jun 7, 2017

    @icewind1991
    Member

    #4410 should be ready for review.

    Any help testing the changes would be greatly appreciated (and helps make sure they actually get merged 😄 )

  13. flypenguin commented on Aug 24, 2017

    @flypenguin
    Author

    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

  14. MorrisJobke commented on Nov 14, 2017

    @MorrisJobke
    Member

    Fixed with #4410

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

      Milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions