search

CHUNGYUN / flylinkdc

Automatically exported from code.google.com/p/flylinkdc
0 stars 0 forks source link

Бот хаба не показывается в списке пользователей #916

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
What steps will reproduce the problem?
1. Зайти на хаб flylinkdc.dyndns.org
2. Попытаться найти в списке пользователей 
ник Invisible

What is the expected output? What do you see instead?
Ожидаю увидеть в списке ник бота, как это и 
происходит в других клиентах. Но его в 
списке нет.

What version of the product are you using? On what operating system?
-=[ FlylinkDC++ r502-beta66 build 13050 HE Compiled on: 2013-02-27 ]=-
-=[ FlylinkDC++ r502-beta66 build 12954 Compiled on: 2013-02-21 ]=-

Please provide any additional information below.
Если это фича, то она должна быть 
отключаемой, разве нет?

Original issue reported on code.google.com by toss.Alexey on 28 Feb 2013 at 2:23

GoogleCodeExporter commented 9 years ago 
Это в Ежовой версии? он там что-то мудрил с 
ними.
почему рассказано про билд 12954:
у меня видно
https://www.box.net/shared/8zj0eaa9hh8wlebsw2in

Original comment by Pavel.Pimenov@gmail.com on 28 Feb 2013 at 2:45

GoogleCodeExporter commented 9 years ago 
Да я и на ежовой это подтвердить не могу 
http://studia2000.sytes.net/image/Flylink/no-error-on-user-filtering.png 
покажите скрин что ли.

p.s:
>>он там что-то мудрил с ними.
поиском по коду FLYLINKDC_HE - там всё одинаково ;)

Original comment by a.rain...@gmail.com on 28 Feb 2013 at 2:53

GoogleCodeExporter commented 9 years ago 
Тут в чатике разрабов высказали мнение, что 
вы не ОП, по этому и бота не видите - 
настройки хаба таковы.

Original comment by a.rain...@gmail.com on 28 Feb 2013 at 2:57

GoogleCodeExporter commented 9 years ago 
Вопрос.
1. откуда вы узнали что он там есть?
2. зачем вам его видеть?

У него статус не видимый - я вот например оп. 
но видеть его тоже не особо хочу.
но объяснить и доказать это не могу.

Original comment by Pavel.Pimenov@gmail.com on 28 Feb 2013 at 2:59

GoogleCodeExporter commented 9 years ago 
По скриншотам: бота нет в списке, пока вы не 
начинаете писать в окошке фильтра списка 
пользователей или пока выключите-включите 
показ списка пользователей. Попробуйте 
найти бота в списке перезайдя на хаб и не 
прибегая к этим фокусам? У меня не 
получилось (см. скриншот).

> "Тут в чатике разрабов высказали мнение, 
что вы не ОП, по этому и бота не видите - 
настройки хаба таковы."

Мнение неправильное, ведь как я уже 
написал, другие клиенты (ApexDC++, EiskaltDC++, Greylink) 
нормально показывают бота в списке без 
лишних телодвижений.

> "У него статус не видимый"

Нет такого статуса в NMDC.

Original comment by toss.Alexey on 28 Feb 2013 at 3:11

Attachments:

GoogleCodeExporter commented 9 years ago 
> "2. зачем вам его видеть?"

Конкретно этот бот мне не сильно нужен, я 
его взял просто для иллюстрации проблемы. А 
вообще боты могут выполнять разнообразные 
задачи: это и комнаты и всякие викторины и 
боты для управления хабом. Что-бы начать 
использовать такого бота, достаточно 
написать ему в приват, но как писать, если 
его нет в списке?

Original comment by toss.Alexey on 28 Feb 2013 at 3:17

GoogleCodeExporter commented 9 years ago 
Вдобавок, совершенно непонятно, по каким 
критериям вы решаете скрыть или показать 
бота.

Original comment by toss.Alexey on 28 Feb 2013 at 3:18

GoogleCodeExporter commented 9 years ago 
CT_HIDDEN = 64 
Вот код оригинального клиента

dcplusplus\dcpp\OnlineUser.h

