[GitHub] trafficserver issue #1375: Incorrectly freeing Http1ClientSession setting up...

2017-01-26 Thread SolidWallOfCode
Github user SolidWallOfCode commented on the issue:

https://github.com/apache/trafficserver/pull/1375
  
Ship it.


---
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.
---


[GitHub] trafficserver issue #1375: Incorrectly freeing Http1ClientSession setting up...

2017-01-26 Thread shinrich
Github user shinrich commented on the issue:

https://github.com/apache/trafficserver/pull/1375
  
The client session gets cleaned up as normal after the response is sent.  
The problem is the original code was cleaning up the client session via the 
kill tunnel before sending the error.


---
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.
---


[GitHub] trafficserver issue #1375: Incorrectly freeing Http1ClientSession setting up...

2017-01-26 Thread SolidWallOfCode
Github user SolidWallOfCode commented on the issue:

https://github.com/apache/trafficserver/pull/1375
  
If the client session is cleared from the tunnel by `reset` how does it 
eventually get cleaned up?


---
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.
---


[GitHub] trafficserver issue #1375: Incorrectly freeing Http1ClientSession setting up...

2017-01-25 Thread zwoop
Github user zwoop commented on the issue:

https://github.com/apache/trafficserver/pull/1375
  
Is this a 7.1.x candidate?


---
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.
---


[GitHub] trafficserver issue #1375: Incorrectly freeing Http1ClientSession setting up...

2017-01-25 Thread atsci
Github user atsci commented on the issue:

https://github.com/apache/trafficserver/pull/1375
  
Linux build *successful*! See 
https://ci.trafficserver.apache.org/job/Github-Linux/1286/ for details.
 



---
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.
---


[GitHub] trafficserver issue #1375: Incorrectly freeing Http1ClientSession setting up...

2017-01-25 Thread atsci
Github user atsci commented on the issue:

https://github.com/apache/trafficserver/pull/1375
  
FreeBSD build *successful*! See 
https://ci.trafficserver.apache.org/job/Github-FreeBSD/1392/ for details.
 



---
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.
---