Skip to content

Commit 364536c

Browse files
committed
improved WebSocket client error reporting
1 parent a00b202 commit 364536c

1 file changed

Lines changed: 9 additions & 2 deletions

File tree

‎Net/src/WebSocket.cpp‎

Lines changed: 9 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
//
22
// WebSocket.cpp
33
//
4-
// $Id: //poco/1.4/Net/src/WebSocket.cpp#6 $
4+
// $Id: //poco/1.4/Net/src/WebSocket.cpp#7 $
55
//
66
// Library: Net
77
// Package: WebSocket
@@ -202,7 +202,14 @@ WebSocketImpl* WebSocket::connect(HTTPClientSession& cs, HTTPRequest& request, H
202202
throw WebSocketException("Not authorized", WS_ERR_UNAUTHORIZED);
203203
}
204204
}
205-
throw WebSocketException("Cannot upgrade to WebSocket connection", response.getReason(), WS_ERR_NO_HANDSHAKE);
205+
if (response.getStatus() == HTTPResponse::HTTP_OK)
206+
{
207+
throw WebSocketException("The server does not understand the WebSocket protocol", WS_ERR_NO_HANDSHAKE);
208+
}
209+
else
210+
{
211+
throw WebSocketException("Cannot upgrade to WebSocket connection", response.getReason(), WS_ERR_NO_HANDSHAKE);
212+
}
206213
}
207214

208215

0 commit comments

Comments
 (0)