ccxt 4.4.49 → 4.4.53

package/js/src/bitfinex1.js

@@ -404,7 +404,7 @@ export default class bitfinex1 extends Exchange {
     }
     /**
      * @method
-     * @name bitfinex#fetchTransactionFees
+     * @name bitfinex1#fetchTransactionFees
      * @deprecated
      * @description please use fetchDepositWithdrawFees instead
      * @see https://docs.bitfinex.com/v1/reference/rest-auth-fees
@@ -441,7 +441,7 @@ export default class bitfinex1 extends Exchange {
     }
     /**
      * @method
-     * @name bitfinex#fetchDepositWithdrawFees
+     * @name bitfinex1#fetchDepositWithdrawFees
      * @description fetch deposit and withdraw fees
      * @see https://docs.bitfinex.com/v1/reference/rest-auth-fees
      * @param {string[]|undefined} codes list of unified currency codes
@@ -481,7 +481,7 @@ export default class bitfinex1 extends Exchange {
     }
     /**
      * @method
-     * @name bitfinex#fetchTradingFees
+     * @name bitfinex1#fetchTradingFees
      * @description fetch the trading fees for multiple markets
      * @see https://docs.bitfinex.com/v1/reference/rest-auth-summary
      * @param {object} [params] extra parameters specific to the exchange API endpoint
@@ -564,7 +564,7 @@ export default class bitfinex1 extends Exchange {
     }
     /**
      * @method
-     * @name bitfinex#fetchMarkets
+     * @name bitfinex1#fetchMarkets
      * @description retrieves data on all markets for bitfinex
      * @see https://docs.bitfinex.com/v1/reference/rest-public-symbols
      * @see https://docs.bitfinex.com/v1/reference/rest-public-symbol-details
@@ -691,7 +691,7 @@ export default class bitfinex1 extends Exchange {
     }
     /**
      * @method
-     * @name bitfinex#fetchBalance
+     * @name bitfinex1#fetchBalance
      * @description query for balance and get the amount of funds available for trading or funds locked in orders
      * @see https://docs.bitfinex.com/v1/reference/rest-auth-wallet-balances
      * @param {object} [params] extra parameters specific to the exchange API endpoint
@@ -749,7 +749,7 @@ export default class bitfinex1 extends Exchange {
     }
     /**
      * @method
-     * @name bitfinex#transfer
+     * @name bitfinex1#transfer
      * @description transfer currency internally between wallets on the same account
      * @see https://docs.bitfinex.com/v1/reference/rest-auth-transfer-between-wallets
      * @param {string} code unified currency code
@@ -835,7 +835,7 @@ export default class bitfinex1 extends Exchange {
     }
     /**
      * @method
-     * @name bitfinex#fetchOrderBook
+     * @name bitfinex1#fetchOrderBook
      * @description fetches information on open orders with bid (buy) and ask (sell) prices, volumes and other data
      * @see https://docs.bitfinex.com/v1/reference/rest-public-orderbook
      * @param {string} symbol unified symbol of the market to fetch the order book for
@@ -858,7 +858,7 @@ export default class bitfinex1 extends Exchange {
     }
     /**
      * @method
-     * @name bitfinex#fetchTickers
+     * @name bitfinex1#fetchTickers
      * @description fetches price tickers for multiple markets, statistical information calculated over the past 24 hours for each market
      * @param {string[]} [symbols] unified symbols of the markets to fetch the ticker for, all market tickers are returned if not assigned
      * @param {object} [params] extra parameters specific to the exchange API endpoint
@@ -878,7 +878,7 @@ export default class bitfinex1 extends Exchange {
     }
     /**
      * @method
-     * @name bitfinex#fetchTicker
+     * @name bitfinex1#fetchTicker
      * @description fetches a price ticker, a statistical calculation with the information calculated over the past 24 hours for a specific market
      * @see https://docs.bitfinex.com/v1/reference/rest-public-ticker
      * @param {string} symbol unified symbol of the market to fetch the ticker for
@@ -1019,7 +1019,7 @@ export default class bitfinex1 extends Exchange {
     }
     /**
      * @method
-     * @name bitfinex#fetchTrades
+     * @name bitfinex1#fetchTrades
      * @description get the list of most recent trades for a particular symbol
      * @see https://docs.bitfinex.com/v1/reference/rest-public-trades
      * @param {string} symbol unified symbol of the market to fetch trades for
@@ -1055,7 +1055,7 @@ export default class bitfinex1 extends Exchange {
     }
     /**
      * @method
-     * @name bitfinex#fetchMyTrades
+     * @name bitfinex1#fetchMyTrades
      * @description fetch all trades made by the user
      * @see https://docs.bitfinex.com/v1/reference/rest-auth-past-trades
      * @param {string} symbol unified market symbol
@@ -1084,7 +1084,7 @@ export default class bitfinex1 extends Exchange {
     }
     /**
      * @method
-     * @name bitfinex#createOrder
+     * @name bitfinex1#createOrder
      * @description create a trade order
      * @see https://docs.bitfinex.com/v1/reference/rest-auth-new-order
      * @param {string} symbol unified symbol of the market to create an order in
@@ -1152,7 +1152,7 @@ export default class bitfinex1 extends Exchange {
     }
     /**
      * @method
-     * @name bitfinex#cancelOrder
+     * @name bitfinex1#cancelOrder
      * @description cancels an open order
      * @see https://docs.bitfinex.com/v1/reference/rest-auth-cancel-order
      * @param {string} id order id
@@ -1195,7 +1195,7 @@ export default class bitfinex1 extends Exchange {
     }
     /**
      * @method
-     * @name bitfinex#cancelAllOrders
+     * @name bitfinex1#cancelAllOrders
      * @description cancel all open orders
      * @see https://docs.bitfinex.com/v1/reference/rest-auth-cancel-all-orders
      * @param {string} symbol not used by bitfinex cancelAllOrders
@@ -1287,7 +1287,7 @@ export default class bitfinex1 extends Exchange {
     }
     /**
      * @method
-     * @name bitfinex#fetchOpenOrders
+     * @name bitfinex1#fetchOpenOrders
      * @description fetch all unfilled currently open orders
      * @see https://docs.bitfinex.com/v1/reference/rest-auth-active-orders
      * @param {string} symbol unified market symbol
@@ -1312,7 +1312,7 @@ export default class bitfinex1 extends Exchange {
     }
     /**
      * @method
-     * @name bitfinex#fetchClosedOrders
+     * @name bitfinex1#fetchClosedOrders
      * @description fetches information on multiple closed orders made by the user
      * @see https://docs.bitfinex.com/v1/reference/rest-auth-orders-history
      * @param {string} symbol unified market symbol of the market orders were made in
@@ -1338,7 +1338,7 @@ export default class bitfinex1 extends Exchange {
     }
     /**
      * @method
-     * @name bitfinex#fetchOrder
+     * @name bitfinex1#fetchOrder
      * @description fetches information on an order made by the user
      * @see https://docs.bitfinex.com/v1/reference/rest-auth-order-status
      * @param {string} id the order id
@@ -1434,7 +1434,7 @@ export default class bitfinex1 extends Exchange {
     }
     /**
      * @method
-     * @name bitfinex#createDepositAddress
+     * @name bitfinex1#createDepositAddress
      * @description create a currency deposit address
      * @see https://docs.bitfinex.com/v1/reference/rest-auth-deposit
      * @param {string} code unified currency code of the currency for the deposit address
@@ -1450,7 +1450,7 @@ export default class bitfinex1 extends Exchange {
     }
     /**
      * @method
-     * @name bitfinex#fetchDepositAddress
+     * @name bitfinex1#fetchDepositAddress
      * @description fetch the deposit address for a currency associated with this account
      * @see https://docs.bitfinex.com/v1/reference/rest-auth-deposit
      * @param {string} code unified currency code
@@ -1484,7 +1484,7 @@ export default class bitfinex1 extends Exchange {
     }
     /**
      * @method
-     * @name bitfinex#fetchDepositsWithdrawals
+     * @name bitfinex1#fetchDepositsWithdrawals
      * @description fetch history of deposits and withdrawals
      * @see https://docs.bitfinex.com/v1/reference/rest-auth-deposit-withdrawal-history
      * @param {string} code unified currency code for the currency of the deposit/withdrawals
@@ -1621,7 +1621,7 @@ export default class bitfinex1 extends Exchange {
     }
     /**
      * @method
-     * @name bitfinex#withdraw
+     * @name bitfinex1#withdraw
      * @description make a withdrawal
      * @see https://docs.bitfinex.com/v1/reference/rest-auth-withdrawal
      * @param {string} code unified currency code
@@ -1672,7 +1672,7 @@ export default class bitfinex1 extends Exchange {
     }
     /**
      * @method
-     * @name bitfinex#fetchPositions
+     * @name bitfinex1#fetchPositions
      * @description fetch all open positions
      * @see https://docs.bitfinex.com/v1/reference/rest-auth-active-positions
      * @param {string[]|undefined} symbols list of unified market symbols

package/js/src/blofin.d.ts

@@ -299,7 +299,7 @@ export default class blofin extends Exchange {
     fetchPosition(symbol: string, params?: {}): Promise<Position>;
     /**
      * @method
-     * @name blofin#fetchPosition
+     * @name blofin#fetchPositions
      * @description fetch data on a single open contract trade position
      * @see https://blofin.com/docs#get-positions
      * @param {string[]} [symbols] list of unified market symbols

package/js/src/blofin.js

@@ -160,6 +160,9 @@ export default class blofin extends Exchange {
                 'api': {
                     'rest': 'https://openapi.blofin.com',
                 },
+                'test': {
+                    'rest': 'https://demo-trading-openapi.blofin.com',
+                },
                 'referral': {
                     'url': 'https://blofin.com/register?referral_code=f79EsS',
                     'discount': 0.05,
@@ -289,7 +292,7 @@ export default class blofin extends Exchange {
                         'trailing': false,
                     },
                     'fetchOHLCV': {
-                        'max': 1440,
+                        'limit': 1440,
                     },
                 },
                 'spot': {
@@ -317,6 +320,7 @@ export default class blofin extends Exchange {
                         'attachedStopLossTakeProfit': {
                             'triggerPriceType': undefined,
                             'limit': true,
+                            'price': undefined,
                         },
                         'hedged': true,
                     },
@@ -1958,7 +1962,7 @@ export default class blofin extends Exchange {
     }
     /**
      * @method
-     * @name blofin#fetchPosition
+     * @name blofin#fetchPositions
      * @description fetch data on a single open contract trade position
      * @see https://blofin.com/docs#get-positions
      * @param {string[]} [symbols] list of unified market symbols

package/js/src/bybit.d.ts

@@ -344,7 +344,7 @@ export default class bybit extends Exchange {
     fetchOrderClassic(id: string, symbol?: Str, params?: {}): Promise<Order>;
     /**
      * @method
-     * @name bybit#fetchOrderClassic
+     * @name bybit#fetchOrder
      * @description  *classic accounts only/ spot not supported*  fetches information on an order made by the user *classic accounts only*
      * @see https://bybit-exchange.github.io/docs/v5/order/order-list
      * @param {string} id the order id
@@ -357,7 +357,7 @@ export default class bybit extends Exchange {
     fetchOrders(symbol?: Str, since?: Int, limit?: Int, params?: {}): Promise<Order[]>;
     /**
      * @method
-     * @name bybit#fetchOrders
+     * @name bybit#fetchOrdersClassic
      * @description fetches information on multiple orders made by the user *classic accounts only*
      * @see https://bybit-exchange.github.io/docs/v5/order/order-list
      * @param {string} symbol unified market symbol of the market orders were made in

package/js/src/bybit.js

@@ -4769,7 +4769,7 @@ export default class bybit extends Exchange {
     }
     /**
      * @method
-     * @name bybit#fetchOrderClassic
+     * @name bybit#fetchOrder
      * @description  *classic accounts only/ spot not supported*  fetches information on an order made by the user *classic accounts only*
      * @see https://bybit-exchange.github.io/docs/v5/order/order-list
      * @param {string} id the order id
@@ -4890,7 +4890,7 @@ export default class bybit extends Exchange {
     }
     /**
      * @method
-     * @name bybit#fetchOrders
+     * @name bybit#fetchOrdersClassic
      * @description fetches information on multiple orders made by the user *classic accounts only*
      * @see https://bybit-exchange.github.io/docs/v5/order/order-list
      * @param {string} symbol unified market symbol of the market orders were made in

package/js/src/coinex.js

@@ -470,7 +470,7 @@ export default class coinex extends Exchange {
                     'ERC20': 'ERC20',
                     'BRC20': 'BRC20',
                     'SOL': 'SOL',
-                    'TON': 'SOL',
+                    'TON': 'TON',
                     'BSV': 'BSV',
                     'AVAXC': 'AVA_C',
                     'AVAXX': 'AVA',
@@ -3931,7 +3931,7 @@ export default class coinex extends Exchange {
             'currency': this.safeCurrencyCode(undefined, currency),
             'network': undefined,
             'address': address,
-            'tag': tag,
+            'tag': this.safeString(depositAddress, 'memo', tag),
         };
     }
     /**
@@ -4805,14 +4805,14 @@ export default class coinex extends Exchange {
         this.checkAddress(address);
         await this.loadMarkets();
         const currency = this.currency(code);
-        if (tag) {
-            address = address + ':' + tag;
-        }
         const request = {
             'ccy': currency['id'],
             'to_address': address,
             'amount': this.numberToString(amount), // the actual amount without fees, https://www.coinex.com/fees
         };
+        if (tag !== undefined) {
+            request['memo'] = tag;
+        }
         let networkCode = undefined;
         [networkCode, params] = this.handleNetworkCodeAndParams(params);
         if (networkCode !== undefined) {

package/js/src/coinmetro.d.ts

@@ -166,7 +166,7 @@ export default class coinmetro extends Exchange {
     cancelOrder(id: string, symbol?: Str, params?: {}): Promise<Order>;
     /**
      * @method
-     * @name coinmetro#cancelOrder
+     * @name coinmetro#closePosition
      * @description closes an open position
      * @see https://documenter.getpostman.com/view/3653795/SVfWN6KS#47f913fb-8cab-49f4-bc78-d980e6ced316
      * @param {string} symbol not used by coinmetro closePosition ()

package/js/src/coinmetro.js

@@ -212,7 +212,7 @@ export default class coinmetro extends Exchange {
             // exchange-specific options
             'options': {
                 'currenciesByIdForParseMarket': undefined,
-                'currencyIdsListForParseMarket': undefined,
+                'currencyIdsListForParseMarket': ['QRDO'],
             },
             'features': {
                 'spot': {
@@ -401,7 +401,12 @@ export default class coinmetro extends Exchange {
         if (this.safeValue(this.options, 'currenciesByIdForParseMarket') === undefined) {
             const currenciesById = this.indexBy(result, 'id');
             this.options['currenciesByIdForParseMarket'] = currenciesById;
-            this.options['currencyIdsListForParseMarket'] = Object.keys(currenciesById);
+            const currentCurrencyIdsList = this.safeList(this.options, 'currencyIdsListForParseMarket', []);
+            const currencyIdsList = Object.keys(currenciesById);
+            for (let i = 0; i < currencyIdsList.length; i++) {
+                currentCurrencyIdsList.push(currencyIdsList[i]);
+            }
+            this.options['currencyIdsListForParseMarket'] = currentCurrencyIdsList;
         }
         return result;
     }
@@ -502,10 +507,22 @@ export default class coinmetro extends Exchange {
         let baseId = undefined;
         let quoteId = undefined;
         const currencyIds = this.safeValue(this.options, 'currencyIdsListForParseMarket', []);
+        // Bubble sort by length (longest first)
+        const currencyIdsLength = currencyIds.length;
+        for (let i = 0; i < currencyIdsLength; i++) {
+            for (let j = 0; j < currencyIdsLength - i - 1; j++) {
+                const a = currencyIds[j];
+                const b = currencyIds[j + 1];
+                if (a.length < b.length) {
+                    currencyIds[j] = b;
+                    currencyIds[j + 1] = a;
+                }
+            }
+        }
         for (let i = 0; i < currencyIds.length; i++) {
             const currencyId = currencyIds[i];
             const entryIndex = marketId.indexOf(currencyId);
-            if (entryIndex !== -1) {
+            if (entryIndex === 0) {
                 const restId = marketId.replace(currencyId, '');
                 if (this.inArray(restId, currencyIds)) {
                     if (entryIndex === 0) {
@@ -1432,7 +1449,7 @@ export default class coinmetro extends Exchange {
     }
     /**
      * @method
-     * @name coinmetro#cancelOrder
+     * @name coinmetro#closePosition
      * @description closes an open position
      * @see https://documenter.getpostman.com/view/3653795/SVfWN6KS#47f913fb-8cab-49f4-bc78-d980e6ced316
      * @param {string} symbol not used by coinmetro closePosition ()

package/js/src/defx.d.ts

@@ -91,7 +91,7 @@ export default class defx extends Exchange {
     fetchTrades(symbol: string, since?: Int, limit?: Int, params?: {}): Promise<Trade[]>;
     /**
      * @method
-     * @name defx#fetchTrades
+     * @name defx#fetchMyTrades
      * @description fetch all trades made by the user
      * @see https://api-docs.defx.com/#06b5b33c-2fc6-48de-896c-fc316f5871a7
      * @param {string} symbol unified symbol of the market to fetch trades for

package/js/src/defx.js

@@ -940,7 +940,7 @@ export default class defx extends Exchange {
     }
     /**
      * @method
-     * @name defx#fetchTrades
+     * @name defx#fetchMyTrades
      * @description fetch all trades made by the user
      * @see https://api-docs.defx.com/#06b5b33c-2fc6-48de-896c-fc316f5871a7
      * @param {string} symbol unified symbol of the market to fetch trades for

package/js/src/deribit.d.ts

@@ -443,7 +443,7 @@ export default class deribit extends Exchange {
      * @param {int} [since] the earliest time in ms to fetch funding rate history for
      * @param {int} [limit] the maximum number of entries to retrieve
      * @param {object} [params] extra parameters specific to the exchange API endpoint
-     * @param {int} [params.end_timestamp] fetch funding rate ending at this timestamp
+     * @param {int} [params.until] fetch funding rate ending at this timestamp
      * @param {boolean} [params.paginate] default false, when true will automatically paginate by calling this endpoint multiple times. See in the docs all the [availble parameters](https://github.com/ccxt/ccxt/wiki/Manual#pagination-params)
      * @returns {object} a [funding rate structure]{@link https://docs.ccxt.com/#/?id=funding-rate-structure}
      */

