rgeissert / http-redirector

Debian mirrors HTTP redirector
http://httpredir.debian.org/
88 stars 8 forks source link

consider redirecting with status code 302 #1

Closed aragilar closed 12 years ago

aragilar commented 12 years ago

When I try using http://http.debian.net/ with apt-cacher-ng, I receive 307 errors.

rgeissert commented 12 years ago

Thanks for your report. 307 is the HTTP code to indicate a temporary redirection. That's how the redirector works.

A quick look at apt-cacher-ng's code indicates it does store the destination, but at a later point it only seems to handle 301 and 302 redirections. While I could silently make it a 301 when apt-cacher-ng is used, I would really like to see it handling 307 correctly. Could you please file a bug report against apt-cacher-ng?

aragilar commented 12 years ago

Bug filed - http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=661971

rgeissert commented 12 years ago

I'm considering switching to a 302 redirection, since the difference between 302, 303 and 307 doesn't affect the redirector. The benefit is that it comes from HTTP 1.0 and as such it should be properly supported by other programs.