search

SpartnerNL / Laravel-Excel

🚀 Supercharged Excel exports and imports in Laravel
https://laravel-excel.com
MIT License
12.18k stars 1.9k forks source link

[Bug]: #4129

Open nickjangveladze opened 3 months ago

nickjangveladze commented 3 months ago

Is the bug applicable and reproducable to the latest version of the package and hasn't it been reported before?

What version of Laravel Excel are you using?

3.1.55

What version of Laravel are you using?

10.48.4

What version of PHP are you using?

8.1.28

Describe your issue

When I use the s3 import, it goes into error in this section (TemporaryFile.php):

Screenshot 2024-05-14 at 21 23 23

$diskInstance->exists($filePath) returns: "Unable to check existence for: imports/01HXW0T96ASCR7F26RXJBD9Q4E.xlsx"

By removing this section, it works without any problems.

How can the issue be reproduced?

Remove this section from TemporaryFile.php:

Screenshot 2024-05-14 at 21 23 23

What should be the expected behaviour?

...

patrickbrouwers commented 2 months ago

Sounds related to https://github.com/SpartnerNL/Laravel-Excel/pull/4132

But not sure, as I'm missing context of what s3 import means for you. (Please show code)

quaddy commented 4 weeks ago

Same error here!

$import = new Import();
$import->import(
    filePath: 'filename-on-s3.csv',
    disk: 's3-import-disk',
    readerType: Excel::CSV,
);

TemporaryFile.php throws FileNotFoundException.