You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
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?
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.
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)
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.
Activity
ainsleyclark commentedon Nov 7, 2022
Any updates on this?
ainsleyclark commentedon Nov 7, 2022
This seems to work:
composer require php-http/guzzle7-adapterThen:
use Http\Adapter\Guzzle7\Client as GuzzleAdapter;phpfui commentedon Nov 7, 2022
@ainsleyclark Thanks for this. You just saved me hours! And just in time as well.
But the package should be updated.
ainsleyclark commentedon Nov 7, 2022
Glad I could help, yeah would be good if this is updated to 7
andrews05 commentedon Feb 16, 2023
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 commentedon Feb 16, 2023
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 commentedon Feb 16, 2023
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 commentedon Feb 16, 2023
It's probably an edge case but I guess we've still encountered it during development.
andrews05 commentedon Feb 16, 2023
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.