Github user SolidWallOfCode commented on the pull request:
https://github.com/apache/trafficserver/pull/145#issuecomment-63715069
I'm a little concerned about the kill_tunnel() in HttpSM::kill_this(). The
problem IMHO is that we may not want to toast off the NetVCs in the tunnel even
if the SM is shutting down because of potential keep alives. There is also the
issue of potentially calling do_io_close() more than once on a VC. I think the
other changes are reasonable.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
