Futures API support?

[0ihsan]

See implementation in https://github.com/wisespace-io/binance-rs/tree/master/src/futures

[Igosuki]

Sure, I'm not currently using futures but it shouldn't be hard to add.

[Igosuki]

Here : #4
You can use it but I haven't tested every endpoint for now.

[0ihsan]

Thank you. But, I am getting runtime panic for this code

use binance::api::*;
use binance::futures::general::*;
use futures::executor::block_on;

async fn get_all_symbols() {

    let general: FuturesGeneral = Binance::new(None, None);

    match general.exchange_info().await {
        Ok(answer) => println!("Exchange information: {:?}", answer),
        Err(e) => println!("Error: {:?}", e),
    }

}

fn main() {

    block_on(get_all_symbols());

}

thread 'main' panicked at 'there is no reactor running, must be called from the context of a Tokio 1.x
runtime', ~/.cargo/registry/src/github.com-1ecc6299db9ec823/tokio-1.7.0/src/runtime/blocking/pool.rs:84:33
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

[0ihsan]

I solved that by adding tokio = { version = "1", features = ["full"] } to dependencies and making these changes

use binance::api::*;
use binance::futures::general::*;
use futures::executor::block_on;
+use tokio;

async fn get_all_symbols() {

    let general: FuturesGeneral = Binance::new(None, None);

    match general.exchange_info().await {
        Ok(answer) => println!("Exchange information: {:?}", answer),
        Err(e) => println!("Error: {:?}", e),
    }

}

+#[tokio::main]
+async fn async_main() {
+
+    block_on(get_all_symbols());
+
+}

fn main() {

-    block_on(get_all_symbols());
+    async_main();

}

[Igosuki]

Didn't see that everything started requiring tokio, I updated the examples accordingly.
The examples are passing so I see no reason to not merge into master. Let me know if you need anymore patches.