[GitHub] trafficserver pull request: TS-3202: Fail the parse if a CTL chara...

2014-11-18 Thread SolidWallOfCode
Github user SolidWallOfCode commented on the pull request: https://github.com/apache/trafficserver/pull/149#issuecomment-63547304 Any reason to not use ParseRules::is_token()? That seems explicitly defined to match the RFC. --- If your project is set up for it, you can reply to this

Re: ATS Integration Testing

2014-11-18 Thread Joshua Blatt
Hi guys, I've booked a room at 2:30pm pacific at Yahoo's campus here - http://yhoo.it/1xMUa5d.   When you arrive please park in guest parking or really anywhere you can, head over to building B, and give me a call at (310) 694-6908.   It'd be great if you can get there around 15 minutes early so

Re: ATS Integration Testing

2014-11-18 Thread Brian Geffon
Hey Josh, James, Thomas, and I have been bouncing some ideas around that we'd like to share too. We can all come by at 2:30 on Thursday if that works for you guys. Brian On Tue, Nov 18, 2014 at 10:24 AM, Narayan Balasubramanian (நாராயண் ) < bv.nara...@gmail.com> wrote: > I can meet Thursday thi

Re: [1/2] trafficserver git commit: [TS-3153]: Ability to disable or modify npn advertisement based on SNI

2014-11-18 Thread James Peach
> On Nov 17, 2014, at 1:12 PM, sudhe...@apache.org wrote: > > Repository: trafficserver > Updated Branches: > refs/heads/master 08f19da9b -> 34ca6f2dc > > > [TS-3153]: Ability to disable or modify npn advertisement based on SNI > > > Project: http://git-wip-us.apache.org/repos/asf/trafficser

Re: ATS Integration Testing

2014-11-18 Thread நாராயண்
I can meet Thursday this week. Thanks, Narayan On Tue, Nov 18, 2014 at 9:30 AM, Joshua Blatt wrote: > Hi guys, > Sorry I've been slow setting this up, but before this slips away from us, > is anyone up for a meeting this week? Say Thursday sometime between 11am > and 4pm? > We could do this

Re: ATS Integration Testing

2014-11-18 Thread Meera Mosale Nataraja
Hello Josh, Please count me in. Regards, Meera. On Tue, Nov 18, 2014 at 9:30 AM, Joshua Blatt wrote: > Hi guys, > Sorry I've been slow setting this up, but before this slips away from us, is > anyone up for a meeting this week? Say Thursday sometime between 11am and > 4pm? > We could do th

Re: [API Review] Add TSSslAdvertiseProtocolSet()

2014-11-18 Thread Sudheer Vinukonda
On 11/18/14, 9:11 AM, "James Peach" wrote: > >> On Nov 18, 2014, at 8:57 AM, Sudheer Vinukonda >> wrote: >> >> >> >> On 11/18/14, 8:48 AM, "James Peach" wrote: >> >>> On Nov 18, 2014, at 8:38 AM, Sudheer Vinukonda wrote: Hi James, How can my_awesome_callba

Re: [API Review] Add TSSslAdvertiseProtocolSet()

2014-11-18 Thread James Peach
> On Nov 17, 2014, at 4:09 PM, Sudheer Vinukonda > wrote: > > Hi All, > > I would like add a new API TSSslAdvertiseProtocolSet(), to address the > requirement of modifying the NPN advertising list for each SSL connection > based on SNI (please refer TS-3153). > > https://issues.apache.org/jir

Re: ATS Integration Testing

2014-11-18 Thread Joshua Blatt
Hi guys, Sorry I've been slow setting this up, but before this slips away from us, is anyone up for a meeting this week?   Say Thursday sometime between 11am and 4pm? We could do this in person and I'm happy to host a meeting at the Yahoo office in Sunnyvale, but if transit time makes it too diff

Apache Traffic Server Code of Conduct

2014-11-18 Thread Leif Hedstrom
This was recently created for ASF, based on CouchDB’s Code of Conduct, please take a look at it. http://www.apache.org/foundation/policies/conduct.html Cheers, — Leif

[GitHub] trafficserver pull request: TS-3202: Fail the parse if a CTL chara...

2014-11-18 Thread shinrich
GitHub user shinrich opened a pull request: https://github.com/apache/trafficserver/pull/149 TS-3202: Fail the parse if a CTL character is found in the method. You can merge this pull request into a Git repository by running: $ git pull https://github.com/shinrich/trafficserve

Re: [API Review] Add TSSslAdvertiseProtocolSet()

2014-11-18 Thread James Peach
> On Nov 18, 2014, at 8:57 AM, Sudheer Vinukonda > wrote: > > > > On 11/18/14, 8:48 AM, "James Peach" wrote: > >> >>> On Nov 18, 2014, at 8:38 AM, Sudheer Vinukonda >>> wrote: >>> >>> Hi James, >>> >>> How can my_awesome_callback() know the specific custom NPN advertisement >>> list for

Re: [API Review] Add TSSslAdvertiseProtocolSet()

2014-11-18 Thread Sudheer Vinukonda
On 11/18/14, 8:48 AM, "James Peach" wrote: > >> On Nov 18, 2014, at 8:38 AM, Sudheer Vinukonda >> wrote: >> >> Hi James, >> >> How can my_awesome_callback() know the specific custom NPN advertisement >> list for a given netVC? > >It would know this in the same way the plugin knows what to pas

Re: [API Review] Add TSSslAdvertiseProtocolSet()

2014-11-18 Thread James Peach
> On Nov 18, 2014, at 8:38 AM, Sudheer Vinukonda > wrote: > > Hi James, > > How can my_awesome_callback() know the specific custom NPN advertisement > list for a given netVC? It would know this in the same way the plugin knows what to pass into your proposed API. > The purpose of the API is

[GitHub] trafficserver pull request: TS-2009: Fail parsing of HTTP headers ...

2014-11-18 Thread shinrich
GitHub user shinrich opened a pull request: https://github.com/apache/trafficserver/pull/148 TS-2009: Fail parsing of HTTP headers if nul character is detected You can merge this pull request into a Git repository by running: $ git pull https://github.com/shinrich/trafficserve

Re: [API Review] Add TSSslAdvertiseProtocolSet()

2014-11-18 Thread Sudheer Vinukonda
Hi James, How can my_awesome_callback() know the specific custom NPN advertisement list for a given netVC? The purpose of the API is not to trigger the callback, but, to update/modify the npnAdvertised list for a given netVC. The callback part is trivial and I basically would do something simil

Re: [API Review] Add TSSslAdvertiseProtocolSet()

2014-11-18 Thread James Peach
> On Nov 17, 2014, at 4:09 PM, Sudheer Vinukonda > wrote: > > Hi All, > > I would like add a new API TSSslAdvertiseProtocolSet(), to address the > requirement of modifying the NPN advertising list for each SSL connection > based on SNI (please refer TS-3153). > > https://issues.apache.org/jir