ATS 6.0.0 unit test templates

2016-01-14 Thread Devaki Chokshi (dchokshi)
Hello, We are in process of upgrading to ATS version 6.0.0 and have couple of questions for the ATS community. 1.)Would like to know whether ATS has any unit test templates that would invoke error messages within the following templates. proxy/config/body_factory/default/redirect#moved_te

[GitHub] trafficserver pull request: TS-4133: Update the url_sig plugin so ...

2016-01-14 Thread jrushf1239k
GitHub user jrushf1239k opened a pull request: https://github.com/apache/trafficserver/pull/424 TS-4133: Update the url_sig plugin so that application query paramete… See TS-4133. This update to url_sig will allow application query parameters destined for an origin server to rema

[GitHub] trafficserver pull request: TS-4099: Implement Lua support for cus...

2016-01-14 Thread shukitchan
Github user shukitchan commented on the pull request: https://github.com/apache/trafficserver/pull/422#issuecomment-171767764 awesome. will take some time to review it today/tomorrow. Would be nice if you can take some time to write the doc to accompany this feature. thanks. ---

[GitHub] trafficserver pull request: TS-4099: Implement Lua support for cus...

2016-01-14 Thread jpeach
Github user jpeach commented on the pull request: https://github.com/apache/trafficserver/pull/422#issuecomment-171772919 There are some documentation in [header comments](https://github.com/jpeach/trafficserver/blob/TS-4099/proxy/config/metrics.config.default), but I'd be happy to ad

[GitHub] trafficserver pull request: Add TSSslContextCreate method

2016-01-14 Thread biilmann
Github user biilmann commented on the pull request: https://github.com/apache/trafficserver/pull/402#issuecomment-171775715 @shinrich I'll send out an API review notice. Before that, what's your thoughts on adding a `TSSslContextDestroy` method to clarify that it's up to the plugin to

[GitHub] trafficserver pull request: TS-4099: Implement Lua support for cus...

2016-01-14 Thread shukitchan
Github user shukitchan commented on the pull request: https://github.com/apache/trafficserver/pull/422#issuecomment-171777895 should we add something here in these 2 places ? https://docs.trafficserver.apache.org/en/latest/admin-guide/files/index.en.html https://docs.tra

[GitHub] trafficserver pull request: TS-4099: Implement Lua support for cus...

2016-01-14 Thread jpeach
Github user jpeach commented on the pull request: https://github.com/apache/trafficserver/pull/422#issuecomment-171817699 Ok, let's defer that to a separate ticket. The ``stats.config.xml`` is not documented so this isn't any worse. --- If your project is set up for it, you can reply

Re: [VOTE] Release v6.1.0

2016-01-14 Thread Leif Hedstrom
> On Jan 13, 2016, at 11:55 PM, Masakazu Kitajo wrote: > > -1 > > TS-4097 and TS-3967 are not backported to 6.1.x. At least we need > TS-4097 because HTTP/2 doesn't work without this fix. Alright. Lets fail this vote / RC, and I’ll respin a new one as soon as possible. We’ll look at the foll

[GitHub] trafficserver pull request: TS-4118: CID 1347871: Add error handli...

2016-01-14 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/trafficserver/pull/423 --- 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: TS-4099: Implement Lua support for cus...

2016-01-14 Thread shukitchan
Github user shukitchan commented on a diff in the pull request: https://github.com/apache/trafficserver/pull/422#discussion_r49808951 --- Diff: cmd/traffic_manager/metrics.cc --- @@ -0,0 +1,385 @@ +/** @file + * + * Traffic Manager custom metrics. + * + * @sect

[GitHub] trafficserver pull request: TS-4099: Implement Lua support for cus...

2016-01-14 Thread shukitchan
Github user shukitchan commented on a diff in the pull request: https://github.com/apache/trafficserver/pull/422#discussion_r49809962 --- Diff: lib/bindings/repl.cc --- @@ -0,0 +1,60 @@ +/** @file + * + * Lua bindings REPL. + * + * @section license License

[GitHub] trafficserver pull request: TS-4134: Fix privilege elevation issue...

2016-01-14 Thread SolidWallOfCode
GitHub user SolidWallOfCode opened a pull request: https://github.com/apache/trafficserver/pull/425 TS-4134: Fix privilege elevation issues. You can merge this pull request into a Git repository by running: $ git pull https://github.com/SolidWallOfCode/trafficserver ts-4134 A

[GitHub] trafficserver pull request: TS-4130: wrong value assigned

2016-01-14 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/trafficserver/pull/421 --- 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: TS-4099: Implement Lua support for cus...

2016-01-14 Thread jpeach
Github user jpeach commented on a diff in the pull request: https://github.com/apache/trafficserver/pull/422#discussion_r49822945 --- Diff: cmd/traffic_manager/metrics.cc --- @@ -0,0 +1,385 @@ +/** @file + * + * Traffic Manager custom metrics. + * + * @section

[GitHub] trafficserver pull request: TS-4099: Implement Lua support for cus...

2016-01-14 Thread jpeach
Github user jpeach commented on a diff in the pull request: https://github.com/apache/trafficserver/pull/422#discussion_r49822970 --- Diff: lib/bindings/repl.cc --- @@ -0,0 +1,60 @@ +/** @file + * + * Lua bindings REPL. + * + * @section license License + *