"WebsocketStreamClientImpl - null" messages in the log

[lz-44]

From time to time, I can see the following non-descriptive messages in the log:
18:14:44.889 [OkHttp https://stream.bybit.com/...] ERROR c.b.a.c.w.i.WebsocketStreamClientImpl - null

Presumably, these messages occur after websocket session is terminated with error.
I also assume it may be logged from here:

public void onError(Throwable t) {
    LOGGER.error(t.getMessage());
}

(WebsocketStreamClientImpl.java:283)

Would be great at least to have some more descriptive and meaningful messages. Maybe also good idea to include smth like WS session id across logs for tracing purposes.

Thanks!

[TheLeBot]

@lz-44
Just introduce a pull request for that. Seems very reasonable. Should be easy to intergrate.

[lz-44]

@TheLeBot
done.
#79

[lz-44]

You may also want to check my fork, ws-onerror branch. I've added there a possibility to pass an error handling callback, which definitely adds more versatility.
I decided not to do a PR for that since it may worth to add more callbacks for each type of ws messages, but I needed this quick enhancement, so I left it without PR for now. If I finished it completely I will do PR, if Victor consider it worthful.