Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[r502] Все скрытые боты хаба вылезли наружу. #982

Closed
pavel-pimenov opened this issue Aug 23, 2015 · 23 comments
Closed

Comments

@pavel-pimenov
Copy link
Owner

@pavel-pimenov pavel-pimenov commented Aug 23, 2015

From zippoz...@gmail.com on March 18, 2013 10:11:50

После недавних правок по "скрытым ботам", в список файлов вылезли абсолютно все боты хаба, даже те, которые не зарегистрированы (не должны находиться в списке файлов).

Скриншот прилагаю. Специально выделил фильтром по букве "д". Но подобных ботов в моём хабе много.
Так-же, бот "--- Последние 25 сообщений ---" имеется в хабе dchub://flylinkdc.dyndns.org

Attachment: Screenshot-240.png

Original issue: http://code.google.com/p/flylinkdc/issues/detail?id=944

@pavel-pimenov
Copy link
Owner Author

@pavel-pimenov pavel-pimenov commented Aug 23, 2015

From mike.kor...@gmail.com on March 18, 2013 10:45:51

Да странная картина, у меня появились MOTD и Reminder , от верлихаба, хотя я всегда считал что сообщения от данных ботов рассылаются открытыми ботами хаба которым я присвоил другие имена (они также видны как и раньше).

@pavel-pimenov
Copy link
Owner Author

@pavel-pimenov pavel-pimenov commented Aug 23, 2015

From a.rain...@gmail.com on March 29, 2013 10:33:58

Всё просто - в обычном Флае фича отключена, в ежовой включена и продолжает работать. Что бы она появилась в обычной - упрашивайте Пашу :)

p.s: так же добавил сокрытие лишних пользоватлей на ADC хабах, алгоритм сокрытия такой же как и для NMDC.

Status: Accepted

@pavel-pimenov
Copy link
Owner Author

@pavel-pimenov pavel-pimenov commented Aug 23, 2015

From a.rain...@gmail.com on March 29, 2013 10:37:30

В общем ставим звёздочки кому нужна эта фича ;) в ежовой эти боты никогда не вылезут, ибо я тоже считаю, что не нужны они.

@pavel-pimenov
Copy link
Owner Author

@pavel-pimenov pavel-pimenov commented Aug 23, 2015

From Pavel.Pimenov@gmail.com on May 01, 2013 05:49:48

This issue was closed by revision r13769 .

Status: Fixed

@pavel-pimenov
Copy link
Owner Author

@pavel-pimenov pavel-pimenov commented Aug 23, 2015

From toss.Alexey on May 01, 2013 12:44:08

В EiskaltDC++ это регулируется через настройку Интерфейс - Чат - Отображать скрытых пользователей. Возможно, стоит позаимствовать такую настройку.

@pavel-pimenov
Copy link
Owner Author

@pavel-pimenov pavel-pimenov commented Aug 23, 2015

From zippoz...@gmail.com on January 19, 2014 11:59:13

Status: Accepted

@pavel-pimenov
Copy link
Owner Author

@pavel-pimenov pavel-pimenov commented Aug 23, 2015

From zippoz...@gmail.com on January 19, 2014 12:02:35

Незарегистрированные боты снова в списке пользователей!
Это сраный косяк!

Пока я не дал команду Core.RegBot(), бота вообще не должно быть в списке.

Проверяем так: хаб dchub://scalolaz.dyndns-server.com
Бот например "ДжинглBells"
Он неЗАРЕГИСТРИРОВАН, тоесть спамит просто в чат, меню для пользователей не имеет, список не занимает и в списке его быть не должно.

Запускаем r400 , смотрим. Запускаем r5xx, смотрим, сравниваем.

@pavel-pimenov
Copy link
Owner Author

@pavel-pimenov pavel-pimenov commented Aug 23, 2015

From zippoz...@gmail.com on January 19, 2014 12:03:39

Пардон. О версии забыл сообщить.

-=[ FlylinkDC++ r503 -beta21-x64 build 16373 Compiled on: 2014-01-17 ]=-
-=[ OS: Microsoft Windows 7 Ultimate Edition Service Pack 1, 64-bit ]=-

@pavel-pimenov
Copy link
Owner Author

@pavel-pimenov pavel-pimenov commented Aug 23, 2015

From mike.kor...@gmail.com on January 19, 2014 12:09:10

Опять все боты вылезли, причем, неправильно.
На некоторых хабах есть возможность писать от другого имени, а также игры с подменой ников.
Так вот, как кто-то напишет от чьего-то имени в списке пользователей появляется пользователь с таким именем.
Страшно подумать если запустится игра с подменой реальных ников на смешные, человек 5 за полчаса нагенерируют несколько сотен лишних пользователей в списке.
На картинке тест с ptokax, причем до написания в чат от админа его в списке флая не было.

Attachment: Fly_r16372_unhidden_bots.png

@pavel-pimenov
Copy link
Owner Author

@pavel-pimenov pavel-pimenov commented Aug 23, 2015

From mike.kor...@gmail.com on January 19, 2014 12:14:10

тесты подтверждают бездумное добавление любого нового ника из чата.

Attachment: Fly_r16372_unhidden_bots2.png

@pavel-pimenov
Copy link
Owner Author

