[zenmoney] Некорректное сопоставление accounts при совместном доступе

[guard43ru]

Проблема

В аккаунте zenmoney подключены две учетные записи, у каждой учетной записи настроена синхронизация счетов через tbc-ge.
При этом последние цифры IBAN у счетов совпадают. Из-за этого счет привязывается к другому аккаунту (даже не спрашивает создать новый или выбрать нужный).

Пример двух счетов одной валюты (первый syncIds это IBAN, второй это номер карты), смотрел через yarn start tbc-ge

{ "id": "819363191","syncIds": [ "0007", "6551" ] }
{ "id": "817368119","syncIds": [ "0007", "0272" ] }

У нас есть две пары syncIds и они уникальны(!), но zenmoney ищет первое совпадение и цепляется по нему.
Как проверял:

1. При подключении синхронизации у второго пользователя, молча привязывает счета в правильной валюте, к счету где есть "0007" (то что второй Ids не совпадает, игнорируется), все транзакции смешиваются.
2. Если сделать счета первого пользователя личными, при подключении синхронизации у второго пользователя создает новые счета (спрашивает), все работает. Но это костыль и неудобно (в планировании).
3. Удаляю у счета "0007", "0272". Запускаю синхронизацию, счет привязывается к личному(!) счету второго пользователя, где есть "0007". При этом сам счет я в приложении не вижу, т.к. он личный.
4. Возвращаю как было, удаляю у счета "0007", остается "0272". Запускаю синхронизацию, счет привязывается к правильному счету, дописывает "0007".

Итог: ищет любое совпадение, игнорирует что другие syncIds не совпадают.

Решение

Достаточно проверять полное совпадение в syncIds, чтобы эта проблема исчезла.
Эта логика в приложении или бекенде, т.к. плагин при работе ничего про второго пользователя и его счета не знает.

@skvav техподдержка пишет:

Это не баг, т.к. в Дзен-мани счета идентифицируются по последним четырем цифрам. Если эти цифры совпадают у двух разных счетов, то они будут объединяться, увы.

Все же надеюсь что это баг, и исправление будет. Я понимаю что ситуация редкая (первый раз за 16 лет?), но вроде бы и фикс не сложный.

[guard43ru]

Обрезка ids происходит не в плагине, а дальше в коде (ensureSyncIDsAreUniqueButSanitized => trimSyncId)

плагин tbc-ge отдает номер счета полностью и последние цифры карты
[
{ id: "817368119", instrument: "GEL", syncIds: ["GE53TB729604506440951", "0272"] },
{ id: "817368120", instrument: "USD", syncIds: ["GE53TB729604506440951", "0272"] },
{ id: "817368121", instrument: "EUR", syncIds: ["GE53TB729604506440951", "0272"] }
]
далее src/common/adapters.js, переименовывает syncIds → syncID
[
{ id: "817368119", instrument: "GEL", syncID: ["GE53TB729604506440951", "0272"] },
{ id: "817368120", instrument: "USD", syncID: ["GE53TB729604506440951", "0272"] },
{ id: "817368121", instrument: "EUR", syncID: ["GE53TB729604506440951", "0272"] }
]
далее src/common/accounts.js, ensureSyncIDsAreUniqueButSanitized() применяет trimSyncId() <== вот тут обрезали, и это уже не плагин
[
{ id: "817368119", instrument: "GEL", syncID: ["0951", "0272"] },
{ id: "817368120", instrument: "USD", syncID: ["0951", "0272"] },
{ id: "817368121", instrument: "EUR", syncID: ["0951", "0272"] }
]
далее ZPAPI.js валидирует и передает в zenmoney

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

Сейчас плагины при каждой синхронизации добавляют недостающие ids (синхронизируют их, следят чтобы они совпадали). И все что нужно это поправить проверку, не одного любого вхождения, а полного совпадения.

А при не совпадении достаточно просто спросить пользователя это появился новый счёт или просто изменился старый (стандартная форма при появлении нового счёта)