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

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



Package: debbugs
Severity: normal

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

In response, the bug page for bug#494058 now contains:

    Forwarded-to-address changed from http://bugzilla.gnome.org/show_bug.cgi?id=547854 to http://bugzilla.gnome.org/show_bug.cgi?id=547854, merged-upstream: http://bugzilla.gnome.org/show_bug.cgi?id=535127.

All those URLs are correctly linked in the HTML code.

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 URL in the href of the second <a> element,
“merged-upstream:%20http://bugzilla.gnome.org/show_bug.cgi?id=535127”; 
is invalid.

This should probably instead be:

   <p>Forwarded to <a href="http://bugzilla.gnome.org/show_bug.cgi?id=547854";>http://bugzilla.gnome.org/show_bug.cgi?id=547854</a>, merged-upstream: <a href="http://bugzilla.gnome.org/show_bug.cgi?id=535127";>http://bugzilla.gnome.org/show_bug.cgi?id=535127</a></p>

-- 
 \         “What is it that makes a complete stranger dive into an icy |
  `\   river to save a solid gold baby? Maybe we'll never know.” —Jack |
_o__)                                                           Handey |
Ben Finney <ben@benfinney.id.au>

Attachment: signature.asc
Description: Digital signature


Reply to: