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

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

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

Comments

@pavel-pimenov
Copy link
Owner

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

From toss.Alexey on February 28, 2013 15:23:54

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: http://code.google.com/p/flylinkdc/issues/detail?id=916

@pavel-pimenov
Copy link
Owner Author

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

From Pavel.Pimenov@gmail.com on February 28, 2013 06:45:37

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

@pavel-pimenov
Copy link
Owner Author

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

From a.rain...@gmail.com on February 28, 2013 06:53:54

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

p.s:

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

Status:
Labels: -Type-Defect -Priority-Medium Type-Other

@pavel-pimenov
Copy link
Owner Author

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

From a.rain...@gmail.com on February 28, 2013 06:57:45

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

Status: Done

@pavel-pimenov
Copy link
Owner Author

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

From Pavel.Pimenov@gmail.com on February 28, 2013 06:59:55

Вопрос.

  1. откуда вы узнали что он там есть?
  2. зачем вам его видеть?

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

Status: Accepted
Owner: Pavel.Pimenov@gmail.com

@pavel-pimenov
Copy link
Owner Author

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

From toss.Alexey on February 28, 2013 07:11:30

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

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

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

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

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

Attachment: 20130228_FLDCPP_HubBot.png

@pavel-pimenov
Copy link
Owner Author

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

From toss.Alexey on February 28, 2013 07:17:11

"2. зачем вам его видеть?"

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

@pavel-pimenov
Copy link
Owner Author

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

From toss.Alexey on February 28, 2013 07:18:25

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

@pavel-pimenov
Copy link
Owner Author

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

From Pavel.Pimenov@gmail.com on February 28, 2013 07:28:34

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
};

@pavel-pimenov
Copy link
Owner Author

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

From toss.Alexey on February 28, 2013 07:33:23

А этот код относится не к ADC?

@pavel-pimenov
Copy link
Owner Author

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

From Pavel.Pimenov@gmail.com on February 28, 2013 07:43:16

Нет - юзается в 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);

@pavel-pimenov
Copy link
Owner Author

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

From Pavel.Pimenov@gmail.com on February 28, 2013 07:44:19

В оригинальном флае будет показывать везде.
в Ежовом будет как раньше - прятаться https://code.google.com/p/flylinkdc/source/detail?r=13071#

@pavel-pimenov
Copy link
Owner Author

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

From a.rain...@gmail.com on February 28, 2013 07:51:21

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

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

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

@pavel-pimenov
Copy link
Owner Author

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

From Pavel.Pimenov@gmail.com on February 28, 2013 07:54:31

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

@pavel-pimenov
Copy link
Owner Author

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

From a.rain...@gmail.com on February 28, 2013 08:22:29

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

@pavel-pimenov
Copy link
Owner Author

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

From Pavel.Pimenov@gmail.com on February 28, 2013 08:44:42

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

@pavel-pimenov
Copy link
Owner Author

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

From a.rain...@gmail.com on February 28, 2013 08:47:31

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

@pavel-pimenov
Copy link
Owner Author

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

From Pavel.Pimenov@gmail.com on February 28, 2013 09:01:59

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

@pavel-pimenov
Copy link
Owner Author

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

From a.rain...@gmail.com on February 28, 2013 09:14:38

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

@pavel-pimenov
Copy link
Owner Author

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

From toss.Alexey on February 28, 2013 10:17:53

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

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

@pavel-pimenov
Copy link
Owner Author

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

From Pavel.Pimenov@gmail.com on February 28, 2013 10:22:50

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

@pavel-pimenov
Copy link
Owner Author

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

From toss.Alexey on February 28, 2013 10:23:32

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

Attachment: 20130228_DCPP_HubBot.PNG

@pavel-pimenov
Copy link
Owner Author

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

From Pavel.Pimenov@gmail.com on February 28, 2013 10:31:12

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

@pavel-pimenov
Copy link
Owner Author

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

From a.rain...@gmail.com on February 28, 2013 10:47:54

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

@pavel-pimenov
Copy link
Owner Author

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

From toss.Alexey on February 28, 2013 10:54:17

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

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

@pavel-pimenov
Copy link
Owner Author

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

From toss.Alexey on February 28, 2013 11:00:22

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

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

@pavel-pimenov
Copy link
Owner Author

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

From Pavel.Pimenov@gmail.com on February 28, 2013 18:46:08

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

@pavel-pimenov
Copy link
Owner Author

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

From Pavel.Pimenov@gmail.com on March 01, 2013 06:35:38

Почему молчим? :)

@pavel-pimenov
Copy link
Owner Author

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

From a.rain...@gmail.com on March 02, 2013 11:27:57

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

@pavel-pimenov
Copy link
Owner Author

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

From a.rain...@gmail.com on March 02, 2013 11:32:44

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

@pavel-pimenov
Copy link
Owner Author

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

From zippoz...@gmail.com on March 14, 2013 03:29:37

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

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

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

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

@pavel-pimenov
Copy link
Owner Author

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

From mike.kor...@gmail.com on March 19, 2013 05:35:42

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

@pavel-pimenov
Copy link
Owner Author

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

From a.rain...@gmail.com on May 01, 2013 05:13:06

Про расплодившихся ботов - Issue 944 .

@pavel-pimenov
Copy link
Owner Author

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

From Pavel.Pimenov@gmail.com on May 01, 2013 05:46:29

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

Status: Done

@pavel-pimenov
Copy link
Owner Author

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

From toss.Alexey on September 27, 2013 11:53:23

Опять сломалось.

@pavel-pimenov
Copy link
Owner Author

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

From Pavel.Pimenov@gmail.com on September 27, 2013 11:56:07

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

Status: Accepted

@pavel-pimenov
Copy link
Owner Author

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

From toss.Alexey on September 27, 2013 12:14:23

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

@pavel-pimenov
Copy link
Owner Author

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

From toss.Alexey on September 27, 2013 14:14:35

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

@pavel-pimenov
Copy link
Owner Author

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

From Pavel.Pimenov@gmail.com on September 27, 2013 15:17:50

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

@pavel-pimenov
Copy link
Owner Author

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

From a.rain...@gmail.com on September 27, 2013 18:59:39

Подписался на апдейты по ишье.

Cc: a.rain...@gmail.com
Labels: Milestone-Release502 Usability

@pavel-pimenov
Copy link
Owner Author

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

From Pavel.Pimenov@gmail.com on December 19, 2013 09:50:11

Status: Started
Cc: -a.rain...@gmail.com

@pavel-pimenov
Copy link
Owner Author

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

From Pavel.Pimenov@gmail.com on December 19, 2013 10:29:59

Хаб

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

@pavel-pimenov
Copy link
Owner Author

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

From Pavel.Pimenov@gmail.com on December 19, 2013 10:55:19

This issue was closed by revision r16252 .

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