[Question] Path Prefixing

[iMusicJJ]

Hello!

How do you set-up the path prefixing?

I've attempted to set it up with Laravel, and it uploads the files as expected in the prefixed path, but upon URL retrieval it throws:
"This driver does not support retrieving URLs.",

Do you have an example with a prefixed path?

[sifex]

Hey, sure thing,

Should be as simple as wrapping your BunnyCDNAdapter inside of a PathPrefixAdapter

Edit 2: Yea sorry you also have to composer require league/flysystem-path-prefixing:^3.3

So update

$adapter = new BunnyCDNAdapter(
    new BunnyCDNClient(
        $config['storage_zone'],
        $config['api_key'],
        $config['region']
    ),
    'http://testing.b-cdn.net' # Optional
);

to be

$adapter = new PathPrefixedAdapter(
    new BunnyCDNAdapter(
        new BunnyCDNClient(
            $config['storage_zone'],
            $config['api_key'],
            $config['region']
        ),
        'http://testing.b-cdn.net' # Optional
    ),
    'prefix'
);

(Disclaimer) I wrote it by hand so no idea if it works. It's got test coverage though

Source: https://flysystem.thephpleague.com/docs/adapter/path-prefixing/

[iMusicJJ]

Funky, that is what I have; I believe.

$adapter = new BunnyCDNAdapter(
    new BunnyCDNClient(
        $config['storage_zone'],
        $config['api_key'],
        $config['region']
    ),
    $config['pullzone_url'] ?? null
);

$adapter = new PathPrefixedAdapter($adapter, 'images/mails');

Which is what throws the exception when trying to generate the URL.

Full example:

Storage::extend('bunnycdn-mail', function (Application $app, array $config) {
            $adapter = new BunnyCDNAdapter(
                new BunnyCDNClient(
                    $config['storage_zone'],
                    $config['api_key'],
                    $config['region']
                ),
                $config['pullzone_url'] ?? null
            );

            $adapter = new PathPrefixedAdapter($adapter, 'images/mails');

            return new FilesystemAdapter(
                new Filesystem($adapter, $config),
                $adapter,
                $config);
        });

[sifex]

Let me have a quick look now for ya

[sifex]

So you're not going to be able to get nice IDE completion, but there is a fall-through for calling adapter methods on the Storage facade.

tldr; You can call Storage::publicUrl instead of Storage::url and that seems to work for the moment.

[iMusicJJ]

Oohhh, this is where it gets funky.
I don't have access to the Storage::url call, it's happening inside mailcoach

I am trying to store email images on Bunny, instead of locally.

[sifex]

Huh.... Well.

Because PathPrefixedAdapter doesn't have a getUrl method (I think v3 changed the method name), it's gonna throw "This driver does not support retrieving URLs.".

The quick & dirty:

Best way I'm going to suggest is to extend the PathPrefixAdapter with a getUrl method.

# /app/Providers/Support/PathPrefixedAdapter.php

<?php

namespace App\Providers\Support;

use League\Flysystem\Config;

class PathPrefixedAdapter extends \League\Flysystem\PathPrefixing\PathPrefixedAdapter
{
    public function getUrl($path): string
    {
        return $this->publicUrl($path, new Config());
    }
}

Then use that PathPrefixedAdapter instead of the one provided by \League\Flysystem

The actual fix:

src/Illuminate/Filesystem/FilesystemAdapter.php needs a lot of attention given the advances in Flysystem v3

[sifex]

I'll open a quick PR for Laravel now

[iMusicJJ]

Ah great! Thanks for the amazingly quick response!