Mailing-List: contact cxf-dev-help@incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: cxf-dev@incubator.apache.org
Received-SPF: neutral (herse.apache.org: local policy)
Message-ID: <4547B4F1.9080802@envoisolutions.com>
Date: Tue, 31 Oct 2006 15:41:21 -0500
From: Dan Diephouse <dan@envoisolutions.com>
User-Agent: Thunderbird 1.5.0.7 (Windows/20060909)
MIME-Version: 1.0
To: cxf-dev@incubator.apache.org
Subject: Re: Servlet Support
References: <4546DBB2.6000507@envoisolutions.com>
 <4546E05B.1050809@envoisolutions.com> <4546ECE3.2060407@iona.com>
In-Reply-To: <4546ECE3.2060407@iona.com>
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: 7bit

Servlet support is so fundamental to a web services framework that I'm 
not sure it can wait. I will try to spend some time on it today/tonight...

- Dan

Freeman Fang wrote:
> Hi Dan,
> 1. 2. 3. should be fixed in my next commit.
>
> And I create JIRA task to track multiple endpoints per servlet issue.
> https://issues.apache.org/jira/browse/CXF-192
>
> CXF-175 also is a good idea, and we can take CXF-175/192 into account 
> after M1 release
>
> Thanks very much
>
>
> Dan Diephouse wrote:
>
>> Dan Diephouse wrote:
>>
>>> Hiya,
>>> I started looking at the CXFServlet tonight, but I'm a little 
>>> confused. It seems that doPost/doGet are associated with one 
>>> specific Endpoint (it is a field in the class). This would mean that 
>>> you could only have one endpoint per servlet, which seems odd. Can 
>>> someone shed some light? Are there any tests for this? I started one 
>>> using HTTP Unit, but am still trying to figure out what is going on.
>>>
>>> - Dan
>>>
>> Other comments:
>>
>> 1. Lots of ex.printStackTraces() - in fact there are LOTS of these 
>> all throughout the CXF code. When we just let these fly by it can 
>> cause lots of damage to a user's experience as they may not have any 
>> clue what went wrong or we may get stuck in some infinite loop or 
>> something similarly horrible. PLEASE PLEASE rethrow these... 
>> ServletException, Fault, and others all have a constcructor which 
>> takes a Throwable.
>> 2. Minor nitpick but: why are doPost/doGet part of 
>> ServletDestination? It seems like these should be part of CXFServlet 
>> and then when we get a request it should send a Message to the 
>> Destination.
>> 3. Why are we doing JaxWsServiceFactoryBean.create() after we already 
>> created an endpoint/service via new EndpointImpl a few lines above 
>> (170/180ish)??  If you need to access the EndpointInfo, you can get 
>> it via Endpoint.getEndpointInfo()
>> 4. ClassLoaderUtils.loadClass(String name) is around for a reason...
>> 5. What about doDelete/doPut? these are needed for REST
>> 6. This would also be good to have for all our Spring users with an 
>> existing ApplicationContext: 
>> http://issues.apache.org/jira/browse/CXF-175
>>
>> - Dan
>>
>
>


-- 
Dan Diephouse
Envoi Solutions
http://envoisolutions.com
http://netzooid.com/blog