[GitHub] trafficserver pull request: TS-4328

2016-04-07 Thread jpeach
Github user jpeach commented on the pull request: https://github.com/apache/trafficserver/pull/554#issuecomment-207223148 I looked at this for a while and I'm not convinced that it doesn't have unindented side-effects. ``is_request_retryable`` is only set once POST data is getting sen

ssl

2016-04-07 Thread Dk Jack
Hi, Is there a way to get access to ssl handshake data in a plugin? I see there's an SNI plugin. I am looking to some data from client hello, ssl hello extensions etc... Bhasker.

Re: Traffic cache config for only 200 status response

2016-04-07 Thread Sudheer Vinukonda
Yeah, Leif's config should definitely work :) About the http.cache setting, I was under the same impression (that it was too late to set it at READ_RESPONSE hook) as Leif until recently, when I accidentally saw that the code actually checks that setting at the read_response hook as well (it ma

Re: Traffic cache config for only 200 status response

2016-04-07 Thread Steve Malenfant
I use what Leif said and it works great. Our problem was origin sending cache control and expire headers and negative response. Looks like the negative caching param doesn't work as intended. I meant to send something about it but could never get around testing it properly. On Wednesday, April 6,

[GitHub] trafficserver pull request: TS-4329 Off-by-one error in NULL termi...

2016-04-07 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/trafficserver/pull/555 --- 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

[GitHub] trafficserver pull request: Workaround to TS-4207

2016-04-07 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/trafficserver/pull/553 --- 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

Re: Congestion_control is it dead?

2016-04-07 Thread Susan Hinrichs
On 4/7/2016 11:53 AM, James Peach wrote: On Apr 7, 2016, at 9:33 AM, Susan Hinrichs wrote: I'm testing congestion_control to enforce per-origin connection limits. In my micro tests, all is looking good. However, as I'm reviewing jira for open issues, I came across the following comment f

[GitHub] trafficserver pull request: TS-4299 Allows for the content length ...

2016-04-07 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/trafficserver/pull/549 --- 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

[GITHUB] MATT setup for committers

2016-04-07 Thread Leif Hedstrom
In preparation for the migration to git-dual, if you are a committer, please visit http://id.apache.org and add your Github user ID to the appropriate field. Then visit the M.A.T.T. page, and activate the connection https://matt.apache.org This should enable your accounts wi

[GitHub] trafficserver pull request: Workaround to TS-4207

2016-04-07 Thread SolidWallOfCode
Github user SolidWallOfCode commented on the pull request: https://github.com/apache/trafficserver/pull/553#issuecomment-206993375 This seems reasonable. I'm not sure about the 120 for host name size but I am unable to offer a number with a better pedigree. --- If your project is set

Re: Congestion_control is it dead?

2016-04-07 Thread James Peach
> On Apr 7, 2016, at 9:33 AM, Susan Hinrichs > wrote: > > I'm testing congestion_control to enforce per-origin connection limits. In > my micro tests, all is looking good. However, as I'm reviewing jira for open > issues, I came across the following comment from Leif in TS-1221 > > "I'm no

Congestion_control is it dead?

2016-04-07 Thread Susan Hinrichs
I'm testing congestion_control to enforce per-origin connection limits. In my micro tests, all is looking good. However, as I'm reviewing jira for open issues, I came across the following comment from Leif in TS-1221 "I'm not sure, but I think congestion control code is missing vital parts t

Reverted last 2 commits

2016-04-07 Thread Masakazu Kitajo
Sorry for the recent borken commit. I couldn't fix the build , so I reverted the commits for now. I'll keep try to figure out the build issue. Msakazu

Re: About ATS plugin to read post data

2016-04-07 Thread Brian Geffon
Rakesh, Also I believe if you NEVER want to forward the request to the origin then all you need is an Intercept Plugin, take a look at the C++ API as it makes it pretty simple, but if you sometimes want to forward the request to the origin based on the request body then take a look at my pull reque

Re: About ATS plugin to read post data

2016-04-07 Thread Brian Geffon
I also have an open pull request : https://github.com/apache/trafficserver/pull/351 that does exactly this, as you've observed with transformations it's unable to block the downstream connection and so this pull request exposes a new API to buffer the request body before allowing the downstream con

Re: About ATS plugin to read post data

2016-04-07 Thread Nick Kew
On Tue, 2016-04-05 at 23:19 -0700, Rakesh Chhabra wrote: > Hi, > > I need to write an ATS plugin where i need to get the post body and a few > request headers in the call. And based on certain condition i want to > return the OK response from ATS itself without forwarding it to any origin > server