Re: SPDY support

2012-03-25 Thread Filip Hanik Mailing Lists
? Filip - Original Message - From: Costin Manolache cos...@gmail.com To: Tomcat Developers List dev@tomcat.apache.org Sent: Saturday, March 24, 2012 10:44:19 PM Subject: Re: SPDY support Hi, I did a first round of backporting to tomcat7 - only the hooks. Please take a look

Re: SPDY support

2012-03-25 Thread Costin Manolache
cos...@gmail.com To: Tomcat Developers List dev@tomcat.apache.org Sent: Saturday, March 24, 2012 10:44:19 PM Subject: Re: SPDY support Hi, I did a first round of backporting to tomcat7 - only the hooks. Please take a look and let me know: https://github.com/costinm/tomcat/blob

Re: SPDY support

2012-03-25 Thread Mark Thomas
On 25/03/2012 17:24, Costin Manolache wrote: For Tomcat7 I'm proposing to only add hooks: 'beforeHandshake' callback in Handler, and an optional NpnHandler class plus setters in protocol. There is no Java7 or spdy dependency at all. Sounds good to me. The main question is if it's ok to add a

Re: SPDY support

2012-03-25 Thread Bill Barker
Mark Thomas wrote in message news:4f6f8735.3090...@apache.org... On 25/03/2012 17:24, Costin Manolache wrote: For Tomcat7 I'm proposing to only add hooks: 'beforeHandshake' callback in Handler, and an optional NpnHandler class plus setters in protocol. There is no Java7 or spdy dependency

Re: SPDY support

2012-03-25 Thread Costin Manolache
On Sun, Mar 25, 2012 at 5:58 PM, Bill Barker billwbar...@verizon.netwrote: Mark Thomas wrote in message news:4F6F8735.3090706@apache.**org... On 25/03/2012 17:24, Costin Manolache wrote: For Tomcat7 I'm proposing to only add hooks: 'beforeHandshake' callback in Handler, and an optional

Re: SPDY support

2012-03-24 Thread Costin Manolache
is: https://github.com/costinm/tomcat/commit/698f4105a818e872877f3e8d9c50e003e2e706f0 The problem is that it add a compile dep on http://wiki.eclipse.org/Jetty/Feature/NPN the classes are modified from openjdk7 - it'll also need to be added at runtime ( if spdy support with nio/jio is needed

SPDY support

2012-03-19 Thread Gus Heck
from SPDY. However, it seems that SPDY support in java is a wee tad nascent. As I understand it, there's an issue with oracle JDK support for NPN that is forcing tomcat to use APR. I also notice that Jetty has just released a version that supports SPDY using the NPN that is apparently

Re: SPDY support

2012-03-19 Thread Costin Manolache
in the near future, which I expect probably stands to benefit significantly from SPDY. However, it seems that SPDY support in java is a wee tad nascent. As I understand it, there's an issue with oracle JDK support for NPN that is forcing tomcat to use APR. I also notice that Jetty has just released

Re: SPDY support

2012-03-14 Thread Konstantin Kolinko
2012/2/2 Costin Manolache cos...@gmail.com: Some initial patch for SPDY - using the NPN SSL extension, as required by chrome/firefox. Firefox 11.0 that just has been released as stable has support for SPDY. It is disabled by default. According to the docs, To use it right now you'll need to

Re: SPDY support

2012-03-14 Thread Costin Manolache
-spdy=no-compress,no-ssl and test with tomcat jio connector, without apr and the new openssl. I'm going to add spdy support to the NIO connector next - need feedback on how to handle the dependency - see http://wiki.eclipse.org/Jetty/Feature/NPN I guess it'll be a separate class implementing 'npn

Re: SPDY support

2012-03-12 Thread Costin Manolache
Submitted the second part - please let me know if there are concerns with the layout/etc. Note that Jetty announced spdy support as well, and they're using a NPN implementation that requires Java7. Their NPN library seems to have ASF licence ( dual eclipse/asf ), I'll try to get it working

Re: SPDY support

2012-02-22 Thread Costin Manolache
First part submitted. For the second: what is the process for adding a dependency ? jzlib is BSD - I added it to 'download' and build.properties. I can make it optional (i.e. skip building the code that depend on it if jzlib is missing ), but it doesn't seem to be worth it. Do automatic builders

