diff options
| author | Eitan Adler <eadler@FreeBSD.org> | 2012-10-22 03:00:10 +0000 |
|---|---|---|
| committer | Eitan Adler <eadler@FreeBSD.org> | 2012-10-22 03:00:10 +0000 |
| commit | c4fa1489ec4f303a73ea1da82583ff61d2d13e95 (patch) | |
| tree | 5d18e5eb8a70ff62d5997b32b56717e2073ae80d | |
| parent | e6c0e200f4f26cffc422f0d42323e2713e06322d (diff) | |
Notes
| -rw-r--r-- | lib/libfetch/http.c | 14 | ||||
| -rw-r--r-- | lib/libfetch/http.errors | 1 |
2 files changed, 13 insertions, 2 deletions
diff --git a/lib/libfetch/http.c b/lib/libfetch/http.c index d10e6445e111..a1166bc4a2d7 100644 --- a/lib/libfetch/http.c +++ b/lib/libfetch/http.c @@ -104,6 +104,7 @@ __FBSDID("$FreeBSD$"); #define HTTP_SEE_OTHER 303 #define HTTP_NOT_MODIFIED 304 #define HTTP_TEMP_REDIRECT 307 +#define HTTP_PERM_REDIRECT 308 #define HTTP_NEED_AUTH 401 #define HTTP_NEED_PROXY_AUTH 407 #define HTTP_BAD_RANGE 416 @@ -1524,8 +1525,7 @@ http_request(struct url *URL, const char *op, struct url_stat *us, /* try the provided URL first */ url = URL; - /* if the A flag is set, we only get one try */ - n = noredirect ? 1 : MAX_REDIRECT; + n = MAX_REDIRECT; i = 0; e = HTTP_PROTOCOL_ERROR; @@ -1772,6 +1772,16 @@ http_request(struct url *URL, const char *op, struct url_stat *us, case hdr_location: if (!HTTP_REDIRECT(conn->err)) break; + /* + * if the A flag is set, we don't follow + * temporary redirects. + */ + if (noredirect && + conn->err != HTTP_MOVED_PERM && + conn->err != HTTP_PERM_REDIRECT) { + n = 1; + break; + } if (new) free(new); if (verbose) diff --git a/lib/libfetch/http.errors b/lib/libfetch/http.errors index 1f38574d463a..e5389fefcc56 100644 --- a/lib/libfetch/http.errors +++ b/lib/libfetch/http.errors @@ -18,6 +18,7 @@ 304 OK Not Modified 305 INFO Use Proxy 307 MOVED Temporary Redirect +308 MOVED Permanent Redirect 400 PROTO Bad Request 401 AUTH Unauthorized 402 AUTH Payment Required |
