несколько ошибок

GoogleCodeExporter commented 8 years ago

What steps will reproduce the problem?
1. -

What environment are you using?
1. My ruTorrent version is 3.6 (2404)
2. My rTorrent version is 0.9.2/0.13.2
3. I use web-server nginx/1.7.0 on OS Debian
4. I use browser all

далее, к сожалению напишу на русском.
накопил ошибок за время использования.

1. некорретное определение 
торрент-клиентов.
точнее: uTorrent 3.4.1, и тп. с 3 цифирями в версии 
определяется как неизвестный.
не знает и сам не нагуглил, следующие 
идентификаторы клиентов: -IL501%DE-, -SKQ7mu-
добавить в common.js определение Zona (azLikeClients4: "ZO" 
: "Zona")
клиент Aria2, не определяется. дописал (
            if(str.match(/^aria2/))
                ret = "Aria2";
            else
), но насколько корректно, не успел 
проверить.
дубль индетификатора (BX : "BittorrentX") в azLikeClients4
скрин в аттаче.
PS. на скрине "подправленный" common.js для 
вывода идентификаторов клиентов.

2. баг при сортировке в Хроме (любая версия) 
под виндой 7 (под другими не проверял).
не срабатывает сортировка по клику по 
заголовку столбца, событие обрабатывается 
как перетаскивание столбца. в хроме под 
линухом (дебиан, убунта) работает 
замечательно.

3. плагин tracklabel тормозит первоначальную 
загрузку страницы в 2-3 раза на FireFox по 
сравнению с Хромом.

4. ruTorrent не считывает комментарии из 
.torrent-файла, если он был загружен из 
каталога автозагрузок.

Original issue reported on code.google.com by ad...@vjean.net on 13 Jul 2014 at 4:57

Attachments:

scr0.png

GoogleCodeExporter commented 8 years ago

еще неизвестный идентификатор (в аттаче)
встречался идент в виде %00%00%00%00%00%00%00... т.е. 
полностью забитый %00

Original comment by ad...@vjean.net on 14 Jul 2014 at 4:38

Attachments:

scr1.png

GoogleCodeExporter commented 8 years ago

Original comment by ad...@vjean.net on 14 Jul 2014 at 4:45

Attachments:

scr2.png

GoogleCodeExporter commented 8 years ago

fixed:

azLikeClients4:
"TB" : "Torch",// браузер на базе вебкита, 
торрент-клиент типа оперы.

azLikeClients3:
"IL" : "iLivid", 

azLikeClientsSpec:
"SK" : "Spark", // тоже самое что и Torch, разработка 
baidu

case 'BF':
case 'LW':
case 'SK':  // не стал мудрить с определением 
версий
    ret = cli;
    break;

собстно, недостающие иденты можно найти 
сорцах libtorrent 
(http://sourceforge.net/p/libtorrent/code/HEAD/tree/trunk/src/identify_client.cp
p)

Original comment by ad...@vjean.net on 14 Jul 2014 at 6:32

GoogleCodeExporter commented 8 years ago

> точнее: uTorrent 3.4.1, и тп. с 3 цифирями в версии 
определяется как неизвестный.

Потому, что это фейк. По стандарту должно 
быть 4 цифры. 

> не знает и сам не нагуглил, следующие 
идентификаторы клиентов

и что, простите я должен сделать по этому 
поводу? Придумать идентификаторы? ;)

> но насколько корректно, не успел 
проверить.

Проверите - приходите ;)

> 3. плагин tracklabel тормозит первоначальную 
загрузку страницы в 2-3 раза на FireFox по 
сравнению с Хромом.

Именно поэтлму данная функциональность 
оформлена плагином. Не желаете терять 
время - отключите плагин.

> 4. ruTorrent не считывает комментарии из 
.torrent-файла, если он был загружен из 
каталога автозагрузок.

Так и должно быть.

> собстно, недостающие иденты можно найти 
сорцах libtorrent

Посмотрю потом. Но, собственно, 
определением пиров должен заниматься сам 
торрент клиент, а не фронт-енд к нему. 
Попробуйте добавить тикет сюда - 
https://github.com/rakshasa/rtorrent/issues

> 2. баг при сортировке в Хроме 

Просьба. В дальнейшем, оформляйте, 
пожалуйста, отдельные ошибки - отдельными 
тикетами.