Re: SPDY support

2012-02-22 Thread Mark Thomas
On 22/02/2012 16:51, Costin Manolache wrote: First part submitted. For the second: what is the process for adding a dependency ? Why do you need the dependency? jzlib is BSD - I added it to 'download' and build.properties. LICENSE and NOTICE need updating as well (multiple locations). So

Re: SPDY support

2012-02-22 Thread Costin Manolache
On Wed, Feb 22, 2012 at 8:57 AM, Mark Thomas ma...@apache.org wrote: On 22/02/2012 16:51, Costin Manolache wrote: First part submitted. For the second: what is the process for adding a dependency ? Why do you need the dependency? Spdy requires header compression - with a pre-defined

Re: SPDY support

2012-02-22 Thread Konstantin Kolinko
2012/2/22 Costin Manolache cos...@gmail.com: On Wed, Feb 22, 2012 at 8:57 AM, Mark Thomas ma...@apache.org wrote: On 22/02/2012 16:51, Costin Manolache wrote: First part submitted. For the second: what is the process for adding a dependency ? Why do you need the dependency? Spdy

Re: SPDY support

2012-02-22 Thread Costin Manolache
On Wed, Feb 22, 2012 at 10:27 AM, Konstantin Kolinko knst.koli...@gmail.com wrote: 2012/2/22 Costin Manolache cos...@gmail.com: On Wed, Feb 22, 2012 at 8:57 AM, Mark Thomas ma...@apache.org wrote: On 22/02/2012 16:51, Costin Manolache wrote: First part submitted. For the second: what

Re: SPDY support

2012-02-22 Thread Mark Thomas
On 22/02/2012 18:41, Costin Manolache wrote: On Wed, Feb 22, 2012 at 10:27 AM, Konstantin Kolinko knst.koli...@gmail.com wrote: 2012/2/22 Costin Manolache cos...@gmail.com: On Wed, Feb 22, 2012 at 8:57 AM, Mark Thomas ma...@apache.org wrote: On 22/02/2012 16:51, Costin Manolache wrote:

Re: SPDY support

2012-02-22 Thread Costin Manolache
On Wed, Feb 22, 2012 at 10:47 AM, Mark Thomas ma...@apache.org wrote: On 22/02/2012 18:41, Costin Manolache wrote: On Wed, Feb 22, 2012 at 10:27 AM, Konstantin Kolinko knst.koli...@gmail.com wrote: 2012/2/22 Costin Manolache cos...@gmail.com: On Wed, Feb 22, 2012 at 8:57 AM, Mark

Re: SPDY support

2012-02-16 Thread Mark Thomas
On 16/02/2012 01:44, Costin Manolache wrote: That doesn't mean the application must receive the entire message as one byte[] or as a Stream. You can have a very large fragment but still read it in smaller buffers and notify the user of message start and for each fragment ( I guess like xml

Re: SPDY support

2012-02-15 Thread Mark Thomas
On 15/02/2012 03:53, Costin Manolache wrote: Uploaded another take. This time I'm just adding https://github.com/costinm/tomcat/blob/trunk/java/org/apache/tomcat/util/net/LightHandler.java which is just a Handler - like AjpConnectionHandler, Http11ConnectionHandler. I believe this is

Re: SPDY support

2012-02-15 Thread Mladen Turk
On 02/15/2012 04:53 AM, Costin Manolache wrote: Uploaded another take. For non-SSL ( JIO, and apr without ssl ) - SPDY just kicks in on all connections, this is just a short-cut for testing. I could also define a SpdyProtocol and set it directly on the connector - but seems too much overhead

Re: SPDY support

2012-02-15 Thread Mark Thomas
On 15/02/2012 03:53, Costin Manolache wrote: Uploaded another take. This time I'm just adding https://github.com/costinm/tomcat/blob/trunk/java/org/apache/tomcat/util/net/LightHandler.java which is just a Handler - like AjpConnectionHandler, Http11ConnectionHandler. I believe this is

Re: SPDY support

