You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@trafficserver.apache.org by jp...@apache.org on 2013/03/29 19:10:08 UTC
git commit: TS-1790: authproxy should accept 2xx as authorization
success
Updated Branches:
refs/heads/master e1e83f874 -> 52f240943
TS-1790: authproxy should accept 2xx as authorization success
Project: http://git-wip-us.apache.org/repos/asf/trafficserver/repo
Commit: http://git-wip-us.apache.org/repos/asf/trafficserver/commit/52f24094
Tree: http://git-wip-us.apache.org/repos/asf/trafficserver/tree/52f24094
Diff: http://git-wip-us.apache.org/repos/asf/trafficserver/diff/52f24094
Branch: refs/heads/master
Commit: 52f240943554ee5fb3d7bbf82e792517878e426d
Parents: e1e83f8
Author: James Peach <jp...@apache.org>
Authored: Fri Mar 29 11:08:46 2013 -0700
Committer: James Peach <jp...@apache.org>
Committed: Fri Mar 29 11:08:46 2013 -0700
----------------------------------------------------------------------
CHANGES | 3 +++
plugins/experimental/authproxy/authproxy.cc | 4 ++--
2 files changed, 5 insertions(+), 2 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/trafficserver/blob/52f24094/CHANGES
----------------------------------------------------------------------
diff --git a/CHANGES b/CHANGES
index b319e6a..5aee40e 100644
--- a/CHANGES
+++ b/CHANGES
@@ -1,6 +1,9 @@
-*- coding: utf-8 -*-
Changes with Apache Traffic Server 3.3.2
+
+ *) [TS-1790] authproxy should accept 2xx as authorization success
+
*) [TS-1780] Fix Debuntu build with hardening flags
*) [TS-1623, TS-1625] Fixes for logging where the host name was not handled
http://git-wip-us.apache.org/repos/asf/trafficserver/blob/52f24094/plugins/experimental/authproxy/authproxy.cc
----------------------------------------------------------------------
diff --git a/plugins/experimental/authproxy/authproxy.cc b/plugins/experimental/authproxy/authproxy.cc
index 9f16f9a..10bc7b5 100644
--- a/plugins/experimental/authproxy/authproxy.cc
+++ b/plugins/experimental/authproxy/authproxy.cc
@@ -461,8 +461,8 @@ StateAuthProxyCompleteHeaders(AuthRequestContext * auth, void * edata)
status = TSHttpHdrStatusGet(auth->rheader.buffer, auth->rheader.header);
AuthLogDebug("authorization proxy returned status %d", (int)status);
- // Authorize the original request on a 200 response.
- if (status == TS_HTTP_STATUS_OK) {
+ // Authorize the original request on a 2xx response.
+ if (status >= 200 && status < 300) {
return TS_EVENT_IMMEDIATE;
}