Bug#501757: debbugs: invalid URL generated in summary when bug contains ‘merged-upstream’ information

["Adam D. Barratt" <adam@adam-barratt.org.uk>]

On Fri, 2008-10-10 at 16:33 +1100, Ben Finney wrote:
> When a bug contains ‘merged-upstream’ information as a result of an 
> appropriate ‘forwarded’ command, the resulting bug report page 
> generated has an invalid URL in the page.
> 
> An example: bug#494058 <URL:http://bugs.debian.org/494058> was altered  
> by the control server in message from ‘bts-link-upstream’ 
> <URL:http://bugs.debian.org/cgi-bin/bugreport.cgi?msg=16;bug=494058>. 
> That control message contained the command:
> 
>     forwarded 494058 http://bugzilla.gnome.org/show_bug.cgi?id=547854, merged-upstream: http://bugzilla.gnome.org/show_bug.cgi?id=535127
[...]
> However, the summary information on the bug then contains (HTML):
> 
>    <p>Forwarded to <a href="http://bugzilla.gnome.org/show_bug.cgi?id=547854";>http://bugzilla.gnome.org/show_bug.cgi?id=547854</a>, <a href="merged-upstream:%20http://bugzilla.gnome.org/show_bug.cgi?id=535127";>merged-upstream: http://bugzilla.gnome.org/show_bug.cgi?id=535127</a></p>

The documentation for 'forwarded' reads:

        forwarded /bugnumber/ /address/
        
        Notes that /bugnumber/ has been forwarded to the upstream
        maintainer at /address/.
        
I'm inclined to say that what bts-link is supplying is not simply
"address" and that whilst it would be nice if debbugs handled the above
as two separate links with text separating them, I don't think it's
unreasonable for the HTML generation code to assume that the content of
the forwarded address is actually an address (and therefore
representable as a hyperlink).

Just MHO,

Regards,

Adam