Skip to content

Conversation

@clue
Copy link
Member

@clue clue commented Nov 15, 2014

It MUST NOT be used in a client side context.

This has caused several issues already, so we should update our documentation accordingly.

The React\Socket namespace is part of the react/socket package which is a "Library for building an evented socket server". Hence the React\Socket\Connection should be used for an incoming connection and not for an outgoing one.

Refs #4, reactphp/stream#11 and reactphp/dns#11.

It MUST NOT be used in a client side context
cboden added a commit that referenced this pull request Dec 4, 2014
Connection is a server side incoming connection
@cboden cboden merged commit 2949cf3 into reactphp:master Dec 4, 2014
@clue clue deleted the patch-1 branch December 4, 2014 14:49
@cboden cboden modified the milestone: v0.4.3 Mar 1, 2016
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