package/js/src/deribit.js

@@ -3164,7 +3164,7 @@ export default class deribit extends Exchange {
      * @param {int} [since] the earliest time in ms to fetch funding rate history for
      * @param {int} [limit] the maximum number of entries to retrieve
      * @param {object} [params] extra parameters specific to the exchange API endpoint
-     * @param {int} [params.end_timestamp] fetch funding rate ending at this timestamp
+     * @param {int} [params.until] fetch funding rate ending at this timestamp
      * @param {boolean} [params.paginate] default false, when true will automatically paginate by calling this endpoint multiple times. See in the docs all the [availble parameters](https://github.com/ccxt/ccxt/wiki/Manual#pagination-params)
      * @returns {object} a [funding rate structure]{@link https://docs.ccxt.com/#/?id=funding-rate-structure}
      */
@@ -3173,19 +3173,38 @@ export default class deribit extends Exchange {
         const market = this.market(symbol);
         let paginate = false;
         [paginate, params] = this.handleOptionAndParams(params, 'fetchFundingRateHistory', 'paginate');
+        const maxEntriesPerRequest = 744; // seems exchange returns max 744 items per request
+        const eachItemDuration = '1h';
         if (paginate) {
-            return await this.fetchPaginatedCallDeterministic('fetchFundingRateHistory', symbol, since, limit, '8h', params, 720);
+            // fix for: https://github.com/ccxt/ccxt/issues/25040
+            return await this.fetchPaginatedCallDeterministic('fetchFundingRateHistory', symbol, since, limit, eachItemDuration, this.extend(params, { 'isDeribitPaginationCall': true }), maxEntriesPerRequest);
         }
-        const time = this.milliseconds();
+        const duration = this.parseTimeframe(eachItemDuration) * 1000;
+        let time = this.milliseconds();
         const month = 30 * 24 * 60 * 60 * 1000;
         if (since === undefined) {
             since = time - month;
         }
+        else {
+            time = since + month;
+        }
         const request = {
             'instrument_name': market['id'],
             'start_timestamp': since - 1,
-            'end_timestamp': time,
         };
+        const until = this.safeInteger2(params, 'until', 'end_timestamp');
+        if (until !== undefined) {
+            params = this.omit(params, ['until']);
+            request['end_timestamp'] = until;
+        }
+        else {
+            request['end_timestamp'] = time;
+        }
+        if ('isDeribitPaginationCall' in params) {
+            params = this.omit(params, 'isDeribitPaginationCall');
+            const maxUntil = this.sum(since, limit * duration);
+            request['end_timestamp'] = Math.min(request['end_timestamp'], maxUntil);
+        }
         const response = await this.publicGetGetFundingRateHistory(this.extend(request, params));
         //
         //    {

package/js/src/ellipx.d.ts

@@ -219,6 +219,7 @@ export default class ellipx extends Exchange {
     fetchTradingFee(symbol?: string, params?: {}): Promise<TradingFeeInterface>;
     /**
      * @method
+     * @name ellipx#withdraw
      * @description Make a withdrawal request
      * @see https://docs.google.com/document/d/1ZXzTQYffKE_EglTaKptxGQERRnunuLHEMmar7VC9syM/edit?tab=t.0#heading=h.zegupoa8g4t9
      * @param {string} code Currency code

package/js/src/ellipx.js

@@ -1712,6 +1712,7 @@ export default class ellipx extends Exchange {
     }
     /**
      * @method
+     * @name ellipx#withdraw
      * @description Make a withdrawal request
      * @see https://docs.google.com/document/d/1ZXzTQYffKE_EglTaKptxGQERRnunuLHEMmar7VC9syM/edit?tab=t.0#heading=h.zegupoa8g4t9
      * @param {string} code Currency code

package/js/src/gate.d.ts

@@ -734,7 +734,7 @@ export default class gate extends Exchange {
     }>;
     /**
      * @method
-     * @name gate#borrowMargin
+     * @name gate#borrowIsolatedMargin
      * @description create a loan to borrow margin
      * @see https://www.gate.io/docs/developers/apiv4/en/#marginuni
      * @param {string} symbol unified market symbol, required for isolated margin

package/js/src/gate.js

@@ -1824,7 +1824,7 @@ export default class gate extends Exchange {
             const active = listed && tradeEnabled && withdrawEnabled && depositEnabled;
             if (this.safeValue(result, code) === undefined) {
                 result[code] = {
-                    'id': code.toLowerCase(),
+                    'id': currency,
                     'code': code,
                     'info': undefined,
                     'name': undefined,
@@ -6456,7 +6456,7 @@ export default class gate extends Exchange {
     }
     /**
      * @method
-     * @name gate#borrowMargin
+     * @name gate#borrowIsolatedMargin
      * @description create a loan to borrow margin
      * @see https://www.gate.io/docs/developers/apiv4/en/#marginuni
      * @param {string} symbol unified market symbol, required for isolated margin

package/js/src/hollaex.d.ts

@@ -93,13 +93,14 @@ export default class hollaex extends Exchange {
     /**
      * @method
      * @name hollaex#fetchOHLCV
-     * @description fetches historical candlestick data containing the open, high, low, and close price, and the volume of a market
+     * @description hollaex has large gaps between candles, so it's recommended to specify since
      * @see https://apidocs.hollaex.com/#chart
      * @param {string} symbol unified symbol of the market to fetch OHLCV data for
      * @param {string} timeframe the length of time each candle represents
      * @param {int} [since] timestamp in ms of the earliest candle to fetch
      * @param {int} [limit] the maximum amount of candles to fetch
      * @param {object} [params] extra parameters specific to the exchange API endpoint
+     * @param {int} [params.until] timestamp in ms of the latest candle to fetch
      * @returns {int[][]} A list of candles ordered as timestamp, open, high, low, close, volume
      */
     fetchOHLCV(symbol: string, timeframe?: string, since?: Int, limit?: Int, params?: {}): Promise<OHLCV[]>;

package/js/src/hollaex.js

@@ -831,13 +831,14 @@ export default class hollaex extends Exchange {
     /**
      * @method
      * @name hollaex#fetchOHLCV
-     * @description fetches historical candlestick data containing the open, high, low, and close price, and the volume of a market
+     * @description hollaex has large gaps between candles, so it's recommended to specify since
      * @see https://apidocs.hollaex.com/#chart
      * @param {string} symbol unified symbol of the market to fetch OHLCV data for
      * @param {string} timeframe the length of time each candle represents
      * @param {int} [since] timestamp in ms of the earliest candle to fetch
      * @param {int} [limit] the maximum amount of candles to fetch
      * @param {object} [params] extra parameters specific to the exchange API endpoint
+     * @param {int} [params.until] timestamp in ms of the latest candle to fetch
      * @returns {int[][]} A list of candles ordered as timestamp, open, high, low, close, volume
      */
     async fetchOHLCV(symbol, timeframe = '1m', since = undefined, limit = undefined, params = {}) {
@@ -847,27 +848,20 @@ export default class hollaex extends Exchange {
             'symbol': market['id'],
             'resolution': this.safeString(this.timeframes, timeframe, timeframe),
         };
-        const duration = this.parseTimeframe(timeframe);
-        if (since === undefined) {
-            if (limit === undefined) {
-                limit = 1000; // they have no defaults and can actually provide tens of thousands of bars in one request, but we should cap "default" at generous amount
-            }
-            const end = this.seconds();
-            const start = end - duration * limit;
-            request['to'] = end;
-            request['from'] = start;
+        const until = this.safeInteger(params, 'until');
+        let end = this.seconds();
+        if (until !== undefined) {
+            end = this.parseToInt(until / 1000);
+        }
+        const defaultSpan = 2592000; // 30 days
+        if (since !== undefined) {
+            request['from'] = this.parseToInt(since / 1000);
         }
         else {
-            if (limit === undefined) {
-                request['from'] = this.parseToInt(since / 1000);
-                request['to'] = this.seconds();
-            }
-            else {
-                const start = this.parseToInt(since / 1000);
-                request['from'] = start;
-                request['to'] = this.sum(start, duration * limit);
-            }
+            request['from'] = end - defaultSpan;
         }
+        request['to'] = end;
+        params = this.omit(params, 'until');
         const response = await this.publicGetChart(this.extend(request, params));
         //
         //     [
@@ -1959,6 +1953,7 @@ export default class hollaex extends Exchange {
         return { 'url': url, 'method': method, 'body': body, 'headers': headers };
     }
     handleErrors(code, reason, url, method, headers, body, response, requestHeaders, requestBody) {
+        // { "message": "Invalid token" }
         if (response === undefined) {
             return undefined;
         }
@@ -1966,7 +1961,7 @@ export default class hollaex extends Exchange {
             //
             //  { "message": "Invalid token" }
             //
-            // different errors return the same code eg:
+            // different errors return the same code eg
             //
             //  { "message":"Error 1001 - Order rejected. Order could not be submitted as this order was set to a post only order." }
             //

package/js/src/htx.d.ts

@@ -1,7 +1,7 @@
 import Exchange from './abstract/htx.js';
 import type { TransferEntry, Int, OrderSide, OrderType, Order, OHLCV, Trade, FundingRateHistory, Balances, Str, Dict, Transaction, Ticker, OrderBook, Tickers, OrderRequest, Strings, Market, Currency, Num, Account, TradingFeeInterface, Currencies, IsolatedBorrowRates, IsolatedBorrowRate, LeverageTiers, LeverageTier, int, LedgerEntry, FundingRate, FundingRates, DepositAddress, BorrowInterest, OpenInterests } from './base/types.js';
 /**
- * @class huobi
+ * @class htx
  * @augments Exchange
  */
 export default class htx extends Exchange {

package/js/src/htx.js

@@ -12,7 +12,7 @@ import { TICK_SIZE, TRUNCATE } from './base/functions/number.js';
 import { sha256 } from './static_dependencies/noble-hashes/sha256.js';
 //  ---------------------------------------------------------------------------
 /**
- * @class huobi
+ * @class htx
  * @augments Exchange
  */
 export default class htx extends Exchange {
@@ -7560,11 +7560,13 @@ export default class htx extends Exchange {
                     'AccessKeyId': this.apiKey,
                     'Timestamp': timestamp,
                 };
+                // sorting needs such flow exactly, before urlencoding (more at: https://github.com/ccxt/ccxt/issues/24930 )
+                request = this.keysort(request);
                 if (method !== 'POST') {
-                    request = this.extend(request, query);
+                    const sortedQuery = this.keysort(query);
+                    request = this.extend(request, sortedQuery);
                 }
-                request = this.keysort(request);
-                let auth = this.urlencode(request);
+                let auth = this.urlencode(request).replace('%2c', '%2C'); // in c# it manually needs to be uppercased
                 // unfortunately, PHP demands double quotes for the escaped newline symbol
                 const payload = [method, hostname, url, auth].join("\n"); // eslint-disable-line quotes
                 const signature = this.hmac(this.encode(payload), this.encode(this.secret), sha256, 'base64');