Should retry set to true by default

[georgi-mirchev]

Scarlet/scarlet-protocol-websocket-okhttp/src/main/java/com/tinder/scarlet/websocket/okhttp/OkHttpWebSocketChannel.kt

Line 110 in a476420

listener.onFailed(this@OkHttpWebSocketChannel, true, t)

What is this? This InnerWebSocketListener() is not injected but rather created in the OkHttpWebSocketChannel.open() function. This leads to the behavior where shouldRetry is always set to true but there are cases where this should be false. Like a 404 response for an example. Why do you need to retry when the URL cannot be found? This needs to be done in a configurable manner meaning InnerWebSocketListener() should be injected and not created inside the function.