2012-02-15 Thread Costin Manolache
On Wed, Feb 15, 2012 at 6:33 AM, Mark Thomas ma...@apache.org wrote: On 15/02/2012 03:53, Costin Manolache wrote: Uploaded another take. This time I'm just adding https://github.com/costinm/tomcat/blob/trunk/java/org/apache/tomcat/util/net/LightHandler.java which is just a Handler -

Re: SPDY support

2012-02-15 Thread Costin Manolache
On Wed, Feb 15, 2012 at 12:56 AM, Mladen Turk mt...@apache.org wrote: On 02/15/2012 04:53 AM, Costin Manolache wrote: Uploaded another take. For non-SSL ( JIO, and apr without ssl ) - SPDY just kicks in on all connections, this is just a short-cut for testing. I could also define a

Re: SPDY support

2012-02-15 Thread Costin Manolache
On Wed, Feb 15, 2012 at 12:34 AM, Mark Thomas ma...@apache.org wrote: Processor is tied to one request/response - Spdy protocol is a multiplexing protocol, so one spdy connection will have 100s of Requests. I am using Processor interface for each muxed spdy stream. Hmm. I wonder if we

Re: SPDY support

2012-02-15 Thread Mark Thomas
On 15/02/2012 16:11, Costin Manolache wrote: After I submit, I'll give a try to changing SocketWrapper (non-generic, add read/write), in the git. If you don't like it - easy to drop the branch, but I think it'll simplify the old connectors enough to compensate for the extra complexity I'm

Re: SPDY support

2012-02-15 Thread Costin Manolache
On Wed, Feb 15, 2012 at 10:31 AM, Mark Thomas ma...@apache.org wrote: On 15/02/2012 16:11, Costin Manolache wrote: After I submit, I'll give a try to changing SocketWrapper (non-generic, add read/write), in the git. If you don't like it - easy to drop the branch, but I think it'll

Re: SPDY support

2012-02-15 Thread Mark Thomas
On 15/02/2012 19:58, Costin Manolache wrote: In spdy I'm using a SpdyFrame class ( spdy allow data frames to be fragmented any way you want by intermediary - so even if I receive a huge frame, I can split it in buffer-sized frames - I believe you can do the same for websocket, as long as

Re: SPDY support

2012-02-15 Thread Costin Manolache
On Wed, Feb 15, 2012 at 12:10 PM, Mark Thomas ma...@apache.org wrote: On 15/02/2012 19:58, Costin Manolache wrote: In spdy I'm using a SpdyFrame class ( spdy allow data frames to be fragmented any way you want by intermediary - so even if I receive a huge frame, I can split it in

Re: SPDY support

2012-02-14 Thread Costin Manolache
Ok, took a bit to get the Apr polling to work and add some minimal tests. Please take another look - in particular to https://github.com/costinm/tomcat/blob/trunk/java/org/apache/tomcat/util/net/AprEndpoint.java The spdy implementation seems to work with chrome, and the client seems to work with

Re: SPDY support

2012-02-14 Thread Yoav Shapira
On Tue, Feb 14, 2012 at 3:29 AM, Costin Manolache cos...@gmail.com wrote: Ok, took a bit to get the Apr polling to work and add some minimal tests. Please take another look - in particular to https://github.com/costinm/tomcat/blob/trunk/java/org/apache/tomcat/util/net/AprEndpoint.java The

Re: SPDY support

2012-02-14 Thread Mark Thomas
to clean them up a bit more. I am all for adding SPDY support to Tomcat asap including if practical to the 7.0.x branch (noting that there may be some API changes that may prevent this). I think the patch needs more discussion before it is committed to trunk. There are several areas where I am

Re: SPDY support

2012-02-14 Thread Costin Manolache
server implementation. The spdy client and tests - probably later, I want to clean them up a bit more. I am all for adding SPDY support to Tomcat asap including if practical to the 7.0.x branch (noting that there may be some API changes that may prevent this). So far I don't think I modified

Re: SPDY support

2012-02-14 Thread Costin Manolache
add the spdy server implementation. The spdy client and tests - probably later, I want to clean them up a bit more. I am all for adding SPDY support to Tomcat asap including if practical to the 7.0.x branch (noting that there may be some API changes that may prevent this). I think the patch

Re: SPDY support

