Skip to content

Conversation

clue
Copy link
Contributor

@clue clue commented Oct 11, 2018

The server may reject an incoming connection during authentication or even before that during the initial connection handshake. The initial handshake error message uses a slightly different message format and also semantically happens before executing a command, so we now make sure to properly reject the following authentication command.

Note:
In case the server sent a ERR packet as first packet it will happen before the client and server negotiated any capabilities. Therefore the ERR packet will not contain the SQL-state.

https://dev.mysql.com/doc/dev/mysql-server/latest/page_protocol_connection_phase.html

Resolves / closes #81

@clue clue added the bug label Oct 11, 2018
@clue clue added this to the v0.4.1 milestone Oct 11, 2018
@jsor jsor merged commit ed7fe8b into friends-of-reactphp:master Oct 12, 2018
@clue clue deleted the too-many-connections branch October 12, 2018 08:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

'Too many connections' error is not properly handled
3 participants