Original comment by novik65 on 14 Jul 2014 at 11:32

Changed state: Accepted

GoogleCodeExporter commented 8 years ago

> Потому, что это фейк. По стандарту должно 
быть 4 цифры. 
-BT791- тоже фейк? (аттач)
все-таки вывод в виде "uTorrent 3.4.1 (Fake)" был бы 
более читабельнее, чем "-UT341-%bla%bla%bla"

azLikeClients4: 
  "TT" : "TuoTu", "tT" : "TuoTu",

switch(sign)
{

  break;
case 'SK':
  ret = cli+" "+shChar(str.charAt(3))+"."+shChar(str.charAt(4))+"."+shChar(str.charAt(5))+"."+shChar(str.charAt(6));
  break;
default:

}

if(str.match(/^-UT\d{3}-/))
  ret = "uTorrent "+str.charAt(3)+"."+str.charAt(4)+"."+str.charAt(5)+" (Fake)";
else
if(str.match(/^-BT\d{3}-/))
  ret = "BitTorrent "+str.charAt(3)+"."+str.charAt(4)+"."+str.charAt(5)+" (Fake)";
else

> Так и должно быть.
если руки дойдут - допилю, чтоб считывало.

> Просьба. В дальнейшем, оформляйте, 
пожалуйста, отдельные ошибки - отдельными 
тикетами.
ок.
с патчами/кодом как быть?

Original comment by ad...@vjean.net on 14 Jul 2014 at 2:06

Attachments:

scr3.png

GoogleCodeExporter commented 8 years ago

> -BT791- тоже фейк? (аттач)

С большой степенью вероятности - да.
Здесь читайте. 
https://wiki.theory.org/BitTorrentSpecification#peer_id

> azLikeClients4: 
>  "TT" : "TuoTu", "tT" : "TuoTu",

TuoTu использует трехсимвольную нотацию для 
маркировки версии. И уже находится в секции 
azLikeClients3. Spark, судя по Вашему коду - 
четырехсимвольную. И выносить его под 
отдельный switch смысла нет.

> все-таки вывод в виде "uTorrent 3.4.1 (Fake)"

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

> если руки дойдут - допилю, чтоб считывало.

Успехов.

> с патчами/кодом как быть?

Так же, как здесь.

Original comment by novik65 on 14 Jul 2014 at 3:32

Changed state: Fixed

GoogleCodeExporter commented 8 years ago

> TuoTu использует трехсимвольную нотацию для 
маркировки версии. И уже находится в секции 
azLikeClients3.
только TuoTu сейчас не детектится по "tT"

> Spark, судя по Вашему коду - 
четырехсимвольную. И выносить его под 
отдельный switch смысла нет.
не спорю, лишнее приписал. но Спарк, так же, 
не детектится на данный момент.

> С большой степенью вероятности - да.
> Некорректных попыток подменить id очень 
много. Смысла в том, чтобы доводить до 
пользователя свои догадки - что именно имел 
ввиду нехороший человек, я не вижу.
в то же время, другие клиенты пишут [FAKE], а не 
ид полностью с мусором.

> Так же, как здесь.
т.е. вы, как автор rutorrent, не заинтересованы в 
допиливании кода от "посторонних"?
вопрос был конкретный, что делать с 
подправленным кодом? выкладывать в issues в 
виде патча, обновлять svn или выкладывать на 
форум?

Original comment by ad...@vjean.net on 14 Jul 2014 at 6:57

GoogleCodeExporter commented 8 years ago

> т.е. вы, как автор rutorrent, не заинтересованы 
в допиливании кода от "посторонних"?

Я, как автор ruTorrent, заинтересован в 
качественном коде. Образец Вашего кода в 
данном issue таковым не является. Вы "не 
успеваете проверить, насколько корректно", 
"не мудрите с определением версий", 
запихиваете id не в ту секцию, и т.п. Мне 
проще действительно посмотреть исходник 
transmission.

> вопрос был конкретный, что делать с 
подправленным кодом? выкладывать в issues в 
виде патча, обновлять svn или выкладывать на 
форум?

Ответ столь же конкретный. Делайте то же, 
что и в данном issue. Если хотите.

Original comment by novik65 on 15 Jul 2014 at 5:54

aleksey7905 / rutorrent

несколько ошибок #928