Opened 8 years ago
Closed 8 years ago
#198 closed баг (fixed)
Устранить ненужные запросы к коммутатору
Reported by: | alx | Owned by: | dimag |
---|---|---|---|
Priority: | minor | Milestone: | 1 очередь |
Component: | ПО MC04-Dispatcher. Пульт диспетчера/техника | Keywords: | login password |
Cc: | san |
Description
Обнаружено (как минимум, в r229), что программа диспетчера запрашивает у коммутатора информацию, которая, как мне кажется, ей, в моем представлении, совершенно не нужна.
Так, я вошел в систему как пользователь 2, и вижу, что программа, например, запрашивает у коммутатора права (атрибут allow) пользователя test11. Секундочку, мы же вошли как пользователь 2, а не test11, зачем ей права пользователя test11? Также вижу, как программа запрашивает пароль пользователя test11. Зачем он ей?
Считаю, что все лишние запросы надо устранить, так как они лишь создают бесполезную нагрузку на сеть и коммутатор.
Change History (33)
comment:1 by , 8 years ago
comment:2 by , 8 years ago
Resolution: | → fixed |
---|---|
Status: | new → closed |
r237, исправил все излишние запросы к серверу и сообщения на перерисовку окна приложения какие возможно без существенной переделки программы.
comment:3 by , 8 years ago
Resolution: | fixed |
---|---|
Status: | closed → reopened |
В r366 программа по-прежнему запрашивает пароли всех пользователей.
follow-up: 5 comment:4 by , 8 years ago
Запрашивать пароль необходимо, я сравниваю пароль который ввёл пользователь в окне входа программы с полученным паролём. Если они не равны, то вывожу в окно входа программы сообщение об ошибке.
follow-up: 7 comment:5 by , 8 years ago
Replying to dimag:
Запрашивать пароль необходимо, я сравниваю пароль который ввёл пользователь в окне входа программы с полученным паролём. Если они не равны, то вывожу в окно входа программы сообщение об ошибке.
- Зачем?
- Пароли у пользователей разные. Следовательно, хоть с каким-то из них пароль, введенный в окне входа программы, не совпадает. Почему я ни разу не видел сообщения об ошибке?
comment:6 by , 8 years ago
Я поменял пароль у пользователя 1@192.168.0.73 с 1 на 11 в программе, перезапустил её, ввёл пароль 1. Этот пароль сравнился с сохранённым паролём 11, который я загрузил.
Так как пароли не равны, то вывелось сообщение "Не совпадает введённый пароль и пароль с FreeSwitch сервера".
Вопрос к Александру, должен ли проверять введённый в строке "Пароль" пароль с паролём который я получаю с FreeSwitch сервера командой user_data имя паользователя param password.
Если да, то пароли надо загружать, если нет, то парользагружать не нужно.
follow-up: 9 comment:8 by , 8 years ago
Replying to dimag:
- Насколько я понимаю у Алексея нет претензий к тому, что ты проверяешь пароль своего пользователя, речь о том, что запрашиваются ещё и пароли остальных абонентов, которые сейчас тебе не нужны.
- насчёт сверки пароля своего пользователя. Есть ли необходимость это делать? если пароль не совпадает, то авторизация в Sip будет неуспешной о чём будет соответствующее сообщение, единственное что мы, может быть, при своей проверке получим более точную причину отказа авторизации: "не совпадает пароль"... Алексей, как ты думаешь, имеет смысл сравнивать пароль пользователя с паролем его абонента в настройках фрисвитча?
comment:9 by , 8 years ago
Replying to san:
- Насколько я понимаю у Алексея нет претензий к тому, что ты проверяешь пароль своего пользователя,
Претензия есть, и она полностью совпадает с тем, что ты написал: эта проверка бессмысленна, так как сервер уже проверил введенный пароль.
единственное что мы, может быть, при своей проверке получим более точную причину отказа авторизации: "не совпадает пароль".
Как мы ее получим, если сервер нас к себе просто не пустит? Если пользователь ввел неверный пароль, мы при всем желании ничего проверить не сможем...
comment:10 by , 8 years ago
Если пользователь ввёл неправильный пароль - ошибка будет на этапе подключения к БД.
Ну а если, например, кто-то случайно изменил пароль пользователя в конфиге FS(ну или при смене паролей, наоборот забыл его изменить), тогда программа полключится к БД, затем к ES программа подключится с паролем взятым из БД, ну а при в авторизаци SIP ей будет отказано, и по выданой ошибке всё будет понятно.
Думаю что можно отказаться от "сверки" паролей.
comment:11 by , 8 years ago
А, верно. Я не подумал, что в BD может быть один пароль, а в FS - другой...
comment:12 by , 8 years ago
Keywords: | login password added |
---|---|
Resolution: | → fixed |
Status: | reopened → closed |
Александр, отказываемся от проверки пароля с паролем полученным от FreeSwitch сервера или проверяем?
Что об этом думает заказчик?
comment:13 by , 8 years ago
Resolution: | fixed |
---|---|
Status: | closed → reopened |
По просьбе Димы уточняю: Нужно убрать из программы "сверку паролей".
comment:14 by , 8 years ago
На всякий случай еще уточню: программа не только не должна сверять пароли, она не должна их запрашивать с FS.
comment:16 by , 8 years ago
Resolution: | → fixed |
---|---|
Status: | reopened → closed |
r379
Сверка с паролём записанным у FreeSwitch сервера отключенна.
comment:17 by , 8 years ago
Увидел, что если в уже вошедшей в сеть программе я нажимаю кнопку "Play" на вкладке записей переговоров, то она (программа) запрашивает у коммутатора список конференций командой "conference list". Зачем она его запрашивает, если этот список уже был запрошен при входе в сеть?
comment:18 by , 8 years ago
Keywords: | login, password → login password |
---|---|
Resolution: | fixed |
Status: | closed → reopened |
Отсутствие ответа я расцениваю как отсутствие необходимости в таких запросах. Следовательно, они должны быть убраны из программы.
follow-up: 20 comment:19 by , 8 years ago
Resolution: | → fixed |
---|---|
Status: | reopened → closed |
Исправленно начиная с r395
comment:20 by , 8 years ago
Resolution: | fixed |
---|---|
Status: | closed → reopened |
comment:21 by , 8 years ago
Resolution: | → fixed |
---|---|
Status: | reopened → closed |
r438
Список конференций не должен больше запрашиваться.
comment:22 by , 8 years ago
Resolution: | fixed |
---|---|
Status: | closed → reopened |
В r438 по-прежнему вижу запросы conference list.
follow-up: 24 comment:23 by , 8 years ago
Список конференций запрашивается после получения сообщения о создание создание конференции conference create.
Я не считаю эти запросы лишними, при обработке данного сообщения.
follow-up: 25 comment:24 by , 8 years ago
Replying to dimag:
Я не считаю эти запросы лишними, при обработке данного сообщения.
Объясните, пожалуйста, почему Вы не считаете эти запросы лишними, учитывая что список конференций и их участников и так есть в памяти программы. А также как это сочетается со сказанным в comment:21:
Список конференций не должен больше запрашиваться.
follow-up: 26 comment:25 by , 8 years ago
Replying to alx:
Replying to dimag:
Я не считаю эти запросы лишними, при обработке данного сообщения.
Объясните, пожалуйста, почему Вы не считаете эти запросы лишними, учитывая что список конференций и их участников и так есть в памяти программы. А также как это сочетается со сказанным в comment:21:
Но не при создание новой конференции при получение сообщения conference_create
Список конференций не должен больше запрашиваться.
При обработке сообщения PlaySoundFile в потоке Commands.
comment:26 by , 8 years ago
Replying to dimag:
Но не при создание новой конференции при получение сообщения conference_create
Будьте добры, переведите это, пожалуйста, на русский или английский язык. Я не понимаю, что Вы написали.
При получение сообщения CUSTOM от FS с параметром Event-Subclass равным conference-create, при обработке это параметра, я всегда получаю список конференции с помощью команды api conference list и обновляю список конференций и их участников с помощью команды api conference list.
Можно конечно считывать при обработке события conference-create информацию только о созданной конференции с помощью команды api conference [имя конференции] list.
follow-up: 28 comment:27 by , 8 years ago
Я вызываю api conference [имя конференции] list только после получения сообщения о создание конференции conference-create.
comment:28 by , 8 years ago
Дима, зачем Вы редактируете мои комментарии? Еще раз раз прошу Вас не делать так.
follow-up: 30 comment:29 by , 8 years ago
Я не меняю ваши комментарии, можете показать какой комментарий я поменял?
comment:30 by , 8 years ago
Replying to dimag:
Я не меняю ваши комментарии, можете показать какой комментарий я поменял?
Вы поменяли comment:26. Вот внесенное Вами исправление: ?action=comment-diff&cnum=26&version=1
comment:32 by , 8 years ago
Milestone: | Текущее → 1 очередь |
---|
comment:33 by , 8 years ago
Resolution: | → fixed |
---|---|
Status: | reopened → closed |
Так как старый код, взаимодействовавший с FS, (классы ESL, Commands, Events) полностью удален из проекта и переписан "с нуля", этот тикет можно считать неактуальным (см. r725). Заркываю его с резолюцией fixed.
r236.
Закончил в этой версии, начал в r234, исправил в этой версии излишние запросы к серверу. Далее хочу исправить излишние перерисовки