2012-02-14 Thread Mark Thomas
changes first, than add the spdy server implementation. The spdy client and tests - probably later, I want to clean them up a bit more. I am all for adding SPDY support to Tomcat asap including if practical to the 7.0.x branch (noting that there may be some API changes that may prevent

Re: SPDY support

2012-02-14 Thread Mark Thomas
On 14/02/2012 21:03, Costin Manolache wrote: BTW - one thing that may simplify the existing code is replacing SocketWrapperE with a simple SocketWrapper, and having JIO and APR define their own SocketWrapper - like NIO does. Worth adding to the 8.0.x TODO list. It would also save a lot of

Re: SPDY support

2012-02-14 Thread Costin Manolache
of bugs - but it's a start. If no objections - I'll start merging the LightProtocol/util.netchanges first, than add the spdy server implementation. The spdy client and tests - probably later, I want to clean them up a bit more. I am all for adding SPDY support to Tomcat asap including

Re: SPDY support

2012-02-03 Thread Pid
On 02/02/2012 13:30, Remy Maucherat wrote: On Thu, 2012-02-02 at 12:28 +, Pid wrote: Of course, I'm even less sure about websockets since it's kind of a mess. I'd say it was the other way round. WebSockets finalised before Xmas, and there's lots of interest from various communities to

Re: SPDY support

2012-02-02 Thread Remy Maucherat
On Wed, 2012-02-01 at 13:54 -0800, Costin Manolache wrote: Hi, Some initial patch for SPDY - using the NPN SSL extension, as required by chrome/firefox. I used github so it's easy to add comments or test: https://github.com/costinm/tomcat/commit/e97def3314216c083cc4b68ad9731e5a94b2af28

Re: SPDY support

2012-02-02 Thread Mark Thomas
On 02/02/2012 10:05, Remy Maucherat wrote: On Wed, 2012-02-01 at 13:54 -0800, Costin Manolache wrote: Hi, Some initial patch for SPDY - using the NPN SSL extension, as required by chrome/firefox. I used github so it's easy to add comments or test:

Re: SPDY support

2012-02-02 Thread Pid
On 02/02/2012 10:05, Remy Maucherat wrote: On Wed, 2012-02-01 at 13:54 -0800, Costin Manolache wrote: Hi, Some initial patch for SPDY - using the NPN SSL extension, as required by chrome/firefox. I used github so it's easy to add comments or test:

Re: SPDY support

2012-02-02 Thread Remy Maucherat
On Thu, 2012-02-02 at 12:28 +, Pid wrote: Of course, I'm even less sure about websockets since it's kind of a mess. I'd say it was the other way round. WebSockets finalised before Xmas, and there's lots of interest from various communities to judge from the BZ. Websocrap is a bad

Re: SPDY support

2012-02-02 Thread Costin Manolache
On Thu, Feb 2, 2012 at 3:33 AM, Mark Thomas ma...@apache.org wrote: On 02/02/2012 10:05, Remy Maucherat wrote: On Wed, 2012-02-01 at 13:54 -0800, Costin Manolache wrote: Hi, Some initial patch for SPDY - using the NPN SSL extension, as required by chrome/firefox. I used github so

Re: SPDY support

2012-02-02 Thread Mark Thomas
On 02/02/2012 14:14, Costin Manolache wrote: On Thu, Feb 2, 2012 at 3:33 AM, Mark Thomas ma...@apache.org wrote: On 02/02/2012 10:05, Remy Maucherat wrote: Ok, I think your light protocol concept to group any upgraded connections is appropriate. Agreed. I'll see if I can wrap this into

SPDY support

2012-02-01 Thread Costin Manolache
Hi, Some initial patch for SPDY - using the NPN SSL extension, as required by chrome/firefox. I used github so it's easy to add comments or test: https://github.com/costinm/tomcat/commit/e97def3314216c083cc4b68ad9731e5a94b2af28 Basic 'hello world' works in both jio and apr, looking for

Re: SPDY support

2012-02-01 Thread Henri Gomez
Github :-) Le 1 févr. 2012 à 22:54, Costin Manolache cos...@gmail.com a écrit : Hi, Some initial patch for SPDY - using the NPN SSL extension, as required by chrome/firefox. I used github so it's easy to add comments or test: