Installation script fails with "curl"

Bug #1470110 reported by Yair Fried
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Rally
Fix Released
Medium
Chris St. Pierre

Bug Description

Using the curl command the script tries to download from https://pypi.python.org/simple/setuptools/../../packages/source/s/setuptools/setuptools-9.1.tar.gz

$ curl -v --insecure -L -s -o setuptools-9.1.tar.gz https://pypi.python.org/simple/setuptools/../../packages/source/s/setuptools/setuptools-9.1.tar.gz

this works with "wget".

Seems like this type of input doesn't work for "curl".

Note: this was done on rhel-7.1 guest image

Yair Fried (yfried)
description: updated
Revision history for this message
Pavel Boldin (pboldin) wrote :

$ curl --version
please

Revision history for this message
Yair Fried (yfried) wrote :

curl 7.29.0

Revision history for this message
Chris St. Pierre (stpierre) wrote :

It looks like wget resolves the ../../ on the client side, while curl does this:

    GET /simple/setuptools/../../packages/source/s/setuptools/setuptools-9.1.tar.gz HTTP/1.1

Although that's the URL we get back from PyPI, it's not valid unless the client fixes up the ../'s itself.

The easy fix (and IMO the correct one) is to just use wget. Trying to fix up that URL ourselves in bash is fraught.

Changed in rally:
assignee: nobody → Chris St. Pierre (stpierre)
Changed in rally:
status: New → In Progress
Revision history for this message
Pavel Boldin (pboldin) wrote :

@stpierre
sed -e 's,[\w-_.]+/..,,g'

Revision history for this message
Pavel Boldin (pboldin) wrote :

My bad, does not work.

But wget seems to be an overkill IMO. Probably curl has an option to resolve these.

Revision history for this message
Chris St. Pierre (stpierre) wrote :

The script already uses wget or curl, so this is just a removal of the curl logic. So it makes the script simpler, and it works.

I couldn't find a curl option to handle these URLs, and searching for this problem is devilish hard.

Changed in rally:
status: In Progress → Fix Committed
Changed in rally:
milestone: none → 0.1.0
Changed in rally:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.