[
https://issues.apache.org/jira/browse/TS-4624?focusedWorklogId=25579&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-25579
]
ASF GitHub Bot logged work on TS-4624:
--------------------------------------
Author: ASF GitHub Bot
Created on: 17/Jul/16 16:07
Start Date: 17/Jul/16 16:07
Worklog Time Spent: 10m
Work Description: Github user yatsukhnenko commented on a diff in the
pull request:
https://github.com/apache/trafficserver/pull/804#discussion_r71084817
--- Diff: proxy/http/HttpTransactHeaders.cc ---
@@ -750,12 +752,10 @@
HttpTransactHeaders::insert_via_header_in_request(HttpTransact::State *s, HTTPHd
via_string += nstrcpy(via_string, s->http_config_param->proxy_hostname);
*via_string++ = '[';
- /* I thought we should use the transaction local outgoing IP address but
- that makes cycle detection (which is the point) unrealiable. We must
- use the same value every time to be sure.
- */
- memcpy(via_string, Machine::instance()->ip_hex_string,
Machine::instance()->ip_hex_string_len);
- via_string += Machine::instance()->ip_hex_string_len;
+ if (uuid != NULL) {
--- End diff --
Reinsurance :) Shoul I delete this check?
Issue Time Tracking
-------------------
Worklog Id: (was: 25579)
Time Spent: 1.5h (was: 1h 20m)
> use the server UUID in the Via header
> -------------------------------------
>
> Key: TS-4624
> URL: https://issues.apache.org/jira/browse/TS-4624
> Project: Traffic Server
> Issue Type: Improvement
> Components: Core
> Reporter: James Peach
> Assignee: Pavlo Yatsukhnenko
> Labels: newbie
> Fix For: 7.0.0
>
> Time Spent: 1.5h
> Remaining Estimate: 0h
>
> [~zwoop] should we use the server UUID in the Via header?
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
