You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@trafficserver.apache.org by br...@apache.org on 2016/04/21 06:28:04 UTC

[trafficserver] branch master updated: TS-4369: Add VIA_ERROR_MOVED_TEMPORARILY. This closes #586

This is an automated email from the ASF dual-hosted git repository.

briang pushed a commit to branch master
in repository https://git-dual.apache.org/repos/asf/trafficserver.git

The following commit(s) were added to refs/heads/master by this push:
       new  af7832d   TS-4369: Add VIA_ERROR_MOVED_TEMPORARILY. This closes #586
af7832d is described below

commit af7832d229d6746edfd5010ab013fce0bbf20c21
Author: Brian Geffon <br...@apache.org>
AuthorDate: Wed Apr 20 21:26:59 2016 -0700

    TS-4369: Add VIA_ERROR_MOVED_TEMPORARILY. This closes #586
---
 cmd/traffic_via/traffic_via.cc | 1 +
 doc/appendices/faq.en.rst      | 1 +
 proxy/http/HttpTransact.cc     | 2 +-
 proxy/http/HttpTransact.h      | 1 +
 proxy/http/README.via          | 3 +++
 5 files changed, 7 insertions(+), 1 deletion(-)

diff --git a/cmd/traffic_via/traffic_via.cc b/cmd/traffic_via/traffic_via.cc
index 82b9ad3..c816f5a 100644
--- a/cmd/traffic_via/traffic_via.cc
+++ b/cmd/traffic_via/traffic_via.cc
@@ -176,6 +176,7 @@ standardViaLookup(char flag)
     viaTable->viaData[(unsigned char)'N'] = "no error";
     viaTable->viaData[(unsigned char)'F'] = "request forbidden";
     viaTable->viaData[(unsigned char)'R'] = "cache read error";
+    viaTable->viaData[(unsigned char)'M'] = "moved temporarily";
     viaTable->viaData[(unsigned char)' '] = "unknown";
     break;
   default:
diff --git a/doc/appendices/faq.en.rst b/doc/appendices/faq.en.rst
index d8646c5..4951403 100644
--- a/doc/appendices/faq.en.rst
+++ b/doc/appendices/faq.en.rst
@@ -229,6 +229,7 @@ F     request forbidden
 H     header syntax unacceptable
 N     no error
 R     cache read error
+M     moved temporarily
 S     server related error
 T     connection timed out
 ===== ==========================
diff --git a/proxy/http/HttpTransact.cc b/proxy/http/HttpTransact.cc
index 867ad2a..aa3f143 100644
--- a/proxy/http/HttpTransact.cc
+++ b/proxy/http/HttpTransact.cc
@@ -8135,7 +8135,7 @@ HttpTransact::build_error_response(State *s, HTTPStatus status_code, const char
     SET_VIA_STRING(VIA_ERROR_TYPE, VIA_ERROR_DNS_FAILURE);
     break;
   case HTTP_STATUS_MOVED_TEMPORARILY:
-    SET_VIA_STRING(VIA_ERROR_TYPE, VIA_ERROR_SERVER);
+    SET_VIA_STRING(VIA_ERROR_TYPE, VIA_ERROR_MOVED_TEMPORARILY);
     break;
   case HTTP_STATUS_PROXY_AUTHENTICATION_REQUIRED:
     SET_VIA_STRING(VIA_CLIENT_REQUEST, VIA_CLIENT_ERROR);
diff --git a/proxy/http/HttpTransact.h b/proxy/http/HttpTransact.h
index 5173229..ad9acea 100644
--- a/proxy/http/HttpTransact.h
+++ b/proxy/http/HttpTransact.h
@@ -183,6 +183,7 @@ enum ViaString_t {
   VIA_ERROR_SERVER = 'S',
   VIA_ERROR_TIMEOUT = 'T',
   VIA_ERROR_CACHE_READ = 'R',
+  VIA_ERROR_MOVED_TEMPORARILY = 'M',
   //
   // Now the detailed stuff
   //
diff --git a/proxy/http/README.via b/proxy/http/README.via
index c5c954f..04c4a81 100644
--- a/proxy/http/README.via
+++ b/proxy/http/README.via
@@ -12,6 +12,7 @@
          A    in cache, not acceptable
          S    in cache, stale
          H    in cache, fresh
+         R    in cache, fresh RAM hit
      
       server stuff
          E    error in response
@@ -35,6 +36,8 @@
          D    dns failure
          F    request forbidden
          H    header syntax unacceptable
+         M    moved temporarily
+         R    cache read error
          S    server related error
          T    connection timed out
      

-- 
To stop receiving notification emails like this one, please contact
['"commits@trafficserver.apache.org" <co...@trafficserver.apache.org>'].