[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#739070: marked as done (qa.debian.org: [new PTS] Links to developer page not working if email contains the symbol +)

Your message dated Thu, 13 Mar 2014 16:02:59 +0100
with message-id <20140313150259.GG18767@x230-buxy.home.ouaza.com>
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 owner@bugs.debian.org
immediately.)


-- 
739070: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=739070
Debian Bug Tracking System
Contact owner@bugs.debian.org 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: emilien+debian@klein.st

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/developer.php?email=emilien+debian@klein.st

The correct link should be:
http://qa.debian.org/developer.php?email=emilien%2Bdebian@klein.st

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 to: