[Java Client] HTTP errors during WebSocket handshake not surfaced as HttpRequestExceptions
#47,597 opened on Apr 6, 2023
Repository metrics
- Stars
- (37,933 stars)
- PR merge metrics
- (Avg merge 16d 9h) (258 merged PRs in 30d)
Description
Is there an existing issue for this?
- I have searched the existing issues
Describe the bug
If an HTTP response for the opening SignalR negotiate call contains an unexpected response code, the SignalR Java client library will convert the error to an HttpRequestException for use by consumers of the library. However, if negotiation is skipped and a similar error occurs during the opening WebSocket handshake, the library does not return a useful exception. Instead, OkHttpWebSocketWrapper.SignalRWebSocketListener.onFailure() ignores the Response returned by OkHttp and wraps the provided Throwable in a RuntimeException.
This makes certain handleable errors, like 401, especially difficult to detect. I believe the only recourse today is to parse the error message string on the underlying Throwable returned by OkHttp.
Expected Behavior
When negotiation is skipped, SignalR should return HttpRequestExceptions for the opening HTTP handshake in the WebSocket protocol, so we can handle the error codes (like 401) appropriately.
Steps To Reproduce
- Set up a basic SignalR Java client and SignalR service that supports authentication.
- Configure the SignalR client to use WebSocket transport and skip negotiation.
- Provide a bad token for the connection.
Expected: SignalR Java client receives an HttpRequestException with a 401 code.
Actual: SignalR Java client receives a generic RuntimeException.
Exceptions (if any)
No response
.NET Version
No response
Anything else?
No response