@pavel-pimenov pavel-pimenov commented Aug 23, 2015

From Pavel.Pimenov@gmail.com on January 19, 2014 20:53:11

В каком билде было хорошо и в каком сломалось?
я просто эту часть не трогал

@pavel-pimenov
Copy link
Owner Author

@pavel-pimenov pavel-pimenov commented Aug 23, 2015

From Pavel.Pimenov@gmail.com on January 19, 2014 21:12:23

"тесты подтверждают бездумное добавление любого нового ника из чата."

Сможешь мне вечером такое повторить на своем хабе?

@pavel-pimenov
Copy link
Owner Author

@pavel-pimenov pavel-pimenov commented Aug 23, 2015

From mike.kor...@gmail.com on January 20, 2014 01:31:25

Это легко делается только на ptokax, а у меня верлихаб. Придется что-то мудрить.

@pavel-pimenov
Copy link
Owner Author

@pavel-pimenov pavel-pimenov commented Aug 23, 2015

From toss.Alexey on January 20, 2014 11:42:45

На имеющемся русхабе/птоке тоже легко повторить - запустите скрипт с этой строчкой:
for i=1,20 do Core.SendToAll("<abc"..i.."> Hello there!") end

В каком билде было хорошо и в каком сломалось?

А сломалось при починке issue #916 . И если выбирать между этими двумя issue, то 916 мне дороже.

@pavel-pimenov
Copy link
Owner Author

@pavel-pimenov pavel-pimenov commented Aug 23, 2015

From mike.kor...@gmail.com on January 20, 2014 13:01:35

Проверил на птохе со скриптом, все подтвердилось.
Сломали, наверняка, тогда когда вытащили скрытых ботов по чьей-то просьбе.

Attachment: Fly_r16381_unhidden_bots3.png

@pavel-pimenov
Copy link
Owner Author

@pavel-pimenov pavel-pimenov commented Aug 23, 2015

From tret2...@gmail.com on January 26, 2014 04:54:25

Сломали, наверняка, тогда когда вытащили скрытых ботов по чьей-то просьбе.
Не наверника а так оно и было.
Кто-то пожаловался что есть скрытый бот который виден на стронге или апексе а у нас не виден.

В связи с чем у меня вопрос - если хаб передаёт ник с флагом скрытый, почему мы должны его показать? (так делает какой либо другой клиент - не аргумент)

@pavel-pimenov
Copy link
Owner Author

@pavel-pimenov pavel-pimenov commented Aug 23, 2015

From Pavel.Pimenov@gmail.com on January 26, 2014 07:12:07

а что плохого в том что мы его покажем?
у юзеров меньше вопросов будет.

@pavel-pimenov
Copy link
Owner Author

@pavel-pimenov pavel-pimenov commented Aug 23, 2015

From tret2...@gmail.com on January 26, 2014 09:17:18

Как практика показала ничего хорошего, повылазило много того чего скрывают владельцы хаба от простого смертного юзера, не просто так же флаг скрытия выставляется.

@pavel-pimenov
Copy link
Owner Author

@pavel-pimenov pavel-pimenov commented Aug 23, 2015

From Pavel.Pimenov@gmail.com on January 26, 2014 18:46:29

Можно ссылку на доку протокола где этот флаг скрытия выставляется?
чем отличается MyINFO скрытого юзера от не скрытого?

@pavel-pimenov
Copy link
Owner Author

@pavel-pimenov pavel-pimenov commented Aug 23, 2015

From mike.kor...@gmail.com on January 27, 2014 03:41:08

Все-таки, хочу пояснить.
В моем примере скрипт хаба подменяет ники в сообщениях главного чата и только.
Эти самые ники не могут быть переданы хабом как пользователи по запросу списка пользователей и никакие флаги тут не помогут.
Если дать команду "Обновить список пользователей" то все "левые" пользователи исчезнут в списке.
Здесь явное выдумывание пользователей из сообщений в чате и внесение их в список.
Также, как если нет тэга, то он все равно отображается только с пустыми полями (заведу отдельную тему и по тэгам).
Если кто-то сломает хаб и начнет сыпать мусор в чат, то клиенту станет нехорошо.

@pavel-pimenov
Copy link
Owner Author

@pavel-pimenov pavel-pimenov commented Aug 23, 2015

From Pavel.Pimenov@gmail.com on January 27, 2014 03:44:47

Появление ников из чата это другая ситуация - и тут явный баг.
я постараюсь вечером тебя поймать и потестить на твоем хабе эту ситуацию.
просто ты как-то написал мне в 1:56 ночи (в это время я уже сплю :)

@pavel-pimenov
Copy link
Owner Author

@pavel-pimenov pavel-pimenov commented Aug 23, 2015

From mike.kor...@gmail.com on January 27, 2014 03:51:18

Включаю хаб для тестов с середины дня до поздней ночи по Москвскому времени.
По моему IP адресу, порт 8411.
Ограничение на частоту запросов списка пользователей поправил, банить не должно.

@pavel-pimenov
Copy link
Owner Author

@pavel-pimenov pavel-pimenov commented Aug 23, 2015

From Pavel.Pimenov@gmail.com on January 28, 2014 08:42:13

This issue was closed by revision r16444 .

Status: Fixed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
1 participant
You can’t perform that action at this time.