Fut rights

CHANGELOG.md

@@ -4,6 +4,10 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [1.2.31] - 2025-01-09
+### Updated
+- [Binance] handle futures trading rights
+
 ## [1.2.30] - 2024-12-02
 ### Updated
 - [Binance] handle sandbox

README.md

@@ -1,4 +1,4 @@
-# trading-backend [1.2.30](https://github.com/Drakkar-Software/trading-backend/tree/master/CHANGELOG.md)
+# trading-backend [1.2.31](https://github.com/Drakkar-Software/trading-backend/tree/master/CHANGELOG.md)
 [![PyPI](https://img.shields.io/pypi/v/trading-backend.svg)](https://pypi.python.org/pypi/trading-backend/)
 [![Downloads](https://pepy.tech/badge/trading-backend/month)](https://pepy.tech/project/trading-backend)
 [![Github-Action-CI](https://github.com/Drakkar-Software/trading-backend/workflows/trading-backend-CI/badge.svg)](https://github.com/Drakkar-Software/trading-backend/actions)

setup.py

@@ -17,7 +17,7 @@
 from setuptools import find_packages
 from setuptools import setup
 
-VERSION = "1.2.30"
+VERSION = "1.2.31"
 PROJECT_NAME = "trading-backend"
 
 PACKAGES = find_packages(exclude=["tests"])

tests/exchanges/test_binance.py

@@ -46,10 +46,11 @@ async def test_get_api_key_rights(binance_exchange):
     exchange = exchanges.Binance(binance_exchange)
     with mock.patch.object(
         exchange._exchange.connector.client, "sapi_get_account_apirestrictions",
-        mock.AsyncMock(return_value={"enableReading": True, "enableSpotAndMarginTrading": False, "enableWithdrawals": False})
+        mock.AsyncMock(return_value={"enableReading": True, "enableSpotAndMarginTrading": False, "enableFutures": True, "enableWithdrawals": False})
     ) as sapi_get_account_apirestrictions_mock:
         assert await exchange._get_api_key_rights() == [
-            trading_backend.enums.APIKeyRights.READING
+            trading_backend.enums.APIKeyRights.READING,
+            trading_backend.enums.APIKeyRights.FUTURES_TRADING,
         ]
         sapi_get_account_apirestrictions_mock.assert_awaited_once()
     with mock.patch.object(
@@ -60,7 +61,6 @@ async def test_get_api_key_rights(binance_exchange):
             trading_backend.enums.APIKeyRights.READING,
             trading_backend.enums.APIKeyRights.SPOT_TRADING,
             trading_backend.enums.APIKeyRights.MARGIN_TRADING,
-            trading_backend.enums.APIKeyRights.FUTURES_TRADING,
             trading_backend.enums.APIKeyRights.WITHDRAWALS
         ]
         sapi_get_account_apirestrictions_mock.assert_awaited_once()

trading_backend/exchanges/binance.py

@@ -97,6 +97,7 @@ async def _get_api_key_rights(self) -> list[trading_backend.enums.APIKeyRights]:
         if restrictions.get('enableSpotAndMarginTrading'):
             rights.append(trading_backend.enums.APIKeyRights.SPOT_TRADING)
             rights.append(trading_backend.enums.APIKeyRights.MARGIN_TRADING)
+        if restrictions.get('enableFutures'):
             rights.append(trading_backend.enums.APIKeyRights.FUTURES_TRADING)
         if restrictions.get('enableWithdrawals'):
             rights.append(trading_backend.enums.APIKeyRights.WITHDRAWALS)

trading_backend/exchanges/bingx.py

@@ -31,7 +31,7 @@ def get_name(cls):
     async def _inner_cancel_order(self):
         # use client api to avoid any ccxt call wrapping and error handling
         try:
-            await self._exchange.connector.client.cancel_order("12345", symbol="BTC/USDT")
+            await self._exchange.connector.client.cancel_order("12345", symbol=self._get_symbol())
         except ccxt.ExchangeError as err:
             # ('bingx {"code":100413,"msg":"Incorrect apiKey","timestamp":1718551786654}',)
             if "Incorrect apiKey".lower() in str(err).lower():

trading_backend/exchanges/exchange.py

@@ -61,9 +61,14 @@ def get_orders_parameters(self, params=None) -> dict:
     def _allow_withdrawal_right(self) -> bool:
         return trading_backend.constants.ALLOW_WITHDRAWAL_KEYS
 
+    def _get_symbol(self):
+        if self._exchange.exchange_manager.is_future:
+            return "BTC/USDT:USDT"
+        return "BTC/USDT"
+
     async def _inner_cancel_order(self):
         # use client api to avoid any ccxt call wrapping and error handling
-        await self._exchange.connector.client.cancel_order("12345", symbol="BTC/USDT")
+        await self._exchange.connector.client.cancel_order("12345", symbol=self._get_symbol())
 
     async def _get_api_key_rights_using_order(self) -> list[trading_backend.enums.APIKeyRights]:
         rights = [trading_backend.enums.APIKeyRights.READING]