From toss.Alexey on April 08, 2013 06:49:27

А если он сразу шару поднимет - зачем ему ждать целых 10 минут?

From a.rain...@gmail.com on April 08, 2013 07:24:31

С долей шутки: автобан - зло :)

Только для дополнительной информации:
Сообщение выдаётся по причине того, что автобан это единственный надёжный признак по которому можно определить наши СТАРЫЕ клиенты.
Цитирую кусочек кода:
if (aSource.isSet(UserConnection::FLAG_SUPPORTS_BANMSG))
aSource.error(UserConnection::PLEASE_UPDATE_YOUR_CLIENT);

Защита от DDOS срабатывает по причине того, что набирается 30 попыток получения одного и того же файла одним и тем же юзером в течении одной минуты.

Дорабатывать тут что либо с протокольной точки зрения (обменом между клиентаи) смысла не вижу, ибо в последних версиях обоих веток r4xx и r5xx (по времени уже около года) флажёк FLAG_SUPPORTS_BANMSG отсутсвует, стало быть сообщения вида:
17:07:43 Client:[Outgoing]$Error Please update your DC++ http://flylinkdc.com|
уйдёт только на наши старые клиенты, ну и на Грей :)

Что касается непосредственно ишьи: смысла убирать именно эти сообщение из лога не вижу, ибо запросы по 30-40 раз в секунду с той стороны это самый настоящий ДДОС. Поясню подробнее: мы посылаем туда ошибки, а клиент на той стороне вместо того что бы добавить нас в плохие источники продолжает попытки с нас скачать. Хотя немного не уверне на счёт этого момента, ибо есть сомнения должна ли та сторона добавлять нас в BadSource. В общем так или иначе для решения, описанной в ишье, проблемы необходима доработка механизма автобана, вероятно проверки автобана надо вынести выше проверок на DDOS в функции UploadManager::prepareFile.

Labels: Usability Component-Logic

From Pavel.Pimenov@gmail.com on April 08, 2013 11:24:06

Подробнее про отсуствие FLAG_SUPPORTS_BANMSG
он в коде есть
на основании какой проблемы он удален кем и когда?

From a.rain...@gmail.com on April 08, 2013 11:33:49

Не совсем точно написал. FLAG_SUPPORTS_BANMSG есть и работает. Однако наши новые клиенты уже не посылают в Supports признак BanMsg (UserConnection::FEATURE_BANMSG).
Отсылка выключена директивой SMT_ENABLE_FEATURE_BAN_MSG для того, что бы в будущем включить фичу IRAINMAN_DISALLOWED_BAN_MSG и тем самым принудить к обновлению сильно устаревшие клиенты, ибо они содержат кучу ошибок, и в них даже авто-апдейт не везде есть.

From JhaoDa on April 08, 2013 11:50:01

А если он сразу шару поднимет - зачем ему ждать целых 10 минут?
Ой, какая беда, прямо жуть, да как же это он, целых 10 минут???

Сообщение выдаётся по причине того, что автобан это единственный надёжный признак по которому можно определить наши СТАРЫЕ клиенты.
Мнэээ... я наблюдал, что клиент может быть, например, 411. Или 502. Или стронг свежий.

From JhaoDa on June 23, 2013 17:40:59

Поразительным образом проблема исчезла примерно месяц назад. Вроде ничего не менял в настройках, значит, починили мимоходом?

From a.rain...@gmail.com on June 28, 2013 10:30:58

Вероятно так, закрою ишью тогда.

Status: Done

From JhaoDa on June 29, 2013 02:15:08

Накаркали. Бага неожиданно вернулась, r14301 .

Status: Accepted

From Pavel.Pimenov@gmail.com on November 04, 2013 09:19:06

https://dl.dropboxusercontent.com/u/12213267/fly_bugs/fly_autoban.png

From Pavel.Pimenov@gmail.com on November 04, 2013 09:23:42

Owner: Pavel.Pimenov@gmail.com