Guzzle requirement missing from composer.json about flarum-ext-auth-steam HOT 3 CLOSED

Actually, it isn't.

Flarum does not require the laravel/framework or laravel/laravel package directly, but instead directly requires some of the individual Laravel packages, none of them require Guzzle.

Laravel only added Guzzle as a requirement to laravel/laravel in a later version, but this has no influence on Flarum because we don't use the Laravel scaffolding.

composer why guzzlehttp/guzzle will show you which other dependencies actually require Guzzle on your forum. No Laravel packages are in there.

You can reproduce the issue by removing all other social login extensions and Upload with Composer. You'll see Guzzle is no longer installed.

Even if one of your dependencies had Guzzle as a dependency (which is not the case here), it's best practice to explicitly require it yourself as that dependency could drop Guzzle.

My report follows the findings discussed at https://discuss.flarum.org/d/23034-flarum-query-extensions-big-data-analysis/2

from flarum-ext-auth-steam.

Hello,
thanks for the info. I'll check it.

from flarum-ext-auth-steam.

Guzzle is required by Laravel, so is not necessary to explicitly require it.

from flarum-ext-auth-steam.