Bug#497001: rtorrent ignores urls starting with Http

2012-03-03 Thread Benoît Knecht
Marc Lehmann wrote:
> [...]
> 
> It shouldn't be that 13 years after it has been superseded you still quote
> from that rfc, especially not to ignore valid bug reports. You are wasting
> mine and your time with this that could have been used to improve debian.

Right. I promise I won't waste my time with this anymore.

-- 
Benoît Knecht



--
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org



Bug#497001: rtorrent ignores urls starting with Http

2012-03-02 Thread Marc Lehmann
On Fri, Mar 02, 2012 at 11:41:16AM +0100, Benoît Knecht 
 wrote:
> Well, that's not entirely accurate. According to RFC 1738 [1], section
> 2.1:

*sigh*, know your rfcs: Your rfc 1738 is obsolete for well over a decade
now.

Let's jump to rfc3986 from 2005, which defines that part of the syntax:

   Scheme names consist of a sequence of characters beginning with a
   letter and followed by any combination of letters, digits, plus ("+"),
   period ("."), or hyphen ("-"). Although schemes are case- insensitive,
   the canonical form is lowercase and documents that specify schemes must
   do so with lowercase letters.

You can read the whole RFC, scheme names *really* are case-insensitive.

> So torrent files using "Http" are actually invalid (and I've never

When bittorrent was invented, that part of the rfc had long been superseded
by rfc2396.

> but that's a "should", not a "must". So I'm downgrading the severity to

Know your rfcs: "SHOULD" has a meaning, and not the meaning you think it
has:

   3. SHOULD This word, or the adjective "RECOMMENDED", mean that there may
  exist valid reasons in particular circumstances to ignore a particular
  item, but the full implications must be understood and carefully
  weighed before choosing a different course.

There is no valid reason to ignore the rfc and not implement uri syntax
properly that would even *remotely* outweigh the inconvenience of parsing
the scheme as per rfc.

It is quite obvious that you didn't carefully understand the implications
or weighed it against the effort of fixing the bug. But thats what
"SHOULD" means in an RFC.

Confusing "should" with "I can just ignore this" is common on the
internets (which is why there is now an RFC to define these terms), but
that doesn't make it right.

> "minor", and I'd suggest the maintainer closes it as "wontfix".
> [1] http://www.ietf.org/rfc/rfc1738.txt

It shouldn't be that 13 years after it has been superseded you still quote
from that rfc, especially not to ignore valid bug reports. You are wasting
mine and your time with this that could have been used to improve debian.

Next time, maybe check wikipedia - wikipedia both gets the
case-insentiveness right, AND tells you that your rfc has been superseded:
http://en.wikipedia.org/wiki/URI_scheme

Or, for utmost professionality, you can check the rfc online using the
ietf tools server:

http://tools.ietf.org/html/rfc1738

Which tells you this:

   Obsoleted by: 4248, 4266 PROPOSED STANDARD
   Updated by: 1808, 2368, 2396, 3986, 6196, 6270 Errata Exist

-- 
The choice of a   Deliantra, the free code+content MORPG
  -==- _GNU_  http://www.deliantra.net
  ==-- _   generation
  ---==---(_)__  __   __  Marc Lehmann
  --==---/ / _ \/ // /\ \/ /  schm...@schmorp.de
  -=/_/_//_/\_,_/ /_/\_\



--
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org



Bug#497001: rtorrent ignores urls starting with Http

2012-03-02 Thread Benoît Knecht
severity 497001 minor
thanks

Marc Lehmann wrote:
> rtorrent ignores tracker urls starting with "Http" even thought it
> supports the http protocol.
> 
> editing the torrent into a lowercase "http" makes it detect the tracker.
> 
> background: uri scheme names are case insensitive, and worse, actual
> torrent files do contain tracker urls with mixed and upper case
> scheme parts (i don't know if bittorrent meta files have additional
> specificaitons that force scheme names to lowercase, but I doubt it).

Well, that's not entirely accurate. According to RFC 1738 [1], section
2.1:

  Scheme names consist of a sequence of characters. The lower case
  letters "a"--"z", digits, and the characters plus ("+"), period ("."),
  and hyphen ("-") are allowed.

So torrent files using "Http" are actually invalid (and I've never
encountered one of these). The same section goes on like this, though:

  For resiliency, programs interpreting URLs should treat upper case
  letters as equivalent to lower case in scheme names (e.g., allow
  "HTTP" as well as "http").

but that's a "should", not a "must". So I'm downgrading the severity to
"minor", and I'd suggest the maintainer closes it as "wontfix".

[1] http://www.ietf.org/rfc/rfc1738.txt

Cheers,

-- 
Benoît Knecht



--
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org



Bug#497001: rtorrent ignores urls starting with Http

2008-08-28 Thread Marc Lehmann
Package: rtorrent
Version: 0.7.9-2+b2
Severity: normal


rtorrent ignores tracker urls starting with "Http" even thought it
supports the http protocol.

editing the torrent into a lowercase "http" makes it detect the tracker.

background: uri scheme names are case insensitive, and worse, actual
torrent files do contain tracker urls with mixed and upper case
scheme parts (i don't know if bittorrent meta files have additional
specificaitons that force scheme names to lowercase, but I doubt it).

-- System Information:
Debian Release: lenny/sid
  APT prefers unstable
  APT policy: (500, 'unstable'), (500, 'testing'), (500, 'stable'), (1, 
'experimental')
Architecture: amd64 (x86_64)

Kernel: Linux 2.6.24-1-amd64 (SMP w/4 CPU cores)
Locale: LANG=C, LC_CTYPE=de_DE.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/bash

Versions of packages rtorrent depends on:
hi  lib 2.7-12   GNU C Library: Shared libraries
ii  lib 1.39+1.40-WIP-2006.11.14+dfsg-2etch1 common error description library
ii  lib 7.18.2-5 Multi-protocol file transfer libra
ii  lib 1:4.3.1-8GCC support library
ii  lib 0.6.5-1  GNU libidn library, implementation
ii  lib 1.6.dfsg.3-2 MIT Kerberos runtime libraries
ii  lib 2.4.7-5  OpenLDAP libraries
ii  lib 5.6+20080531-1   Shared libraries for terminal hand
ii  lib 2.0.18-2 type-safe Signal Framework for C++
ii  lib 0.18-1   SSH2 client-side library
ii  lib 0.9.8g-10.1  SSL shared libraries
ii  lib 4.3.1-8  The GNU Standard C++ Library v3
ii  lib 0.11.9-1.1   a C++ BitTorrent library
ii  lib 1.06.27-1A lightweight RPC library based on
ii  zli 1:1.2.3.3.dfsg-12compression library - runtime

rtorrent recommends no packages.

-- no debconf information



-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]