Skip to content

Guzzle 6 #207

@AlexanderBech

Description

@AlexanderBech

Hi,

I got a composer package that requires guzzle 7.4. But can't upgrade to guzzle 7.4 because it says php-sparkpost needs guzzle 6.
What to do?

Activity

ainsleyclark

ainsleyclark commented on Nov 7, 2022

@ainsleyclark

Any updates on this?

ainsleyclark

ainsleyclark commented on Nov 7, 2022

@ainsleyclark

This seems to work:
composer require php-http/guzzle7-adapter
Then:
use Http\Adapter\Guzzle7\Client as GuzzleAdapter;

phpfui

phpfui commented on Nov 7, 2022

@phpfui

@ainsleyclark Thanks for this. You just saved me hours! And just in time as well.

But the package should be updated.

ainsleyclark

ainsleyclark commented on Nov 7, 2022

@ainsleyclark

Glad I could help, yeah would be good if this is updated to 7

andrews05

andrews05 commented on Feb 16, 2023

@andrews05

I'm a little confused about the issue here. The sparkpost package itself does not require guzzle at all, it's only the readme that tells you to install guzzle 6. Are you saying the readme should be updated?

ainsleyclark

ainsleyclark commented on Feb 16, 2023

@ainsleyclark

The sparkpost composer.json references Guzzle 6. It would be good to get it updated to 7 to fix above. It's cross referenced; you can't use this package with another package that uses Guzzle 7.

andrews05

andrews05 commented on Feb 16, 2023

@andrews05

The reference is in dev dependencies though, which is only relevant to the package developers, not the package users (us). I just tried a fresh install, it works fine with guzzle 7. (The readme says to install guzzle 6, but there's no reason why you can't install guzzle 7 instead)

ainsleyclark

ainsleyclark commented on Feb 16, 2023

@ainsleyclark

It's probably an edge case but I guess we've still encountered it during development.

andrews05

andrews05 commented on Feb 16, 2023

@andrews05

Yeah, I admit I've had difficulty upgrading from guzzle 6 to 7 when v6 is currently installed. But this isn't really a fault of the sparkpost package - changing the dev dependency to v7 wouldn't resolve that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

      Development

      No branches or pull requests

        Participants

        @AlexanderBech@andrews05@phpfui@ainsleyclark

        Issue actions

          Guzzle 6 · Issue #207 · SparkPost/php-sparkpost