Your message dated Thu, 13 Mar 2014 16:02:59 +0100
with message-id <[email protected]>
and subject line Re: Bug#739070: qa.debian.org: [new PTS] Links to developer 
page not working if email contains the symbol +
has caused the Debian Bug report #739070,
regarding qa.debian.org: [new PTS] Links to developer page not working if email 
contains the symbol +
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact [email protected]
immediately.)


-- 
739070: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=739070
Debian Bug Tracking System
Contact [email protected] with problems
--- Begin Message ---
Package: qa.debian.org
Severity: important

Dear Maintainer,

This applies to the new PTS.
My email address contains a plus "+", like this: [email protected]

The Maintainer link to a maintainer's QA page can be invalid:
Example on http://pts.debian.net/pkg/nautilus-image-manipulator the link is:
http://qa.debian.org/[email protected]

The correct link should be:
http://qa.debian.org/developer.php?email=emilien%[email protected]

Reason: the + symbol is not escape/urlencoded, the browser treats it as a 
space. I suppose the fix is to urlencode any link that could contain an email 
address.

I marked this bug as important following the classification "a bug which has a 
major effect on the usability of a package, without rendering it completely 
unusable to everyone.". It obviously works for almost everybody, but not at all 
for me ;)
Note: I had reported a similar issue for the mentors page: 622503

-- System Information:
Debian Release: jessie/sid
  APT prefers unstable
  APT policy: (500, 'unstable'), (500, 'stable'), (1, 'experimental')
Architecture: i386 (i686)

Kernel: Linux 3.12-1-686-pae (SMP w/1 CPU core)
Locale: LANG=en_US.UTF-8, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash

--- End Message ---
--- Begin Message ---
Hello,

On Mon, 10 Mar 2014, Christophe Siraut wrote:
> Hi distro-tracker developers,
> 
> Please review my patch for properly encoding the developer pages URL.
> The second file should not be applied, it just helped me while diving
> into distro-tracker.

Applied. Thank you!

Next time, please send a file generated with "git format-patch" so that I
have a good commit message and proper attribution without extra effort.

Also please keep the bug submitter in copy, otherwise he's not informed
of your work.

Now I committed this to the git repository but I don't plan to update
pts.debian.net so this fix won't be deployed until I complete the setup
on tracker.debian.org (its new home).

Cheers,
-- 
Raphaël Hertzog ◈ Debian Developer

Discover the Debian Administrator's Handbook:
→ http://debian-handbook.info/get/

--- End Message ---

Reply via email to