Неправильный логин или пароль при регистрации внешней БД

[gsbelarus]

Если при регистрации внешней БД указать неверный логин или пароль, то система повисит-повисит и вернет список баз данных в исходное состояние при этом никаких сообщений не будет ни на экране, ни в логе. А должно сказать что-то понятное пользователю.

[gsbelarus]

То есть сейчас рассматриваем два варианта:

1. Сначала отдельно подключаемся к БД без пула. Если ошибка, то вы водим ее в лог и передаем на клиента. Если успех, то отключаемся и подключаемся уже через пул. Выглядит конечно не очень профессионально. Как затычка какая-то.
2. Сделать форк библиотеки пула и подправить там все, что нам надо. Отправить PR автору. Пока он не подправит -- использовать наш форк. Выглядит уже получше.

[gsbelarus]

И, как я понял, еще и на клиенте что-то придется допиливать, чтобы ловились ошибки правильно.

[sywka]

По поводу пункта 2 - уже есть PR, но без апрува coopernurse/node-pool#184

[gsbelarus]

Судя по датам там этот PR никогда не будет смержен. Надо делать свой форк и подправить или вообще выкинуть эту библиотеку. Она у нас только для конектов к БД используется?

[sywka]

Скорее всего да, не будут его мержить). Кроме драйвера эта либа используется для реализации пула дочерних процессов.

В идеале, конечно нужно доделать либу пулов, но сейчас мне кажется нет смысла тратить силы на совершенствование своего пула, если с выходом Firebird 4, его скорее всего заменит встроенный. Чисто для процессов разве что.

Самый оптимальный вариант, как мне кажется, сделать затычку (1 пункт)
Добавить в ADatabase метод:

public async ping(): Promise<void> {
  const {driver, connectionOptions} = this.dbDetail;
  const testConnection = driver.newConnection();

  try {
    await testConnection.connect(connectionOptions);
  } finally {
    if (testConnection.connected) {
      await testConnection.disconnect();
    }
  }
}

и внутри реализации команды CREATE_APP вызывать.

По сути мы это же делаем только с целым приложением. И там у нас идет полная инициализация приложения, а с этим методом мы ограничимся только коннектом к БД. Будет быстрее

[ilonaand]

Уже написали метод, показывает ошибку правильно

[sywka]

А вообще повторюсь, основная проблема в драйвере, т.к при большом количестве реконнектов (в следствии особенности работы пула), драйвер кладет весь процесс с каким-то error кодом

[gsbelarus]

Т.е. если написать маленький тестик на голом драйвере, который просто в цикле будет пытаться много раз подключиться к существующей БД с неверным логином/паролем, то драйвер упадет? Илона, надо написать -- отправим фернандесу пусть исправляет.

…

On Wed, Jun 26, 2019 at 8:33 PM Lomashko Yuri ***@***.***> wrote: А вообще повторюсь, основная проблема в драйвере, т.к при большом количестве реконнектов ( в следствии особенности работы пула), драйвер кладет весь процесс с каким-то кодом — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#157?email_source=notifications&email_token=ABHPTVDORVGE4DDGICY2QULP4OR6BA5CNFSM4H2Y6IVKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODYUIONA#issuecomment-505972532>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ABHPTVADCDVUQVU7W6GCVLLP4OR6BANCNFSM4H2Y6IVA> .