[
https://issues.apache.org/jira/browse/TS-4694?focusedWorklogId=25970&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-25970
]
ASF GitHub Bot logged work on TS-4694:
--------------------------------------
Author: ASF GitHub Bot
Created on: 25/Jul/16 06:26
Start Date: 25/Jul/16 06:26
Worklog Time Spent: 10m
Work Description: GitHub user masaori335 opened a pull request:
https://github.com/apache/trafficserver/pull/825
TS-4694: Some refactoring after SPDY is removed
- Remove PluginIdentity class from base classes of Http2ClientSession
- Add get_protocol_string() to ProxyClientSession to identify if the
session is HTTP/2 or HTTP/1.x
- Add "client_protocol" to HttpSM to track client protocol versions
- Drop HTTP/0.9 support from cqpv (HTTP/0.9 is already dropped by TS-3327)
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/masaori335/trafficserver ts-4694
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/trafficserver/pull/825.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #825
----
commit 4a9c564ff7727cab7929a29e38f5ff5ae0c62c62
Author: Masaori Koshiba <[email protected]>
Date: 2016-07-22T06:17:50Z
TS-4694: Some refactoring after SPDY is removed
- Remove PluginIdentity class from base classes of Http2ClientSession
- Add get_protocol_string() to ProxyClientSession to identify if the
session is HTTP/2 or HTTP/1.x
- Add "client_protocol" to HttpSM to track client protocol versions
- Drop HTTP/0.9 support from cqpv (HTTP/0.9 is already dropped by TS-3327)
----
Issue Time Tracking
-------------------
Worklog Id: (was: 25970)
Time Spent: 10m
Remaining Estimate: 0h
> Some refactoring after SPDY is removed
> --------------------------------------
>
> Key: TS-4694
> URL: https://issues.apache.org/jira/browse/TS-4694
> Project: Traffic Server
> Issue Type: Improvement
> Components: Core, HTTP
> Reporter: Leif Hedstrom
> Assignee: Susan Hinrichs
> Fix For: 7.0.0
>
> Time Spent: 10m
> Remaining Estimate: 0h
>
> There's a place in the code with a comment about refactoring this once we
> remove SPDY (see below). I'm filing this separates from TS-4311, because I'm
> not sure what this refactoring entails. [~shinrich] had some ideas around
> this code :).
> {code}
> void
> ProxyClientTransaction::new_transaction()
> {
> ink_assert(current_reader == NULL);
> // Defensive programming, make sure nothing persists across
> // connection re-use
> ink_release_assert(parent != NULL);
> current_reader = HttpSM::allocate();
> current_reader->init();
> DebugHttpTxn("[%" PRId64 "] Starting transaction %d using sm [%" PRId64
> "]", parent->connection_id(),
> parent->get_transact_count(), current_reader->sm_id);
> // This is a temporary hack until we get rid of SPDY and can use virutal
> methods entirely
> // to track protocol.
> PluginIdentity *pi = dynamic_cast<PluginIdentity *>(this->get_netvc());
> if (pi) {
> current_reader->plugin_tag = pi->getPluginTag();
> current_reader->plugin_id = pi->getPluginId();
> } 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 (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
> // and to detect a case in HttpTransaction (TS-3954)
> }
> }
> current_reader->attach_client_session(this, sm_reader);
> }
> {code}
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
