[
https://issues.apache.org/jira/browse/TS-4457?focusedWorklogId=30032&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-30032
]
ASF GitHub Bot logged work on TS-4457:
--------------------------------------
Author: ASF GitHub Bot
Created on: 30/Sep/16 20:49
Start Date: 30/Sep/16 20:49
Worklog Time Spent: 10m
Work Description: Github user zwoop commented on a diff in the pull
request:
https://github.com/apache/trafficserver/pull/1066#discussion_r81413339
--- Diff: proxy/http/HttpTransactHeaders.cc ---
@@ -812,26 +813,16 @@
HttpTransactHeaders::insert_via_header_in_response(HttpTransact::State *s, HTTPH
}
char *incoming_via = s->via_string;
- int scheme = s->next_hop_scheme;
+ ink_assert(s->state_machine);
--- End diff --
Looking at the code, it seems it's setup in the ::init(), and then never
unset / cleared? If so, do we even need this assert? I guess it doesn't hurt,
but I don't see us do this anywhere else.
Issue Time Tracking
-------------------
Worklog Id: (was: 30032)
Time Spent: 4.5h (was: 4h 20m)
> Via header always reports http1
> -------------------------------
>
> Key: TS-4457
> URL: https://issues.apache.org/jira/browse/TS-4457
> Project: Traffic Server
> Issue Type: Bug
> Reporter: Miles Libbey
> Assignee: Eric Schwartz
> Fix For: 7.1.0
>
> Time Spent: 4.5h
> Remaining Estimate: 0h
>
> When using http2, the Via header says http1.1.
> $ curl -D- -o/dev/null -s "https://docs.trafficserver.apache.org/"; | grep via
> via:http/1.1 ATS (ApacheTrafficServer/6.2.0 [cRs f ])
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
