API Error: /panel/api/inbounds/update returns "Success" but doesn't save changes to the DB

[AlexanderVechkasov]

Describe the bug

When attempting to update an inbound connection via the /panel/api/inbounds/update endpoint, the API returns a 200 OK response with an empty body, which signals a successful operation. However, the actual changes are not written to the x-ui.db database.

For example, when programmatically changing the expiryTime for a client, this value remains unchanged in the database and the web interface.

Environment Information:

• Operating System: Ubuntu 22.04.5 LTS
• Installation Method: Docker

Important Notes:

• Updating the same client via the web panel's interface works correctly and saves the data.
• Creating new clients via the API (/panel/api/inbounds/addClient) works correctly.
• The issue specifically occurs when trying to update an existing inbound connection via the API.

How to repeat the problem?

The issue can be reproduced using an external client (e.g., a Python script) that performs the following actions:

1. Authentication: Successfully authenticates via POST /login with valid credentials, receiving all necessary session cookies.
2. Read: Performs a GET /panel/api/inbounds/list to retrieve the current configuration of the desired inbound. The request is successful.
3. Modification: Programmatically changes the expiryTime value for one of the clients within the JSON string in the settings field.
4. Data Filtering and Preparation: A payload for the update request is formed. To avoid errors with extra fields, only the fields that are part of the inbound configuration (such as id, remark, port, settings, streamSettings, etc.) are included in the payload.
5. Write: A POST request is sent to /panel/api/inbounds/update with the prepared payload and all session cookies.

Expected action

The server should respond with 200 OK, and the new data (specifically, expiryTime) should be correctly saved to the database and reflected in the panel.

Received action

The server consistently responds to the request from step 5 with a 200 OK status and an empty response body. The external client's logs confirm a successful request/response cycle.

**Client Log Example:**INFO - UPDATE: Starting update for UUID 489f9b32-4b93-466c-af65-7916dcb3717d for 30 days.
INFO - API_REQUEST: Calling GET .../panel/api/inbounds/list
INFO - API_REQUEST: Response from .../panel/api/inbounds/list: Status 200
INFO - API_REQUEST: Calling POST .../panel/api/inbounds/update
INFO - API_REQUEST: Response from .../panel/api/inbounds/update: Status 200

Despite the 200 OK status, upon subsequent verification via the API or in the web interface, it is clear that the client's expiryTime value has not changed.

3x-ui Version

v2.6.0

Xray-core Version

25.5.16

Checklist

• This bug report is written entirely in English.
• This bug report is new and no one has reported it before me.

[kaivdev]

Я думаю где-то проблема в коде, либо какие-то данные не передаются, время передаешь в милисекундах? У меня например проблема когда заканчивается подписка и ставится флаг исчерпано, если продлить ее через запрос, все работает но горит красная точка, если я перезагружаю саму панель то у клиента просто пропадает интернет, но когда я вручную меняю любое поле у клиента, то эта точка начинает гореть зеленым, хотя когда я отправляю это же изменение также через запрос точка не меняет свой цвет :)

[AlexanderVechkasov]

Ты был абсолютно прав с самого начала — проблема оказалась очень глубокой и нетривиальной. Мы с коллегой провели полное расследование, и вот к чему мы пришли.

Проблема не в коде клиента, а в багах самого API панели 3X-UI (по крайней мере, на моей версии).

Мы перепробовали все возможные эндпоинты и методы:

POST /panel/api/inbounds/update: Этот эндпоинт принимает любые данные (идеально сформированные, полные, урезанные, все в строках), отвечает 200 OK, но молча не сохраняет изменения в базу данных.
POST /panel/inbound/updateClient/{uuid}: Этот эндпоинт, который использует веб-интерфейс, оказался еще более забагованным. Даже если ему отправить идеально чистый список клиентов без дубликатов, он возвращает ошибку Duplicate email. Мы с помощью логов доказали, что дубликат создается где-то внутри самого API в момент обработки.
Твоя проблема с "красной/зеленой точкой" — это, по сути, тот же самый баг, просто в более легкой форме. Панель выполняет часть операции (например, продлевает срок), но не доводит ее до конца (не обновляет кеш статуса), так же как у меня она не доводит до конца сохранение в БД. Ручное изменение через веб-интерфейс запускает другой, более полный и правильный код на сервере, который все исправляет.

Вывод: API обновления в старых версиях панели крайне нестабилен и сломан. Единственное надежное решение — обновлять саму панель 3X-UI до последней версии, либо не использовать эту функцию через API вообще.

