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

Argument 2 passed to Amp\\Artax\\ConnectionInfo::__construct() must b… #4

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

positive2407
Copy link

Argument 2 passed to Amp\Artax\ConnectionInfo::__construct() must be of the type string, null given.
In some cases cannon retrieve remote address, method $socket->getRemoteAddress() return null, but for class ConnectionInfo second argument must be a string only.

@kelunik
Copy link
Member

kelunik commented Sep 16, 2019

If this is the case, the connection probably broke already?

@positive2407
Copy link
Author

Yes, it's not happy path, after my fix exception is: "Amp/ByteStream/StreamException" with message:
"The stream was closed by the peer"

But it's better than type error :)

It's case like: I need fix error to see Error :)

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