#270 Follow 302 redirects when checking for upstream sources
Closed: Invalid None Opened 8 years ago by ncoghlan.

Reviewing https://bugzilla.redhat.com/show_bug.cgi?id=1323966 I got the following warning:

WARNING: Cannot download url: https://github.com/micropython/micropython/archive/v1.6.tar.gz

Checking with curl shows this to be a false alarm, based on that URL being a 302 redirect to https://codeload.github.com/micropython/micropython/tar.gz/v1.6


hm... This is, as they use to say, odd. The downloading is done using urllib.FancyUrlOpener, and according to the docs this should transparently handle 302 redirects.

Furthermore, I cannot reproduce this: the specfile new gives a 404 error:

{{{

$ ./try-fedora-review -b 1323966

INFO: Processing bugzilla bug: 1323966
INFO: Getting .spec and .srpm Urls from : 1323966
INFO:   --> SRPM url: http://rpms.jdulaney.com/review/micropython-1.6-3.fc25.src.rpm
INFO:   --> Spec url: http://rpms.jdulaney.com/review/micropython.spec
INFO: Using review directory: /home/mk/tmp/FedoraReview/1323966-micropython
INFO: Downloading .spec and .srpm files
ERROR: 'Error 404 downloading http://rpms.jdulaney.com/review/micropython.spec' (logs in /home/mk  /.cache/fedora-review.log)

}}}

Aye, I ended up using the spec file from the SRPM itself, since the separately hosted one went away.

However, I now suspect this was a transient glitch accessing GitHub, as that later run worked properly.

Login to comment on this ticket.

Metadata