chore: auth method (closes: #10)

Author: valeratrades

v_exchanges/src/binance/mod.rs

@@ -1,11 +1,9 @@
 pub mod futures;
+use adapters::binance::BinanceOption;
 use color_eyre::eyre::Result;
 use derive_more::{Deref, DerefMut};
 use v_exchanges_adapters::{Client, binance};
-use v_utils::{
-	macros::WrapNew,
-	trades::{Asset, Pair, Timeframe},
-};
+use v_utils::trades::{Asset, Pair, Timeframe};
 
 use crate::core::{AssetBalance, Exchange, Klines, KlinesRequestRange};
 
@@ -14,6 +12,11 @@ pub struct Binance(pub Client);
 
 //? currently client ends up importing this from crate::binance, but could it be possible to lift the [Client] reexport up, and still have the ability to call all exchange methods right on it?
 impl Exchange for Binance {
+	fn auth<S: Into<String>>(&mut self, key: S, secret: S) {
+		self.update_default_option(BinanceOption::Key(key.into()));
+		self.update_default_option(BinanceOption::Secret(secret.into()));
+	}
+
 	async fn futures_klines(&self, symbol: Pair, tf: Timeframe, range: KlinesRequestRange) -> Result<Klines> {
 		futures::market::klines(&self.0, symbol, tf, range).await
 	}

v_exchanges/src/bybit/mod.rs

@@ -1,6 +1,7 @@
 mod account;
 mod market;
 
+use adapters::bybit::BybitOption;
 use color_eyre::eyre::Result;
 use derive_more::derive::{Deref, DerefMut};
 use v_exchanges_adapters::{Client, bybit};
@@ -13,6 +14,11 @@ pub struct Bybit(pub Client);
 
 //? currently client ends up importing this from crate::binance, but could it be possible to lift the [Client] reexport up, and still have the ability to call all exchange methods right on it?
 impl Exchange for Bybit {
+	fn auth<S: Into<String>>(&mut self, key: S, secret: S) {
+		self.update_default_option(BybitOption::Key(key.into()));
+		self.update_default_option(BybitOption::Secret(secret.into()));
+	}
+
 	async fn futures_klines(&self, symbol: Pair, tf: Timeframe, range: KlinesRequestRange) -> Result<Klines> {
 		market::klines(&self.0, symbol, tf, range).await
 	}

v_exchanges/src/core.rs

@@ -1,11 +1,13 @@
 use chrono::{DateTime, TimeDelta, Utc};
 use color_eyre::eyre::Result;
-use v_utils::trades::{Asset, Kline, Pair, Timeframe};
 use derive_more::{Deref, DerefMut};
+use v_utils::trades::{Asset, Kline, Pair, Timeframe};
 
 //TODO!!!!!!!!!!!!!: klines switch to defining the range via an Enum over either limit either start and end times
 
 pub trait Exchange {
+	fn auth<S: Into<String>>(&mut self, key: S, secret: S);
+
 	//? should I have Self::Pair too? Like to catch the non-existent ones immediately? Although this would increase the error surface on new listings.
 	fn futures_klines(&self, symbol: Pair, tf: Timeframe, range: KlinesRequestRange) -> impl std::future::Future<Output = Result<Klines>> + Send;
 	fn futures_price(&self, symbol: Pair) -> impl std::future::Future<Output = Result<f64>> + Send;
@@ -79,10 +81,7 @@ impl From<DateTime<Utc>> for KlinesRequestRange {
 impl From<TimeDelta> for KlinesRequestRange {
 	fn from(value: TimeDelta) -> Self {
 		let now = Utc::now();
-		KlinesRequestRange::StartEnd {
-			start: now - value,
-			end: None,
-		}
+		KlinesRequestRange::StartEnd { start: now - value, end: None }
 	}
 }
 impl From<u32> for KlinesRequestRange {