You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@trafficserver.apache.org by so...@apache.org on 2016/05/03 23:05:45 UTC
[trafficserver] branch 6.2.x updated: TS-4406: Fix % tag.
This closes #610.
This is an automated email from the ASF dual-hosted git repository.
sorber pushed a commit to branch 6.2.x
in repository https://git-dual.apache.org/repos/asf/trafficserver.git
The following commit(s) were added to refs/heads/6.2.x by this push:
new 9f74372 TS-4406: Fix %<cqpv> tag. This closes #610.
9f74372 is described below
commit 9f74372ceeecdb03bfd2315575c5edb10aa7f09a
Author: Susan Hinrichs <sh...@draggingnagging.corp.ne1.yahoo.com>
AuthorDate: Mon May 2 15:59:53 2016 +0000
TS-4406: Fix %<cqpv> tag. This closes #610.
(cherry picked from commit 2e12b62b6e5af184ee507bc11746d991a61d8642)
---
proxy/ProxyClientTransaction.cc | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/proxy/ProxyClientTransaction.cc b/proxy/ProxyClientTransaction.cc
index 98c6ab5..835dc5a 100644
--- a/proxy/ProxyClientTransaction.cc
+++ b/proxy/ProxyClientTransaction.cc
@@ -54,7 +54,7 @@ ProxyClientTransaction::new_transaction()
} else {
const char *protocol_str = this->get_protocol_string();
// We don't set the plugin_tag for http, though in future we should probably log http as protocol
- if (strncmp("http", protocol_str, 4)) {
+ if (strlen(protocol_str) != 4 || strncmp("http", protocol_str, 4)) {
current_reader->plugin_tag = protocol_str;
// Since there is no more plugin, there is no plugin id for http/2
// We are copying along the plugin_tag as a standin for protocol name for logging
--
To stop receiving notification emails like this one, please contact
['"commits@trafficserver.apache.org" <co...@trafficserver.apache.org>'].