Shameless(negotiate)Plug - plugging in sending negotiate request when st...

...arting a connection

Bonus changes:
- renaming some variables for consistency
- removing trailing whitespaces
I could add a test but I am not sure if it adds any value. Currently I know that  the initial state is `disconnect`, `change_state` logs and that the only way to change state is by calling `change_state`. As a result if the state is `connected` it had to be logged. Given that a set up of a test that connects is kind of complicated (the process requires being able to handle 3 requests (negotiate, connect, start) across 2 protocols (HTTP, websockets) and the return value is small I decided not to do this. Once we have a kind of functional/manual tests it should be easy to notice if there is anything wrong with logging transitions.