[kaivdev]

Я провел вчера тоже небольшое исследование и выяснил что при обновление клиента в бд в таблице client_traffics не ставится enable в значение 1, и именно это значение отвечает за статус исчерпано, хотя в таблице inbounds в разделе settings сам конфиг включен

[kaivdev]

Я могу показать какие данные я отправляю обратно в панель, возможно что нибудь будет полезным, также все запросы я отправляю через aiohttp, кстати возможно я не отправляю данные up и down клиента и поэтому не происходит сброс в enable 1, но я сомневаюсь конечно

1. Основные настройки inbound (верхнего уровня)

{
"id": inbound_id, # ID самого inbound
"up": inbound_info.get('up', 0), # Входящий трафик
"down": inbound_info.get('down', 0), # Исходящий трафик
"total": inbound_info.get('total', 0), # Общий трафик
"remark": inbound_info.get('remark', ''), # Заметка для inbound
"enable": inbound_info.get('enable', True), # Статус активности inbound
"expiryTime": inbound_info.get('expiryTime', 0), # Время окончания
"listen": inbound_info.get('listen', ''), # Настройки прослушивания
"port": inbound_info.get('port', 0), # Порт
"protocol": inbound_info.get('protocol', ''), # Протокол
"streamSettings": inbound_info.get('streamSettings', ''), # Настройки потока
"sniffing": inbound_info.get('sniffing', ''), # Настройки сниффинга
"allocate": inbound_info.get('allocate', ''), # Настройки выделения
"tag": inbound_info.get('tag', '') # Тег
}
2. Настройки клиентов (внутри settings)
"settings": json.dumps({
"clients": clients, # Массив клиентов
"decryption": settings.get('decryption', 'none'), # Настройки дешифрования
"fallbacks": settings.get('fallbacks', []) # Настройки fallback

Где каждый клиент в массиве clients имеет свою структуру:

{
"id": client_id, # UUID клиента
"email": email, # Email клиента
"flow": "xtls-rprx-vision", # Тип потока
"enable": True, # Статус активности клиента
"expiryTime": expiry_time_ms, # Время окончания подписки
"totalGB": total_gb, # Лимит трафика
"tgId": user_id, # ID пользователя в Telegram
"subId": subscription_id, # ID подписки
"comment": comment, # Комментарий
"limitIp": 3 # Лимит IP-адресов

Таким образом, когда мы обновляем клиента, мы отправляем полную конфигурацию всего inbound, включая:
1.Настройки самого inbound (порт, протокол, и т.д.)
2.Настройки всех клиентов в этом inbound
3.Общие настройки для всех клиентов (decryption, fallbacks)

[AlexanderVechkasov]

Вот логика моего решения на Python:

Сначала я получаю полный объект подключения (inbound) через эндпоинт /panel/api/inbounds/list, чтобы работать с актуальной конфигурацией.
В памяти изменяю данные нужного клиента: нахожу его по UUID в списке clients и обновляю expiryTime.
Сериализую измененный объект settings обратно в JSON-строку.
Самый важный шаг: Создаю новый словарь для отправки, где все ключи и значения из полученного на шаге 1 объекта преобразованы в строковый тип. Это касается и id, и port, и settings (который становится строкой, содержащей JSON), и всех остальных полей.
Отправляю этот "остроковлённый" (stringified) payload на эндпоинт /panel/api/inbounds/update.
Ну и часть кода на которой я остановился:
import json
from datetime import datetime, timedelta
import httpx
import logging

Предполагается, что api_client - это настроенный httpx.AsyncClient,

а PANEL_URL и DEFAULT_INBOUND_ID определены.

logger = logging.getLogger(name)

async def update_client_expiry(client_uuid: str, days_to_add: int) -> tuple[bool, str]:
"""
Обновляет срок действия клиента с использованием обходного пути для бага API 3x-ui.
Эта версия ("все в строки") является наиболее надежной.
"""
logger.info(f"Начало обновления для UUID {client_uuid} на {days_to_add} дней.")

# 1. Получаем полный объект inbound
try:
    get_list_url = f"{PANEL_URL}/panel/api/inbounds/list"
    response_list = await api_client.get(get_list_url) # ваш api_request
    if not response_list: return False, "Не удалось получить список подключений от API."

    list_data = response_list.json()
    if not (list_data.get("success") and list_data.get("obj")):
        return False, "API вернул неуспешный ответ при получении списка."

    inbound_to_update = next((ib for ib in list_data["obj"] if ib.get("id") == DEFAULT_INBOUND_ID), None)
    if not inbound_to_update: return False, f"В списке не найдено подключение с ID {DEFAULT_INBOUND_ID}."

except Exception as e:
    logger.error(f"Ошибка на шаге 1 (Получение данных): {e}", exc_info=True)
    return False, "Ошибка при получении данных из панели."

# 2. Модифицируем settings и подготавливаем данные
try:
    payload_data = inbound_to_update.copy()
    settings_obj = json.loads(payload_data.get("settings", "{}"))
    clients_list = settings_obj.get("clients", [])

    new_expiry_dt = datetime.now()
    client_found = False
    for i, client in enumerate(clients_list):
        if client.get("id") == client_uuid:
            current_expiry_ms = client.get("expiryTime", 0)
            start_date = datetime.now()
            if current_expiry_ms > 0 and datetime.fromtimestamp(current_expiry_ms / 1000) > start_date:
                start_date = datetime.fromtimestamp(current_expiry_ms / 1000)
            
            new_expiry_dt = start_date + timedelta(days=days_to_add)
            clients_list[i]["expiryTime"] = int(new_expiry_dt.timestamp() * 1000)
            client_found = True
            break

    if not client_found: return False, "Клиент с указанным UUID не найден."

    payload_data['settings'] = json.dumps(settings_obj)
    if 'clientStats' in payload_data:
        del payload_data['clientStats']

    # --- КЛЮЧЕВОЕ РЕШЕНИЕ: ПРЕВРАЩАЕМ ВСЕ В СТРОКИ ---
    stringified_payload = {str(k): str(v) for k, v in payload_data.items()}
    logger.info(f"Финальный payload для отправки: {stringified_payload}")

except Exception as e:
    logger.error(f"Ошибка на шаге 2 (Подготовка данных): {e}", exc_info=True)
    return False, "Ошибка при подготовке данных."

# 3. Отправляем запрос на обновление
update_url = f"{PANEL_URL}/panel/api/inbounds/update"
response_update = await api_client.post(update_url, data=stringified_payload) # ваш api_request

if not response_update: return False, "Не удалось выполнить запрос на обновление к API."

logger.info(f"Ответ от {update_url}: Статус {response_update.status_code}, Тело: '{response_update.text}'")
# ... дальнейшая обработка ответа ...
# (ваш код для обработки ответа тут полный и правильный)
if response_update.status_code == 200:
    # и так далее
    return True, new_expiry_dt.strftime("%Y-%m-%d %H:%M") # Упрощенный пример
else:
    return False, "Ошибка"

Проблема все равно осталась(((((

[kaivdev]

Я посмотрел твой разбор и код — ты всё делаешь логично, но есть важный нюанс, из-за которого обновление через API 3x-ui может не работать.

В чём проблема?
API 3x-ui очень чувствителен к формату данных.
Если отправлять payload, где все значения превращены в строки (как у тебя), или использовать data=... вместо json=..., то сервер возвращает 200 OK, но ничего не сохраняет.
Веб-интерфейс панели всегда отправляет JSON с “нативными” типами (int, bool, str), а не строками.

1.Формируй payload как обычный словарь Python:
Все значения должны быть “нативных” типов: числа — int, булевы — bool, строки — str.
Только поле settings должно быть строкой с сериализованным JSON.
payload = { "id": inbound_id, # int! "up": inbound_info.get('up', 0), # int "down": inbound_info.get('down', 0), # int "total": inbound_info.get('total', 0), # int "remark": inbound_info.get('remark', ''), # str "enable": inbound_info.get('enable', True), # bool "expiryTime": inbound_info.get('expiryTime', 0), # int "listen": inbound_info.get('listen', ''), # str "port": inbound_info.get('port', 0), # int "protocol": inbound_info.get('protocol', ''), # str "settings": json.dumps(settings_obj), # str (JSON) "streamSettings": inbound_info.get('streamSettings', ''), # str "sniffing": inbound_info.get('sniffing', ''), # str "allocate": inbound_info.get('allocate', ''), # str "tag": inbound_info.get('tag', '') # str }

2. Отправляй запрос именно так:
   response = await api_client.post(update_url, json=payload)

Не используй data=... и не превращай всё в строки!
Удаляй только реально лишние поля (например, clientStats), но не убирай нужные даже если они пусты
Если отправить всё строками или через form-data (data=...), панель “молча” игнорирует изменения, хотя возвращает 200 OK.
Через веб-интерфейс всё работает, потому что он отправляет корректный JSON

Если не поможет
Сравни payload, который отправляет веб-интерфейс (через DevTools), и свой.
Попробуй отправить такой же JSON через Postman — если работает, значит дело в типах/структуре.

Вот пример рабочего кода из нашего бота, который корректно обновляет клиента через API, не считая ошибки исчерпано

`# Получаем актуальный inbound
inbound_info = await api.get_inbound_details(inbound_id)
if not inbound_info:
logger.error(f"Не удалось получить информацию о inbound {inbound_id}")
return False

Парсим settings

settings = json.loads(inbound_info.get('settings', '{}'))
clients = settings.get('clients', [])

Ищем нужного клиента и обновляем expiryTime

client_found = None
for client in clients:
if client.get('id') == client_uuid:
client_found = client
client['expiryTime'] = new_expiry_time_ms # новое время окончания
break

if not client_found:
logger.error(f"Клиент с UUID {client_uuid} не найден в inbound {inbound_id}")
return False

Формируем payload для обновления inbound

data = {
"id": inbound_id,
"up": inbound_info.get('up', 0),
"down": inbound_info.get('down', 0),
"total": inbound_info.get('total', 0),
"remark": inbound_info.get('remark', ''),
"enable": inbound_info.get('enable', True),
"expiryTime": inbound_info.get('expiryTime', 0),
"listen": inbound_info.get('listen', ''),
"port": inbound_info.get('port', 0),
"protocol": inbound_info.get('protocol', ''),
"settings": json.dumps(settings), # settings — это строка с JSON!
"streamSettings": inbound_info.get('streamSettings', ''),
"sniffing": inbound_info.get('sniffing', ''),
"allocate": inbound_info.get('allocate', ''),
"tag": inbound_info.get('tag', '')
}

Отправляем запрос на обновление (именно через json=...)

response = await api_client.post(
f"{PANEL_URL}/panel/api/inbounds/update/{inbound_id}",
json=data
)

if response.status_code == 200:
logger.info("Обновление прошло успешно")
else:
logger.error(f"Ошибка при обновлении: {response.status_code}, {response.text}")`

Все поля payload — “нативные” типы (int, bool, str), только settings — строка с JSON.
Используется json=data, а не data=....

[AlexanderVechkasov]

Спасибо Вам большое!!! За направление в нужную сторону) Эту функцию починил))))

[kaivdev]

Напиши если возникнет ошибка такая же как у меня с тегом исчерпано, будем решать что с этим сделать можно будет.
для воспроизведения у клиента должно закончиться время, и клиент должен быть отключен, дальше через апи обновляем время и включаем клиента обратно, просто интересно это реальный баг или что-то у меня в коде

[AlexanderVechkasov]

Давай проверим, щас сделаю клиента чтобы завтра истёк и посмотрю что будет. Будет ли возникать эта ошибка как у вас.

[AlexanderVechkasov]

Насколько я тебя правильно понял, у меня ключ должен истечь я его должен продлить как пользователь при этом трафик у меня должен перестать идти до того момента пока ты в админке не вкл/выкл меня. Как бы, только применимо ко мне соответственно чтобы воспроизвести твою ошибку)
Правильно понимаю?
Учти пожалуйста что я не настолько профи(

[kaivdev]

Для воспроизведения нужно чтобы у клиента закончилось время и он загорелся красным, можно даже в панели сразу сделать заведомо истекший клиент, дальше как у меня это происходит, каждые 12 часов идет синхронизация с панелью, и если у клиента закончилось время, приходит уведомление что подписка отключена, и ползунок enable c true переключается на false, дальше если продлить подписку любым способом, промокод, реферальная акция или просто через покупку дни начисляются, ползунок enable становится уже true, сбрасывается трафик но сам индикатор горит красным. Продление идет по способу который я прислал тебе выше, по сути я оставляю все поля, просто меняю expiretime, делаю клиента enable true, и сбрасываю его трафик. У меня есть идея конечно как это исправить, например удалять клиента, и формировать нового со старыми данными, но это конечно костыль уже получается, мне важно понять это баг или нет

[kaivdev]

Вот что в логах например

[AlexanderVechkasov]

Суть вообщем : у юзара все истекло трафик перестал идти при этом ползунок не переключился в режим выкл (скорее всего я на этот ползунок не обращал внимания и не реализовал в коде взаимодействие с ним просто перекрыл трафик).Затем продлил клиента через бота, трафик пошел.