wisespace-io / binance-rs

Rust Library for the Binance API
Other
665 stars 297 forks source link

Fixing websocket DayTicker failing on unknown trade id's #27

Closed LucasSimpson closed 5 years ago

LucasSimpson commented 5 years ago

The first_trade_id and last_trade_id are declared as u64, but the binance API sometimes returns -1, causing a panic. This changes those types to i64 to accommodate the possible negative values.

wisespace-io commented 5 years ago

@LucasSimpson Thanks for the contribution