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

Deprecated dependecy with guzzle #385

Closed
fbecker-complex opened this issue Jul 27, 2021 · 3 comments
Closed

Deprecated dependecy with guzzle #385

fbecker-complex opened this issue Jul 27, 2021 · 3 comments
Labels

Comments

@fbecker-complex
Copy link

Bug Report

Q A
BC Break no
Version 1.7.0

Summary

Since I updated https://github.com/guzzle/guzzle to v7.3.0 and https://github.com/guzzle/psr7 to v2.0.0, I get the following error message on soap-requests:

'Error' with message 'Call to undefined function GuzzleHttp\Psr7\stream_for()' (0)
> in .../vendor/php-http/message/src/StreamFactory/GuzzleStreamFactory.php:21

The given class is deprecated:
image

The function used to set a StreamFactory, StreamFactoryDiscovery::find() in HttPlugHandle.php(l.63), is also deprecated:
image

Solution

I fixed it for my project by limiting guzzlehttp/psr7 to v1.7.0, but that's not a permanent solution

Question

Would you consider exchanging the deprecated methods, so we can remove the fixed version workaround?

@veewee
Copy link
Contributor

veewee commented Jul 27, 2021

This looks like an issue in one of the httplug packages rather than this package..
Did you tried upgrading php-http/message as well?

@fbecker-complex
Copy link
Author

fbecker-complex commented Jul 28, 2021

This looks like an issue in one of the httplug packages rather than this package..
Did you tried upgrading php-http/message as well?

Thanks for the quick reply!
I tried including all php-http/* dependencies directly, but still got the same result.
I'm gonna investigate a little further tomorrow and tell you about my findings.

@fbecker-complex
Copy link
Author

This looks like an issue in one of the httplug packages rather than this package..
Did you tried upgrading php-http/message as well?

You were right, the error was in the php-http/message. A colleague of mine already developed a fix, which will be merged shortly: php-http/message#141

I am going to close this issue.

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

No branches or pull requests

2 participants