Uploading to S3 with Laravel Intervention image returns boolean and not path

Question

I am having an issue with uploading an image in Laravel. Sometimes the image needs reorienting - when doing this with Intervention image it simply returns a boolean instead of the path of the image in s3. When using it without Intervention Image it works fine

I've tried exif reading data and using imagerotate to no avail, it errors out

I run the following

$image = $request->file('photo');
$path = \Storage::disk('s3')->put('users/'.\Auth::id().'/posts', $image, 'public');
dd($path); // /users/1/posts/39grjigrje.jpg

the $path variable is great and is the s3 path, however running:

$image = $request->file('photo');
$image = \Image::make($image);
$image->orientate();
$path = \Storage::disk('s3')->put('users/'.\Auth::id().'/posts', $image, 'public');
dd($path); // true

the $path variable is simply just a boolean and doesn't return the stored file path

I expect a path e.g. /images/1/kfjeieuge.jpg which I get when i don't use intervention image, when I use intervention I get a boolean.

Do you get false? Maybe that's an indication it did not work. Usually you have to store strings in files. — Roland Starke
– Roland Starke, Commented Jun 27, 2019 at 6:15
You can convert the intervention image to a string like ->put(..., (string) $image->encode('jpg', 75), ...); — Roland Starke
– Roland Starke, Commented Jun 27, 2019 at 6:19
@iLC, can you confirm if in the first case the file actually got uploaded also? $path = \Storage::disk('s3')->put('users/'.\Auth::id().'/posts', $image, 'public'); — Tarun Lalwani
– Tarun Lalwani, Commented Jul 4, 2019 at 9:30

Nitesh · Accepted Answer · 2019-07-02 06:55:34Z

4

I too had faced the same issue a few years back. I solved that by using following steps:

1.You may have to encode the image after an Intervention operation on it. To achieve encoding refer this documentation on stream method:Create encoded image stream

Encodes the current image in given format and given image quality and creates new PSR-7 stream based on image data.

$image = Image::make(request()->file('image'))->orientate()->encode('jpg');

2.And then storing it with a stream

$path = Storage::disk('s3')->put('users/'.\Auth::id().'/posts', $image->stream(),'public');

This should achieve your objective.

answered Jul 2, 2019 at 6:55

Nitesh

871 silver badge7 bronze badges

Sign up to request clarification or add additional context in comments.

1 Comment

iLC Over a year ago

Doing this still returns 1

dns_nx · Accepted Answer · 2021-11-03 06:09:18Z

In first example:

$image = $request->file('photo');
$path = \Storage::disk('s3')->put('users/'.\Auth::id().'/posts', $image, 
'public');

$request->file('photo') file will return instance of Illuminate\Http\UploadedFile class. you can check it here in documentation link.

And as Tarun said put method check for instance of Illuminate\Http\UploadedFile class. So in that case, it will uploaded successful.

In second example:

$image = $request->file('photo');
$image = \Image::make($image); //here
$image->orientate();
$path = \Storage::disk('s3')->put('users/' . \Auth::id() . '/posts', $image, 'public');

You are overwriting $image variable with instance Image class (second line). Which is wrong. You need to pass Stream or file.

Try below code:

$image = $request->file('photo');
$image = \Image::make($image); //here
$image->orientate()->encode('jpg');
$filename = time() . '.jpg';
$path = \Storage::disk('s3')->put('users/' . \Auth::id() . '/posts/'.$filename, $image->getEncoded(), 'public');

Tarun Lalwani · Accepted Answer · 2019-07-05 07:27:17Z

0

You can solve it as described in the below link

https://laracasts.com/discuss/channels/laravel/saving-an-intervention-image-instance-into-amazon-s3

$path = \Storage::disk('s3')->put('users/'.\Auth::id().'/posts', $image->stream()->__toString(), 'public');

Update: 5 Jul

If you look at the source code it is supposed to return bool only

https://github.com/laravel/framework/blob/c7bdf66062af3c63648f7c29591476e6db92c885/src/Illuminate/Filesystem/FilesystemAdapter.php#L192

/**
     * Write the contents of a file.
     *
     * @param  string  $path
     * @param  string|resource  $contents
     * @param  mixed  $options
     * @return bool
     */
    public function put($path, $contents, $options = [])
    {
        $options = is_string($options)
                     ? ['visibility' => $options]
                     : (array) $options;
        // If the given contents is actually a file or uploaded file instance than we will
        // automatically store the file using a stream. This provides a convenient path
        // for the developer to store streams without managing them manually in code.
        if ($contents instanceof File ||
            $contents instanceof UploadedFile) {
            return $this->putFile($path, $contents, $options);
        }
        return is_resource($contents)
                ? $this->driver->putStream($path, $contents, $options)
                : $this->driver->put($path, $contents, $options);
    }

And as you can see a Stream and file is handled differently

And the ideal thing is to use url function to get the url

Storage::disk('s3')->url($filename)

edited Jul 5, 2019 at 7:27

answered Jul 2, 2019 at 7:14

Tarun Lalwani

147k11 gold badges217 silver badges278 bronze badges

2 Comments

iLC Over a year ago

this just returns true

Dimitri Mostrey Over a year ago

@iLC yes, it does return true if all went well. But did it save the picture? Turan is right, you need to stringify the image in order to save an object on the S3. What do you see if you run \Storage::allDirectories('users'); in tinker? If you see nothing, then there is something wrong with the IAM settings.

bhavinG · Accepted Answer · 2019-07-06 10:31:05Z

0

Try this

$image = \Image::make($request->file('photo')->getRealpath());
$image->orientate();


$image->encode('jpg');
 OR
$image->response('jpg', 100);


$path = \Storage::disk('s3')->put('users/'.\Auth::id().'/posts', $image, 'public');

edited Jul 6, 2019 at 10:31

answered Jul 6, 2019 at 9:10

bhavinG

193 bronze badges

Collectives™ on Stack Overflow

Uploading to S3 with Laravel Intervention image returns boolean and not path

4 Answers 4

1 Comment

Comments

2 Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

1 Comment

Comments

2 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related