You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@trafficserver.apache.org by zw...@apache.org on 2015/05/11 23:59:57 UTC

[1/2] trafficserver git commit: TS-3565 Only start up the continuation from DoRemap if the request has a Range: header

Repository: trafficserver
Updated Branches:
  refs/heads/master 02bfcb3eb -> dcb5baa08


TS-3565 Only start up the continuation from DoRemap if the request has a Range: header


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

Branch: refs/heads/master
Commit: 51cbfa902a4726f72003bc943641dc5a9b596d0a
Parents: 02bfcb3
Author: Leif Hedstrom <zw...@apache.org>
Authored: Tue Apr 28 18:17:06 2015 -0600
Committer: Leif Hedstrom <zw...@apache.org>
Committed: Mon May 11 15:58:46 2015 -0600

----------------------------------------------------------------------
 .../background_fetch/background_fetch.cc        | 21 +++++++++++++++-----
 1 file changed, 16 insertions(+), 5 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/trafficserver/blob/51cbfa90/plugins/experimental/background_fetch/background_fetch.cc
----------------------------------------------------------------------
diff --git a/plugins/experimental/background_fetch/background_fetch.cc b/plugins/experimental/background_fetch/background_fetch.cc
index e5cdb6b..780de5c 100644
--- a/plugins/experimental/background_fetch/background_fetch.cc
+++ b/plugins/experimental/background_fetch/background_fetch.cc
@@ -640,13 +640,24 @@ TSRemapDoRemap(void *ih, TSHttpTxn txnp, TSRemapRequestInfo * /* rri */)
     return TSREMAP_NO_REMAP;
   }
 
-  BgFetchConfig *config = static_cast<BgFetchConfig *>(ih);
+  TSMBuffer bufp;
+  TSMLoc req_hdrs;
 
-  config->acquire(); // Inc refcount
-  TSHttpTxnHookAdd(txnp, TS_HTTP_READ_RESPONSE_HDR_HOOK, config->getCont());
-  TSHttpTxnHookAdd(txnp, TS_HTTP_TXN_CLOSE_HOOK, config->getCont());
+  if (TS_SUCCESS == TSHttpTxnClientReqGet(txnp, &bufp, &req_hdrs)) {
+    TSMLoc field_loc = TSMimeHdrFieldFind(bufp, req_hdrs, TS_MIME_FIELD_RANGE, TS_MIME_LEN_RANGE);
+
+    if (field_loc) {
+      BgFetchConfig *config = static_cast<BgFetchConfig *>(ih);
 
-  TSDebug(PLUGIN_NAME, "background fetch TSRemapDoRemap...");
+      config->acquire(); // Inc refcount
+      TSHttpTxnHookAdd(txnp, TS_HTTP_READ_RESPONSE_HDR_HOOK, config->getCont());
+      TSHttpTxnHookAdd(txnp, TS_HTTP_TXN_CLOSE_HOOK, config->getCont());
+
+      TSDebug(PLUGIN_NAME, "background fetch TSRemapDoRemap...");
+      TSHandleMLocRelease(bufp, req_hdrs, field_loc);
+    }
+    TSHandleMLocRelease(bufp, TS_NULL_MLOC, req_hdrs);
+  }
 
   return TSREMAP_NO_REMAP;
 }


[2/2] trafficserver git commit: Added TS-3565, fixed punctuation.

Posted by zw...@apache.org.
Added TS-3565, fixed punctuation.


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

Branch: refs/heads/master
Commit: dcb5baa0898b9189f01f454dbbb6c6bcabed9522
Parents: 51cbfa9
Author: Leif Hedstrom <zw...@apache.org>
Authored: Mon May 11 15:58:39 2015 -0600
Committer: Leif Hedstrom <zw...@apache.org>
Committed: Mon May 11 15:58:47 2015 -0600

----------------------------------------------------------------------
 CHANGES | 13 ++++++++-----
 1 file changed, 8 insertions(+), 5 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/trafficserver/blob/dcb5baa0/CHANGES
----------------------------------------------------------------------
diff --git a/CHANGES b/CHANGES
index 4e91d6f..073c070 100644
--- a/CHANGES
+++ b/CHANGES
@@ -1,23 +1,26 @@
                                                          -*- coding: utf-8 -*-
 Changes with Apache Traffic Server 6.0.0
 
+  *) [TS-3565] Only start up the continuation from DoRemap if the request has
+   a Range: header.
+
   *) [TS-3562] background_fetch: refactor some code, elimiante the std::map
    for rules, split things out into manageable sub-modules.
 
   *) [TS-3592] Set cacheurl default config location to config directory.
 
-  *) [TS-3590]: Mark H2 fetch streams as non-internal requests
+  *) [TS-3590]: Mark H2 fetch streams as non-internal requests.
 
   *) [TS-3370] header_rewrite plugin uses dead continuation.
 
-  *) [TS-3588]: Fix continuation leak in background_fetch in remap mode
+  *) [TS-3588]: Fix continuation leak in background_fetch in remap mode.
 
   *) [TS-3029]: Fix spdy related seg fault in production, due to resetting
-                req->fetch_sm to null
+   req->fetch_sm to null.
 
-  *) [TS-2513] Missing SOCKS documentation
+  *) [TS-2513] Missing SOCKS documentation.
 
-  *) [TS-3582] Add @in_ip to remap filters
+  *) [TS-3582] Add @in_ip to remap filters.
 
   *) [TS-3484] Make inactivity_cop frequency (schedule) configurable,
    proxy.config.net.inactivity_check_frequency.