Skip to content
This repository has been archived by the owner on Aug 8, 2024. It is now read-only.

Guzzle problem #23

Closed
gmatkowski opened this issue Oct 12, 2020 · 15 comments
Closed

Guzzle problem #23

gmatkowski opened this issue Oct 12, 2020 · 15 comments

Comments

@gmatkowski
Copy link

Problem 1
- Installation request for matthewbdaly/laravel-azure-storage ^1.4 -> satisfiable by matthewbdaly/laravel-azure-storage[1.4.0].
- Can only install one of: guzzlehttp/guzzle[6.5.x-dev, 7.2.0].
- Can only install one of: guzzlehttp/guzzle[7.2.0, 6.5.x-dev].
- Can only install one of: guzzlehttp/guzzle[6.5.x-dev, 7.2.0].
- Conclusion: install guzzlehttp/guzzle 6.5.x-dev
- Installation request for guzzlehttp/guzzle (locked at 7.2.0, required as ^7.0.1) -> satisfiable by guzzlehttp/guzzle[7.2.0].

So it looks like it's not supported for latest Guzzle 7.2.0 version

@matthewbdaly
Copy link
Owner

Looks like a duplicate of #16.

@gmatkowski
Copy link
Author

Looks like a duplicate of #16.

yes, You are right sorry for duplicating ...

@matthewbdaly
Copy link
Owner

I will keep an eye on the upstream package and update this package once a suitable version becomes available.

@matthewbdaly
Copy link
Owner

Going to leave this open to prevent more duplication and indicate it's not been forgotten.

@matthewbdaly matthewbdaly reopened this Oct 25, 2020
@brpetrov
Copy link

Hi there, Sorry for my question, but as Laravel 8.0 doesn't specifically require guzzlehttp 7.0, would downgrading to 6.0 fix the issue?

@matthewbdaly
Copy link
Owner

matthewbdaly commented Nov 17, 2020

@brpetrov This package doesn't itself depend on Guzzle. It's just a Laravel wrapper for Flysystem's Azure integration, which does depend on Guzzle.

Downgrading Laravel is an option, but not a great one TBH.

@brpetrov
Copy link

@matthewbdaly Thanks for responding. How would you recommend going about it without downgrading? I don't see what else..

@matthewbdaly
Copy link
Owner

Yeah, I don't think there's anything much I can do about it until Azure/azure-storage-common upgrades the Guzzle version they use. There's a ticket open for this, but until that's sorted there's not much that can be done.

I'd really prefer that package vendors used HTTPlug rather than relying explicitly on Guzzle for that reason since that way it's dependent on an interface rather than a specific package, but it is what it is.

@matthewbdaly
Copy link
Owner

Looks like the November release of the Azure storage PHP SDK will include support for this - see Azure/azure-storage-php#253 for details. Once that's done it should be possible to update this package accordingly.

@MrThePlague
Copy link

Looks like the November release of the Azure storage PHP SDK will include support for this - see Azure/azure-storage-php#253 for details. Once that's done it should be possible to update this package accordingly.

Looks like they just merged these changes!

@matthewbdaly
Copy link
Owner

Still waiting on a new release before I can do anything about this...

@WESquad-oldmember
Copy link

Found a temporary solution on this repo: squigg/azure-queue-laravel

Laravel 8 has moved to Guzzle 7.x, but the upstream dependency microsoft/azure-storage-queue from this package still uses Guzzle 6. This will cause composer to fail during dependency resolution.

Tests so far have not identified any impacting breaking changes between Guzzle 6 and 7, so while we wait for the upstream package to be updated, you can work around this issue by adding/updating your root composer.json file to use an inline alias for guzzlehttp/guzzle:

"guzzlehttp/guzzle": "7.0.1 as 6.5.5"

Or run this command:

composer require guzzlehttp/guzzle:"7.0.1 as 6.5.5" 

@Oxicode
Copy link

Oxicode commented Dec 9, 2020

Vuelve intentar, vi que ya hicieron la actualización por acá
Azure/azure-storage-php@5dc7592

@matthewbdaly
Copy link
Owner

OK, so there's a new version of the upstream Azure storage PHP SDK, but it looks like we're still waiting for it to be pushed to Packagist. I've made a new release that updates the version dependency on league/flysystem-azure-blob-storage and I think this will work once it goes live, but will check.

@matthewbdaly
Copy link
Owner

Looks like this is now resolved. The package microsoft/azure-storage-blob now shows as having released version 1.5.2 on Packagist, and adding this package to a new Laravel 8 project works OK. The tests pass too, so it should be OK.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

6 participants