Core: Consider update of AWS PHP library

Created on 3 Jul 2017  路  9Comments  路  Source: owncloud/core

We last updated it 3 years ago https://github.com/owncloud/core/commit/e0056c4a0dc5be96cefe34461498350ac7d907a1 and there have been numerous fixes since. Can we also import it with composer? I just had to add a hack to fix signature creation for the London region for the files_external app.

files_external enhancement

Most helpful comment

Is there a timeline for the deprecation ?

I think we should move out the Amazon S3 into a separate app anyway and tackle this at the same time.

All 9 comments

Might also solve https://github.com/owncloud/core/issues/14893 or am i wrong?

Yes quite possibly. Amazon has some funky logic with their signature requirements - adding it as a parameter would get around this but doing it automatically is also cool

@pmaier1 @PVince81 We should add some prio here as this is hurting us more as/when Amazon are depreciating V3 signature and we don't have updated defaults for the regions. Either we update the lib or have to code in the default signature logic like I have done once already.

Is there a timeline for the deprecation ?

I think we should move out the Amazon S3 into a separate app anyway and tackle this at the same time.

files_external_s3 and files_primary_s3

Actually they moved all regions to supporting v4 - so we made core default to v4 now so this is less important.

Hey, this issue has been closed because the label status/STALE is set and there were no updates for 7 days. Feel free to reopen this issue if you deem it appropriate.

move to composer once we move this to an app

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

Was this page helpful?
0 / 5 - 0 ratings

Related issues

LukasReschke picture LukasReschke  路  91Comments

Revisor01 picture Revisor01  路  205Comments

jnfrmarks picture jnfrmarks  路  89Comments

sonanchenko picture sonanchenko  路  81Comments

bonanza123 picture bonanza123  路  81Comments