diff --git a/freqtrade/rpc/api_server/api_v1.py b/freqtrade/rpc/api_server/api_v1.py index ccbcbb94f..2354c4bf8 100644 --- a/freqtrade/rpc/api_server/api_v1.py +++ b/freqtrade/rpc/api_server/api_v1.py @@ -45,7 +45,8 @@ logger = logging.getLogger(__name__) # 2.25: Add several profit values to /status endpoint # 2.26: increase /balance output # 2.27: Add /trades//reload endpoint -API_VERSION = 2.27 +# 2.28: Switch reload endpoint to Post +API_VERSION = 2.28 # Public API, requires no auth. router_public = APIRouter() @@ -133,7 +134,7 @@ def trade_cancel_open_order(tradeid: int, rpc: RPC = Depends(get_rpc)): return rpc._rpc_trade_status([tradeid])[0] -@router.get('/trades/{tradeid}/reload', response_model=OpenTradeSchema, tags=['trading']) +@router.post('/trades/{tradeid}/reload', response_model=OpenTradeSchema, tags=['trading']) def trade_reload(tradeid: int, rpc: RPC = Depends(get_rpc)): rpc._rpc_reload_trade_from_exchange(tradeid) return rpc._rpc_trade_status([tradeid])[0] diff --git a/tests/rpc/test_rpc_apiserver.py b/tests/rpc/test_rpc_apiserver.py index 44d70e318..1e76ce557 100644 --- a/tests/rpc/test_rpc_apiserver.py +++ b/tests/rpc/test_rpc_apiserver.py @@ -755,7 +755,7 @@ def test_api_trade_reload_trade(botclient, mocker, fee, markets, ticker, is_shor cancel_stoploss_order=stoploss_mock, ) - rc = client_get(client, f"{BASE_URI}/trades/10/reload") + rc = client_post(client, f"{BASE_URI}/trades/10/reload") assert_response(rc, 502) assert 'Could not find trade with id 10.' in rc.json()['error'] assert ftbot.handle_onexchange_order.call_count == 0 @@ -763,7 +763,7 @@ def test_api_trade_reload_trade(botclient, mocker, fee, markets, ticker, is_shor create_mock_trades(fee, is_short=is_short) Trade.commit() - rc = client_get(client, f"{BASE_URI}/trades/5/reload") + rc = client_post(client, f"{BASE_URI}/trades/5/reload") assert ftbot.handle_onexchange_order.call_count == 1