Skip to content
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

Avoid reading ~/.aws/config when using S3 provider #23273

Closed
wants to merge 1 commit into from
Closed

Avoid reading ~/.aws/config when using S3 provider #23273

wants to merge 1 commit into from

Conversation

FlorentCoppint
Copy link
Contributor

When using S3 storage, we sometimes have open_basedir errors due to aws-sdk-php trying to read ~/.aws/config which is out of open_basedir restrictions.
Christoph Wurst already added csm=false config in #21406 but it wasn't enough, we also need to set use_arn_region=false, added in this commit.

Signed-off-by: Florent [email protected]

@FlorentCoppint
Copy link
Contributor Author

@ChristophWurst
Copy link
Member

/backport to stable20

@ChristophWurst
Copy link
Member

/backport to stable19

@ChristophWurst
Copy link
Member

/backport to stable18

@ChristophWurst ChristophWurst added this to the Nextcloud 21 milestone Oct 8, 2020
When using S3 storage, we sometimes have open_basedir errors due to aws-sdk-php trying to read ~/.aws/config which is out of open_basedir restrictions.
Christoph Wurst already added csm=false config in #21406 but it wasn't enough, we also need to set use_arn_region=false, added in this commit.

Signed-off-by: Florent <[email protected]>
@rullzer
Copy link
Member

rullzer commented Oct 24, 2020

mmmm I don't have a test setup for this.
But does it still read the file now if one of the other profiles has needs this info?

@edek51
Copy link

edek51 commented Nov 12, 2020

I can confirm that the problem still exist.
Zrzut ekranu 2020-11-13 o 00 08 13

mmmm I don't have a test setup for this.
But does it still read the file now if one of the other profiles has needs this info?

@rullzer rullzer mentioned this pull request Dec 14, 2020
59 tasks
@kesselb
Copy link
Contributor

kesselb commented Dec 14, 2020

@FlorentCoppint mind to rebase to have a recent CI run?

@rullzer rullzer mentioned this pull request Dec 18, 2020
39 tasks
This was referenced Dec 28, 2020
@rullzer rullzer modified the milestones: Nextcloud 21, Nextcloud 22 Jan 8, 2021
@J0WI
Copy link
Contributor

J0WI commented Apr 28, 2021

🏓 @FlorentCoppint

@MorrisJobke
Copy link
Member

Rebased version is in #27040 Let's continue there.

@MorrisJobke
Copy link
Member

#27040 is merged

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.

Setting up S3 External Storage with open_basedir restriction throws InvalidArgumentException
7 participants