fix(pally): use InvId instead of custom field for order ID lookup

api/routes/pally.py

@@ -34,7 +34,7 @@ async def pally_callback(
     bot: Bot = Depends(get_bot),
     deps: APIDependenciesDTO = Depends(get_deps),
 ):
-    oid = custom or ""
+    oid = InvId
     raw_string = f"{OutSum}:{InvId}:{settings.pally_token}"
     expected_signature = hashlib.md5(raw_string.encode("utf-8")).hexdigest().upper()
 
@@ -49,7 +49,7 @@ async def pally_callback(
     logger.info("Received successfully paid bill %s", TrsId)
 
     if not oid.isdigit():
-        logger.critical("Invalid custom field format for TrsId %s", TrsId)
+        logger.critical("Invalid InvId format for TrsId %s", TrsId)
         return "OK"
 
     bill = await deps.bills_repository.get_bill_by_id(session, int(oid))

bot/texts.py

@@ -174,15 +174,17 @@ REFERALS_MENU = (
 # ОФОРМЛЕНИЕ ПОДПИСКИ
 # ============================================================
 
-SUBSCRIPTION_DEVICE_SELECTOR = f"<b>{MALENIA_QUESTION} Сколько устройств вы хотите подключить?</b>\n\nЦена: <b>{{price}}₽</b>"
-
-SUBSCRIPTION_DURATION_SELECTOR = (
-    f"<b>{MALENIA_QUESTION} На какой срок хотите оформить подписку?</b>\n\nИтого: <b>{{price}}₽</b> <i>(скидка: {{discount}}%)</i>"
+SUBSCRIPTION_DEVICE_SELECTOR = (
+    f"<b>{MALENIA_QUESTION} Сколько устройств вы хотите подключить?</b>\n\nЦена: <b>{{price}}₽</b>"
 )
 
+SUBSCRIPTION_DURATION_SELECTOR = f"<b>{MALENIA_QUESTION} На какой срок хотите оформить подписку?</b>\n\nИтого: <b>{{price}}₽</b> <i>(скидка: {{discount}}%)</i>"
+
 WHITELISTS_ALREADY_ON = "Белые списки уже активны."
 
-CHECKOUT_PAYMENT_CHOICE = f"<b>{MALENIA_BANK} Пополнение баланса:</b>\n\n<i>❓ Выберите способ оплаты:</i>"
+CHECKOUT_PAYMENT_CHOICE = (
+    f"<b>{MALENIA_BANK} Пополнение баланса:</b>\n\n<i>❓ Выберите способ оплаты:</i>"
+)
 
 
 # ============================================================
@@ -203,13 +205,13 @@ BILL_CREATED = _BILL_TEMPLATE + _DIVIDER + _BILL_LINK
 
 INSUFFICIENT_FUNDS = f"<b>{MALENIA_CROSS} Недостаточно средств для проведения операции.</b>\n\n<b>💰 Текущий баланс:</b> <code>{{balance}}₽</code>\n<b>💸 Требуется:</b> <code>{{amount}}₽</code>\n\n<i>{MALENIA_CARD} Пополните баланс на <code>{{diff}}₽</code></i>"
 
-SUCCESSFULLY_CREATED_SUBSCRIPTION = (
-    f"<b>{MALENIA_HEART} Ваша подписка успешно оплачена!</b> Ссылка на добавления доступна в главном меню."
-)
+SUCCESSFULLY_CREATED_SUBSCRIPTION = f"<b>{MALENIA_HEART} Ваша подписка успешно оплачена!</b> Ссылка на добавления доступна в главном меню."
 
 DEPOSIT_AMOUNT = f"<b>{MALENIA_COINS} Укажите сумму пополнения:</b>"
 
-DEPOSIT_AMOUNT_FALLBACK = f"<b>{MALENIA_CROSS} Сумма пополнения указана неверно, повторите попытку.</b>"
+DEPOSIT_AMOUNT_FALLBACK = (
+    f"<b>{MALENIA_CROSS} Сумма пополнения указана неверно, повторите попытку.</b>"
+)
 
 DEPOSIT_AMOUNT_BELOW_MINIMAL = (
     f"<b>{MALENIA_CROSS} Сумма пополнения ниже минимальной - <code>{{threshold}}₽</code>.</b>"

tests/send_pally_webhook.py

@@ -21,6 +21,7 @@ load_dotenv()
 
 ENDPOINT = "/pally/result"
 
+
 def calculate_signature(
     out_sum: str,
     inv_id: str,
@@ -46,25 +47,21 @@ def main():
 
     pally_token = os.environ["PALLY_TOKEN"]
 
-    # Эти поля могут быть любыми строками — вебхук их не использует содержательно,
-    # только включает в подпись. Главное чтобы совпадали с тем, что пойдёт в подпись.
-    payment_id = "test_payment_001"
-    bill_id = f"test_bill_{args.bill_id}"
+    inv_id = str(args.bill_id)
+    out_sum = str(args.amount)
+    trs_id = f"test_trs_{args.bill_id}"
     currency = "RUB"
-    order_id = str(args.bill_id)  # order_id == id счёта в нашей БД
 
-    signature = calculate_signature(
-        payment_id, args.status, str(args.amount), currency, order_id, pally_token
-    )
+    signature = calculate_signature(out_sum, inv_id, pally_token)
 
     payload = {
-        "id": payment_id,
-        "bill_id": bill_id,
-        "status": args.status,
-        "req_amount": str(args.amount),
-        "currency": currency,
-        "order_id": order_id,
-        "signature": signature,
+        "InvId": inv_id,
+        "OutSum": out_sum,
+        "Commission": "0",
+        "TrsId": trs_id,
+        "Status": args.status,
+        "CurrencyIn": currency,
+        "SignatureValue": signature,
     }
 
     print(f"{args.host}{ENDPOINT}")