-
Notifications
You must be signed in to change notification settings - Fork 821
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Require guzzlehttp/psr7 ^2 #10253
Comments
guzzlehttp/guzzle is not required by embed/embed v4 guzzlehttp/psr7 ^2 is supported by embed/embed v4, though not a composer required. A few other psr7 implementation are supported by embed/embed b4 https://github.com/oscarotero/Embed/blob/master/src%2FHttp%2FFactoryDiscovery.php#L15 silverstripe/assets requires intervention/image https://github.com/silverstripe/silverstripe-assets/blob/1/composer.json#L26 intervention/image requires guzzlehttp/psr7 ~1.1 || ^2.0 https://github.com/Intervention/image/blob/master/composer.json#L17 guzzlehttp/guzzle ^6, and 7.0.0 < 7.3.0 will require guzzlehttp/psr7 ^1.6.1 https://github.com/guzzle/guzzle/blob/6.5/composer.json#L28 guzzlehttp/guzzle ^7.3.0 requires guzzlehttp/psr7 ^1.8.1 || ^2.1 https://github.com/guzzle/guzzle/blob/master/composer.json#L57 |
I think we need to update |
I merged what I could. But I would like to get this in an installable state so I can make sure ckan and webauth work with the newer version of Guzzle. |
The parts that were blocking the embed upgrade have been resolved. |
All done. |
The upgrade to embed/embed v4 uncovered the need to only allow recent versions of guzzlehttp/guzzle
ACs
References
PRs
The text was updated successfully, but these errors were encountered: