Ubuntu
devscripts package

rmadison needs update to new rmadison.cgi url

Bug #399891 reported by Marian Sigler on 2009-07-15

This bug affects 1 person

	Status	Importance	Assigned to
devscripts (Ubuntu)	Fix Released	High	Colin Watson
Hardy	Fix Released	High	Unassigned
Jaunty	Won't Fix	High	Unassigned
Karmic	Fix Released	High	Colin Watson

Bug Description

Binary package hint: devscripts

Recently, the url of the rmadison.cgi script must have changed (from people.ubuntu.com to people.canonical.com). This in combination with a brain damage in curl which causes it to not follow redirects per default makes rmadison display a html page with a link to the new url.

Fix (attached):
- replace people.ubuntu.com by people.canonical.com
- replace curl by curl -L to avoid such bugs in the future
(the patch is against the hardy version)

Tags:

Related branches

lp:ubuntu/karmic/devscripts

lp:ubuntu/hardy-proposed/devscripts

Revision history for this message

Marian Sigler (maix42) wrote on 2009-07-15:

Patch for rmadison (changes url and adds -L) Edit (881 bytes, text/plain)

Revision history for this message

Marian Sigler (maix42) wrote on 2009-07-15:

I've tested with jaunty and karmic, the bug exists there, too. (In karmic one hunk of the patch fails, I don't know why, but you may apply it by hand, the idea is clear I think :-) )

In case you cannot reproduce it: Ensure that curl is installed, if it is not wget is used (which *wohoo!* is able to follow redirects per default)

Revision history for this message

Colin Watson (cjwatson) wrote on 2009-07-15:

Thanks. I'm applying a slightly more extensive version, but basically the same idea.

I agree that this ought to be fixed in hardy and jaunty as well, but don't have time right now ...

Changed in devscripts (Ubuntu Karmic):
assignee:	nobody → Colin Watson (cjwatson)
importance:	Undecided → High
status:	New → In Progress

Colin Watson (cjwatson) on 2009-07-15

Changed in devscripts (Ubuntu Hardy):
importance:	Undecided → High
status:	New → Triaged
Changed in devscripts (Ubuntu Jaunty):
importance:	Undecided → High
status:	New → Triaged

Revision history for this message

Launchpad Janitor (janitor) wrote on 2009-07-15:

This bug was fixed in the package devscripts - 2.10.48ubuntu2

---------------
devscripts (2.10.48ubuntu2) karmic; urgency=low

  [ Marian Sigler ]
  * Update rmadison's Ubuntu URL to cope with people.ubuntu.com ->
    people.canonical.com; invoke curl in such a way as to follow redirects
    by default (LP: #399891).

-- Colin Watson <email address hidden> Wed, 15 Jul 2009 22:20:54 +0100

Changed in devscripts (Ubuntu Karmic):
status:	In Progress → Fix Released

Revision history for this message

Onkar Shinde (onkarshinde) wrote on 2009-07-16:

hardy.debdiff Edit (4.7 KiB, text/plain)

Please find attached the debdiff for hardy. I have tested it on fully updated hardy system.

Revision history for this message

Onkar Shinde (onkarshinde) wrote on 2009-07-17:

TEST CASE:
$ rmadison electric

Expected results:
Information about electric package should be shown.

Actual results:
Contents of HTML page, which contains the new URL for madison, are shown.

SRU: minimal patch for Hardy is attached.

Nominating for SRU, fulfills: (1) have an obviously safe patch and (2) affect an application rather than critical infrastructure packages

The fix is copied from karmic version 2.10.48ubuntu2 and has been verified in Hardy.

Revision history for this message

Marian Sigler (maix42) wrote on 2009-07-17:

BTW, I did only not nominate this bug for intrepid because I could not test it there (I have jaunty and karmic in VMs), I think it should be fixed there too.

Revision history for this message

Martin Pitt (pitti) wrote on 2009-07-29:

Accepted devscripts into hardy-proposed, the package will build now and be available in a few hours. Please test and give feedback here. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Thank you in advance!

Changed in devscripts (Ubuntu Hardy):
status:	Triaged → Fix Committed
tags:	added: verification-needed

Revision history for this message

Marian Sigler (maix42) wrote on 2009-07-30:

Thanks Martin. I've tested the hardy-backports package and it works.

However they have changed the server configuration: On people.ubuntu.com there is no longer a redirect to people.canonical.com but the output is displayed directly, so this fix is actually not important anymore (It should be applied nevertheless I think: if there is the -L switch there won't be such problems in the future if something changes again. But maybe for jaunty and intrepid it doesn't need to be fixed)

Revision history for this message

Martin Pitt (pitti) wrote on 2009-07-30:

#10

Right, thanks for testing. I'll set the jaunty task to "wontfix" then.

tags:	added: verification-done removed: verification-needed
Changed in devscripts (Ubuntu Jaunty):
status:	Triaged → Won't Fix

Revision history for this message

Launchpad Janitor (janitor) wrote on 2009-08-17:

#11

This bug was fixed in the package devscripts - 2.10.11ubuntu5.8.04.3

---------------
devscripts (2.10.11ubuntu5.8.04.3) hardy-proposed; urgency=low

  [ Marian Sigler ]
  * Update rmadison's Ubuntu URL to cope with people.ubuntu.com ->
    people.canonical.com; invoke curl in such a way as to follow redirects
    by default (LP: #399891).

-- Onkar Shinde <email address hidden> Thu, 16 Jul 2009 13:59:23 +0530