/** One of possibly many identities of a user, mainly for UI purposes */
class Identity : public Flags {
public:
        enum IdentityFlagBits {
                GOT_INF_BIT,
                NMDC_PASSIVE_BIT
        };
        enum IdentityFlags {
                GOT_INF = 1 << GOT_INF_BIT,
                NMDC_PASSIVE = 1 << NMDC_PASSIVE_BIT
        };
        enum ClientType {
                CT_BOT = 1,
                CT_REGGED = 2,
                CT_OP = 4,
                CT_SU = 8,
                CT_OWNER = 16,
                CT_HUB = 32,
                CT_HIDDEN = 64
        };

Original comment by Pavel.Pimenov@gmail.com on 28 Feb 2013 at 3:28

GoogleCodeExporter commented 9 years ago 
А этот код относится не к ADC?

Original comment by toss.Alexey on 28 Feb 2013 at 3:33

GoogleCodeExporter commented 9 years ago 
Нет - юзается в NMDC

D:\bz-src\dcplusplus\dcpp\NmdcHub.cpp

                OnlineUser& u = getUser(nick);

                // If he is already considered to be the hub (thus hidden), probably should appear in the UserList
                if(u.getIdentity().isHidden()) {
                        u.getIdentity().setHidden(false);
                        u.getIdentity().setHub(false);

Original comment by Pavel.Pimenov@gmail.com on 28 Feb 2013 at 3:43

GoogleCodeExporter commented 9 years ago 
В оригинальном флае будет показывать везде.
в Ежовом будет как раньше - прятаться

https://code.google.com/p/flylinkdc/source/detail?r=13071#

Original comment by Pavel.Pimenov@gmail.com on 28 Feb 2013 at 3:44

GoogleCodeExporter commented 9 years ago 
>>Конкретно этот бот мне не сильно нужен, я 
его взял просто для иллюстрации проблемы. А 
вообще боты могут выполнять разнообразные 
задачи: это и комнаты и всякие викторины и 
боты для управления хабом. Что-бы начать 
использовать такого бота, достаточно 
написать ему в приват, но как писать, если 
его нет в списке?
Скрываются только те боты для которых хаб 
прислал соответствующий флаг, если флага 
нет - бот будет в списке.

>>Вдобавок, совершенно непонятно, по каким 
критериям вы решаете скрыть или показать 
бота.
нам этот флаг со стороны хаба приходит.

p.s: в ежовой починю баг с отображением таких 
ботов.

Original comment by a.rain...@gmail.com on 28 Feb 2013 at 3:51

GoogleCodeExporter commented 9 years ago 
Давайте еще уточним этот флаг есть в 
стандарте или нет?

Original comment by Pavel.Pimenov@gmail.com on 28 Feb 2013 at 3:54

GoogleCodeExporter commented 9 years ago 
В самом протоколе - точно нет. Однако с NMDC у 
всех разработчиков возникает попоболь в 
плане совместимости, из-за того, что есть 
куча расширений, которые все поголовно 
используют, но которые официально не 
приняты.

Original comment by a.rain...@gmail.com on 28 Feb 2013 at 4:22

GoogleCodeExporter commented 9 years ago 
Флаг мальчик-девочка и то был бы полезней :)

Original comment by Pavel.Pimenov@gmail.com on 28 Feb 2013 at 4:44

GoogleCodeExporter commented 9 years ago 
Паш, вот скажи, зачем видить ботов, которым 
не написать нельзя, не скачать с них ничего 
нельзя, и вообще ничего полезного с ними 
сделать нельзя? :)

Original comment by a.rain...@gmail.com on 28 Feb 2013 at 4:47

GoogleCodeExporter commented 9 years ago 
Тогда их не нужно создавать вообще в 
клиенте.
а ты их героически прячешь выполняя всегда 
фильтрацию всех юзеров по этому флагу.
Также если они такие бесполезные зачем они 
нужны?

Original comment by Pavel.Pimenov@gmail.com on 28 Feb 2013 at 5:01

