tree 4f0a84e4c1343ce10380783b071e8e49edd77e7b
parent 8ff5a8d87fa99f3e9bbc2b7d68f922fcc0fedd31
author Willy Tarreau <w@1wt.eu> 1503927961 +0200
committer Willy Tarreau <w@1wt.eu> 1504069444 +0200
encoding latin1

MEDIUM: connection: get rid of data->init() which was not for data

The ->init() callback of the connection's data layer was only used to
complete the session's initialisation since sessions and streams were
split apart in 1.6. The problem is that it creates a big confusion in
the layers' roles as the session has to register a dummy data layer
when waiting for a handshake to complete, then hand it off to the
stream which will replace it.

The real need is to notify that the transport has finished initializing.
This should enable a better splitting between these layers.

This patch thus introduces a connection-specific callback called
xprt_done_cb() which informs about handshake successes or failures. With
this, data->init() can disappear, CO_FL_INIT_DATA as well, and we don't
need to register a dummy data->wake() callback to be notified of errors.