[ http://issues.apache.org/jira/browse/AXIS2-121?page=comments#action_12318096 ]
Davanum Srinivas commented on AXIS2-121: ---------------------------------------- After reviewing our code in org.apache.axis2.transport.http.SimpleHTTPServer and reviewing Jetty and AXIS-2082. Here's what i think we should do. - Pick the source code for the simple server from httpclient test (http://svn.apache.org/viewcvs.cgi/jakarta/commons/proper/httpclient/trunk/src/test/org/apache/commons/httpclient/server/) - Change the package name to org.apache.axis2.transport.http.server - Don't touch the code we borrowed too much (will land up in trouble trying to sync up later) - Rewrite out SimpleHTTPServer based on this code and throw away out HTTPWorker/ChunkedOutputStream etc. (all unnecessary code) I believe this code in commons httpclient is much more robust than our code (Remember, we are not in the business of writing a http server :). So we should just use that code. I believe it will help with AXIS2-128 as well. thanks, dims > Jetty based SimpleHTTPServer > ---------------------------- > > Key: AXIS2-121 > URL: http://issues.apache.org/jira/browse/AXIS2-121 > Project: Apache Axis 2.0 (Axis2) > Type: Improvement > Reporter: Davanum Srinivas > > This is yet another lesson learned from Axis 1.X. A lot of code is not tested > well because a typical deployment is a servlet and none of the tests in the > harness are run inside a servlet. (http chunking, gzip compression, anything > that exercises the thread context loader, class loader stuff etc...). Just > like the move to CommonsTransportSender based on jakarta commons client, I > think we should port the JettyAxisServer in Axis 1.X clean it up and use that > in Axis 2.0 as the default environment for testing (and get rid of the > SimpleHTTPServer). > thanks, > dims -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: http://issues.apache.org/jira/secure/Administrators.jspa - For more information on JIRA, see: http://www.atlassian.com/software/jira