GoogleCodeExporter commented 9 years ago 
Признак скрытости передаётся не сразу, по 
этому при обновлении инфы о юзере надо его 
проверять, и если юзер теперь скрыт - 
дельнуть из списка. В общем этот флаг 
необходим  для полностью информационных 
ботов - они могут что то писать в чатик, но 
видить их в списке при этом нет 
необходимости. К примеру можно создать 
бота "Illegal search filter" который будет уведомлять 
пользователя о том, что он ищет что то 
запрещённое и по этому поиск не будет 
осуществлён, очевидно, что у такого бота не 
может быть настроек, или чего то подобного, 
по этому в списке он не нужен. Так же скрыт 
бот хаба, он тоже может писать в чатик, но 
более ничего делать не может, и как либо 
взаимодействовать с ним не имеет смысла.
Возможно не стоит их создавать вообще, 
однако не уверен, что при этом будут 
корректно работать фильтры спама от 
фейковых юзеров - это надо посмотреть.

Original comment by a.rain...@gmail.com on 28 Feb 2013 at 5:14

GoogleCodeExporter commented 9 years ago 
> "Скрываются только те боты для которых хаб 
прислал соответствующий флаг, если флага 
нет - бот будет в списке."

Лично я из приведённого кода не понял, что 
понимается под таким флагом в NMDC? Можно 
пример в терминах NMDC?

Original comment by toss.Alexey on 28 Feb 2013 at 6:17

GoogleCodeExporter commented 9 years ago 
Я предложил Алексею просто таким 
ботам-нивидимкам иконку менять
это будет намного эффективней
не хочет - говорит невидимый значит никто 
не видит.
кино про невидимку не смотрел наверно. 
кинуть ему чтоль магнит )

Original comment by Pavel.Pimenov@gmail.com on 28 Feb 2013 at 6:22

GoogleCodeExporter commented 9 years ago 
Скриншот клиента DC++, бот отображается 
нормально без всякой магии

Original comment by toss.Alexey on 28 Feb 2013 at 6:23

Attachments:

GoogleCodeExporter commented 9 years ago 
Это инициатива исклбючительно Алексея. 
он считает, что если невидимка, то значит не 
нужно видеть

Original comment by Pavel.Pimenov@gmail.com on 28 Feb 2013 at 6:31

GoogleCodeExporter commented 9 years ago 
Не совсем так. Эта инициатива SMT, т.е. ещё со 
времён Apex Speed Mode, просто я переделывал, как и 
многое другое в клиенте.
Ну а так - да, имхо - инициатива правильная.

Original comment by a.rain...@gmail.com on 28 Feb 2013 at 6:47

GoogleCodeExporter commented 9 years ago 
Бот отображается нормально в -=[ ApexDC++ s16.4 
Compiled on: 2008-06-12 ]=-

Я ещё раз прошу показать, что вы считаете 
невидимкой в NMDC?

Original comment by toss.Alexey on 28 Feb 2013 at 6:54

GoogleCodeExporter commented 9 years ago 
> "Флаг мальчик-девочка и то был бы полезней 
:)"

Про этот флаг можно забыть в том виде, в 
котором он реализован в грейлинке - 
конфликтует с PtokaX 0.5.0.0.

Original comment by toss.Alexey on 28 Feb 2013 at 7:00

GoogleCodeExporter commented 9 years ago 
Алексей, про невидимку я тоже хочу услышать 
ответ. и уточни урл где SMT это затеял.

Original comment by Pavel.Pimenov@gmail.com on 1 Mar 2013 at 2:46

GoogleCodeExporter commented 9 years ago 
Почему молчим? :)

Original comment by Pavel.Pimenov@gmail.com on 1 Mar 2013 at 2:35

GoogleCodeExporter commented 9 years ago 
Надеялся, что после полного экранирования 
фичи сами посмотрите :) А так молчал, потому 
как до компа добрался только сейчас.

Original comment by a.rain...@gmail.com on 2 Mar 2013 at 7:27

GoogleCodeExporter commented 9 years ago 
Там нету какого либо флажка, просто 
условие: если бот не в списке 
пользователей, он отмечается как хаб, и 
скрывается.

Original comment by a.rain...@gmail.com on 2 Mar 2013 at 7:32

