You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@trafficserver.apache.org by su...@apache.org on 2014/11/11 20:34:35 UTC

trafficserver git commit: [TS-2812]: Adding a comment to optimize the hook to send_request_hdr_hook

Repository: trafficserver
Updated Branches:
  refs/heads/master 989162895 -> 177e8da43


[TS-2812]: Adding a comment to optimize the hook to send_request_hdr_hook


Project: http://git-wip-us.apache.org/repos/asf/trafficserver/repo
Commit: http://git-wip-us.apache.org/repos/asf/trafficserver/commit/177e8da4
Tree: http://git-wip-us.apache.org/repos/asf/trafficserver/tree/177e8da4
Diff: http://git-wip-us.apache.org/repos/asf/trafficserver/diff/177e8da4

Branch: refs/heads/master
Commit: 177e8da431f073606b22ceb740b1009308f2cc23
Parents: 9891628
Author: Sudheer Vinukonda <su...@yahoo-inc.com>
Authored: Tue Nov 11 19:33:46 2014 +0000
Committer: Sudheer Vinukonda <su...@yahoo-inc.com>
Committed: Tue Nov 11 19:33:46 2014 +0000

----------------------------------------------------------------------
 plugins/experimental/header_normalize/header_normalize.cc | 4 ++++
 1 file changed, 4 insertions(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/trafficserver/blob/177e8da4/plugins/experimental/header_normalize/header_normalize.cc
----------------------------------------------------------------------
diff --git a/plugins/experimental/header_normalize/header_normalize.cc b/plugins/experimental/header_normalize/header_normalize.cc
index 3afe633..cfe0d14 100644
--- a/plugins/experimental/header_normalize/header_normalize.cc
+++ b/plugins/experimental/header_normalize/header_normalize.cc
@@ -5,6 +5,10 @@
 // interworking issues with legacy origins not supporting lower case headers
 // required by protocols such as spdy/http2 etc.
 //
+// Note that the plugin currently uses READ_REQUEST_HDR_HOOK to camel-case
+// the headers. As an optimization, it can be changed to SEND_REQUEST_HDR_HOOK 
+// so that it only converts, if/when the request is being sent to the origin
+//
 // To use this plugin, configure a remap.config rule like
 //
 //   map http://foo.com http://bar.com @plugin=header_normalize.so