search

alpacahq / alpaca-trade-api-go

Go client for Alpaca's trade API
Apache License 2.0
325 stars 93 forks source link

Broken Stream Client Connection #300

Open nmoniz opened 2 months ago

nmoniz commented 2 months ago

I was trying out the SDK and I found an issue while testing the crypto stream and the data stream examples. The issue may be with the actual stream.CryptoClient / stream.StocksClient and how it tries to connect.

When I called the Connect(...) method I got the following error:

datav2stream: irrecoverable error: websocket dial: failed to WebSocket dial: expected handshake response status code 101 but got 404

The key to fixing it was in the log line immediately before:

datav2stream: connecting to wss://paper-api.alpaca.markets/stream/us, attempt 1/20 ...

The /us shouldn't be there.

I was able to get around this error by creating the clients with marketdata.Feed("") for the first param. This way I got a log:

datav2stream: established connection
gnvk commented 1 month ago

Hi! paper-api.alpaca.markets is definitely the wrong URL both for stream.CryptoClient and for stream.StocksClient. If you provide a minimal reproducible example I can help you fix it.