GoogleCodeExporter commented 9 years ago 
Я волью капельку скипидара в ваши 
обсуждения:

После недавних действий со "скрытыми 
ботами" в транке (в серой ветке), в NMDC хабах я 
вижу ВСЕХ ботов, даже тех, которые не 
зарегистрированы хабом, то-есть для них не 
выполнялась команда Core.RegBot(). Эти боты 
обычно сидят молча, пока в менюшке не 
выбирают какой-либо пункт, например 
"Показать мне анекдот". Тогда такие боты под 
своим ником срут в чат или личку.

Теперь же, ВСЕ эти боты в списке 
пользователей. Ответьте мне,- нафига???
Сочиняя скрипт, я специально не публикую 
бота, чтобы его не торкали запросами 
списков файлов, личками и прочей хернёй. А 
тут нате - вывалили всё что непопадя наружу.

За сим, предлагаю вернуть всё как было и как 
есть в Ежовой сборке.

Original comment by zippoz...@gmail.com on 14 Mar 2013 at 10:29

GoogleCodeExporter commented 9 years ago 
Согласен с предыдущим оратором, ненужных 
ботов расплодилось, а нужные у меня никуда 
и не пропадали. Да есть небольшая разница в 
отображении ботов с грейлинком на adc хабах, 
но что нам до него.

Original comment by mike.kor...@gmail.com on 19 Mar 2013 at 12:35

GoogleCodeExporter commented 9 years ago 
Про расплодившихся ботов  - Issue 944.

Original comment by a.rain...@gmail.com on 1 May 2013 at 12:13

GoogleCodeExporter commented 9 years ago 
Исправлено - закрываю
 https://www.box.com/files/0/f/215289294/1/f_7837967230
Продолжение в Issue 944

Original comment by Pavel.Pimenov@gmail.com on 1 May 2013 at 12:46

GoogleCodeExporter commented 9 years ago 
Опять сломалось.

Original comment by toss.Alexey on 27 Sep 2013 at 6:53

GoogleCodeExporter commented 9 years ago 
В каком билде?
я вроде в той части ничего не трогал...

Original comment by Pavel.Pimenov@gmail.com on 27 Sep 2013 at 6:56

GoogleCodeExporter commented 9 years ago 
В 72 бете ещё работало, в сотых бетах - уже 
нет, а в какой бете заметил первый раз - уже 
не помню. Где достать бинарников меньше 100 
беты я не нашёл. Не из инсталляторов же 
выдирать.

Original comment by toss.Alexey on 27 Sep 2013 at 7:14

GoogleCodeExporter commented 9 years ago 
Похоже, сломали в r14764, так как за пару 
ревизий до него сабж на месте.

Original comment by toss.Alexey on 27 Sep 2013 at 9:14

GoogleCodeExporter commented 9 years ago 
ок. завтра утром посмотрю.
p.s.
Все билды сейчас лежат тут
http://www.fly-server.ru/install/r5xx/src-bin

Original comment by Pavel.Pimenov@gmail.com on 27 Sep 2013 at 10:17

GoogleCodeExporter commented 9 years ago 
Подписался на апдейты по ишье.

Original comment by a.rain...@gmail.com on 28 Sep 2013 at 1:59

GoogleCodeExporter commented 9 years ago

Original comment by Pavel.Pimenov@gmail.com on 19 Dec 2013 at 5:50

GoogleCodeExporter commented 9 years ago 
Хаб

dchub://scalolaz.dyndns-server.com

В r422 бот мастерйода есть
http://www.flickr.com/photos/96019675@N02/11453881553/

в r503 уже нет
http://www.flickr.com/photos/96019675@N02/11453844556/

[22:24:33] <wasd123> Нет такого понятия "флаг 
невидимости".
[22:24:43] <wasd123> В NMDC.
[22:25:16] <wasd123> В ADC есть, но здесь вам не ADC!

Original comment by Pavel.Pimenov@gmail.com on 19 Dec 2013 at 6:29

GoogleCodeExporter commented 9 years ago 
This issue was closed by revision r16252.

Original comment by Pavel.Pimenov@gmail.com on 19 Dec 2013 at 6:55