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

Storage::download not working with S3 #31586

Closed
caseydwyer opened this issue Feb 25, 2020 · 5 comments
Closed

Storage::download not working with S3 #31586

caseydwyer opened this issue Feb 25, 2020 · 5 comments
Labels

Comments

@caseydwyer
Copy link

  • Laravel Version: 6.16.0
  • PHP Version: 7.3.8
  • Database Driver & Version: N/A

Description:

The Storage::download method seems broken. Might be an issue with flysystem-aws-s3-v3, but thought I'd start here.

Steps To Reproduce:

  1. Fresh Laravel install.
  2. composer require league/flysystem-aws-s3-v3 ~1.0
  3. Enter S3 credentials into .env
  4. Route for downloading an existing, public file. eg:
return \Storage::disk('s3')->download( '/_testing/sample-image.jpg' );
  1. Observe failed download.

failed-download

@driesvints
Copy link
Member

Wondering if it's related to #31163

Does downgrading to v6.11.0 fix the problem? And if so: does upgrading to v6.12.0 break it again?

@ghost
Copy link

ghost commented Feb 25, 2020

Please give Laravel error logs file content.

@caseydwyer
Copy link
Author

@driesvints - Thanks for the suggestion! Can confirm. Going down to 6.11.0 resolves the issue, going back up to 6.12 breaks it again. @voyula - logs are empty/non-existent.

@ghost
Copy link

ghost commented Feb 25, 2020

I think output buffering a problem for downloading data.

I was say. #31163 (comment)

@driesvints
Copy link
Member

We've reverted the pr that caused this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants