Skip to content

Commit a20ba8f

Browse files
committed
Consolidate throwing exception for disconnected websockets
1 parent f80a429 commit a20ba8f

3 files changed

Lines changed: 4 additions & 3 deletions

File tree

README.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -162,6 +162,7 @@ alpacaAPI.broker().trading()
162162
.side(OrderSide.SELL)
163163
.timeInForce(TimeInForce.GTC)
164164
.type(OrderType.MARKET));
165+
165166
// Wait to be filled
166167
Thread.sleep(2000);
167168

src/main/java/net/jacobpeterson/alpaca/websocket/AlpacaWebsocket.java

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -209,6 +209,9 @@ protected void cleanupState() {
209209
* @param message the message
210210
*/
211211
protected void sendWebsocketMessage(String message) {
212+
if (!isConnected()) {
213+
throw new IllegalStateException("This websocket must be connected before send a message!");
214+
}
212215
LOGGER.trace("Websocket message sent: {}", message);
213216
websocket.send(message);
214217
}

src/main/java/net/jacobpeterson/alpaca/websocket/updates/UpdatesWebsocket.java

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -153,9 +153,6 @@ public void subscribeToTradeUpdates(boolean subscribe) {
153153
}
154154

155155
private void sendTradeUpdatesListenMessage() {
156-
if (!isConnected()) {
157-
throw new IllegalStateException("This websocket must be connected before subscribing to streams!");
158-
}
159156
final JsonObject requestObject = new JsonObject();
160157
requestObject.addProperty("action", "listen");
161158
final JsonArray streamsArray = new JsonArray();

0 commit comments

Comments
 (0)