From 185471c84479a23ff1cd2eed751002c986287524 Mon Sep 17 00:00:00 2001 From: Adrien Destugues Date: Thu, 10 Oct 2013 17:25:18 +0200 Subject: [PATCH] HttpRequest: follow 302 redirects by default. --- src/kits/network/libnetapi/HttpRequest.cpp | 10 +++------- 1 file changed, 3 insertions(+), 7 deletions(-) diff --git a/src/kits/network/libnetapi/HttpRequest.cpp b/src/kits/network/libnetapi/HttpRequest.cpp index a49db2f06d..0c2a91d8f9 100644 --- a/src/kits/network/libnetapi/HttpRequest.cpp +++ b/src/kits/network/libnetapi/HttpRequest.cpp @@ -40,7 +40,8 @@ BHttpRequest::BHttpRequest(const BUrl& url, BUrlResult& result, bool ssl, fRequestStatus(kRequestInitialState), fOptHeaders(NULL), fOptInputData(NULL), - fOptInputDataSize(-1) + fOptInputDataSize(-1), + fOptFollowLocation(true) { _ResetOptions(); if (ssl) @@ -299,12 +300,7 @@ BHttpRequest::_ProtocolLoop() if (fResult.StatusCode() == B_HTTP_STATUS_MOVED_PERMANENTLY) { BString locationUrl = fHeaders["Location"]; - // Absolute path - if (locationUrl[0] == '/') - fUrl.SetPath(locationUrl); - // URI - else - fUrl.SetUrlString(locationUrl); + fUrl.Redirect(locationUrl); if (--maxRedirs > 0) { newRequest = true;