[GitHub] trafficserver issue #712: TS-4523 - Add pause functionality to the Transform...

2016-10-07 Thread ushachar
Github user ushachar commented on the issue:

https://github.com/apache/trafficserver/pull/712
  
@jpeach / @SolidWallOfCode  - Since there were a couple of concerns about 
this API in the beginning could one of you ack this as well?


---
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 #712: TS-4523 - Add pause functionality to the Transform...

2016-10-07 Thread davidbz
Github user davidbz commented on the issue:

https://github.com/apache/trafficserver/pull/712
  
@ushachar Done. Thanks


---
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 #712: TS-4523 - Add pause functionality to the Transform...

2016-10-04 Thread atsci
Github user atsci commented on the issue:

https://github.com/apache/trafficserver/pull/712
  
Linux build *successful*! See 
https://ci.trafficserver.apache.org/job/Github-Linux/823/ 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 #712: TS-4523 - Add pause functionality to the Transform...

2016-10-04 Thread atsci
Github user atsci commented on the issue:

https://github.com/apache/trafficserver/pull/712
  
FreeBSD build *successful*! See 
https://ci.trafficserver.apache.org/job/Github-FreeBSD/929/ 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 #712: TS-4523 - Add pause functionality to the Transform...

2016-10-04 Thread ushachar
Github user ushachar commented on the issue:

https://github.com/apache/trafficserver/pull/712
  
[approve ci]


---
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 #712: TS-4523 - Add pause functionality to the Transform...

2016-08-16 Thread ushachar
Github user ushachar commented on the issue:

https://github.com/apache/trafficserver/pull/712
  
The reason for wanting this feature is to support prolonged content related 
actions -- in @davidbz  case out-of-process content malware analysis.

When acting as a forward proxy and handling a request for a very large 
file, he needs to support an initial analysis on the first N MBs of the file in 
order to decide if he must analyze the full file or can release it to the 
client.
This analysis can take a significant amount of time and continuing to 
buffer the contents can lead to resource exhaustion.
(When combined with turning on flow control to make sure that we throttle 
the proxy<->upstream tcp connection)

@bgaff - I know you had some concerns regarding this - does this use case 
seem reasonable to you?


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