Skip to content
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

composer Install guzzlehttp/guzzle 6.5.8 #143

Closed
wants to merge 1 commit into from
Closed

composer Install guzzlehttp/guzzle 6.5.8 #143

wants to merge 1 commit into from

Conversation

abhisekmazumdar
Copy link

@abhisekmazumdar abhisekmazumdar commented Sep 18, 2023

While installing the Drupal module odata_client in the latest version of Drupal 9, we encountered the following issue mentioned here.

So, this patch is a temporary fix until the maintainer fixes it or everyone upgrades to the latest version of Drupal 10.

More info: https://www.drupal.org/project/odata_client/issues/3388005

@n8rowley
Copy link
Contributor

n8rowley commented Sep 18, 2023

I'm worried this is going to cause issues with apps using Laravel 9 or 10. Guzzle ^7.2 has been required since Laravel 9.

PR #84 already aims to be more flexible with Guzzle versions. Is that a better way to fix this issue?

@abhisekmazumdar
Copy link
Author

Thanks, @n8rowley, for pointing out the existing PR. I tested the #84 PR, and it works with Drupal 9.x, the latest version.
I had to rebase the PR, but it works. Closing this as #84 covers what I needed.

Leaving the rebased version here for anyone who is working with D9.x might need it:

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

Successfully merging this pull request may close these issues.

2 participants