You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@geronimo.apache.org by "Vincent Massol (JIRA)" <de...@geronimo.apache.org> on 2006/02/11 17:37:57 UTC

[jira] Created: (GERONIMO-1621) [Daytrader] TradeWSAction class is not a Servlet

[Daytrader] TradeWSAction class is not a Servlet
------------------------------------------------

         Key: GERONIMO-1621
         URL: http://issues.apache.org/jira/browse/GERONIMO-1621
     Project: Geronimo
        Type: Bug
  Components: sample apps  
    Versions: 1.0    
    Reporter: Vincent Massol


When trying to deploy the web/ module to Jetty I'm getting the following error:

Servlet class org.apache.geronimo.samples.daytrader.TradeWSAction is not a javax.servlet.Servlet

Indeed, looking at the TradeWSAction.java class reveals that it's not a Servlet and thus it shouldn't be declared as a Servlet in web.xml:

      <servlet id="Servlet_30">
         <display-name>org_apache_geronimo_samples_daytrader_TradeWSAction</display-name>
         <servlet-name>org_apache_geronimo_samples_daytrader_TradeWSAction</servlet-name>
         <servlet-class>org.apache.geronimo.samples.daytrader.TradeWSAction</servlet-class>
      </servlet>


-- 
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


Re: [jira] Commented: (GERONIMO-1621) [Daytrader] TradeWSAction class is not a Servlet

Posted by Matt Hogstrom <ma...@hogstrom.org>.
BTW, I'm modifying the DayTrader config at 
http://svn.apache.org/repos/asf/geronimo/daytrader which is where I think you're 
looking.  Should have it ready to release with 1.1 when we'll remove it from 
trunk so work from this area if possible.

Thanks.

Vincent Massol (JIRA) wrote:
>     [ http://issues.apache.org/jira/browse/GERONIMO-1621?page=comments#action_12366089 ] 
> 
> Vincent Massol commented on GERONIMO-1621:
> ------------------------------------------
> 
> Thanks Matt. I didn't know about this JSR-181.
> 
> No this is different from the JBoss issue I raised. As I mentioned above it happened when trying to deploy in Jetty (Jetty 5.x to be precise).
> 
>  I'm checking with the Jetty team why it is happening.
> 
> 
>>[Daytrader] TradeWSAction class is not a Servlet
>>------------------------------------------------
>>
>>         Key: GERONIMO-1621
>>         URL: http://issues.apache.org/jira/browse/GERONIMO-1621
>>     Project: Geronimo
>>        Type: Bug
>>  Components: sample apps
>>    Versions: 1.0
>>    Reporter: Vincent Massol
> 
> 
>>When trying to deploy the web/ module to Jetty I'm getting the following error:
>>Servlet class org.apache.geronimo.samples.daytrader.TradeWSAction is not a javax.servlet.Servlet
>>Indeed, looking at the TradeWSAction.java class reveals that it's not a Servlet and thus it shouldn't be declared as a Servlet in web.xml:
>>      <servlet id="Servlet_30">
>>         <display-name>org_apache_geronimo_samples_daytrader_TradeWSAction</display-name>
>>         <servlet-name>org_apache_geronimo_samples_daytrader_TradeWSAction</servlet-name>
>>         <servlet-class>org.apache.geronimo.samples.daytrader.TradeWSAction</servlet-class>
>>      </servlet>
> 
> 

[jira] Commented: (GERONIMO-1621) [Daytrader] TradeWSAction class is not a Servlet

Posted by "Vincent Massol (JIRA)" <de...@geronimo.apache.org>.
    [ http://issues.apache.org/jira/browse/GERONIMO-1621?page=comments#action_12366089 ] 

Vincent Massol commented on GERONIMO-1621:
------------------------------------------

Thanks Matt. I didn't know about this JSR-181.

No this is different from the JBoss issue I raised. As I mentioned above it happened when trying to deploy in Jetty (Jetty 5.x to be precise).

 I'm checking with the Jetty team why it is happening.

> [Daytrader] TradeWSAction class is not a Servlet
> ------------------------------------------------
>
>          Key: GERONIMO-1621
>          URL: http://issues.apache.org/jira/browse/GERONIMO-1621
>      Project: Geronimo
>         Type: Bug
>   Components: sample apps
>     Versions: 1.0
>     Reporter: Vincent Massol

>
> When trying to deploy the web/ module to Jetty I'm getting the following error:
> Servlet class org.apache.geronimo.samples.daytrader.TradeWSAction is not a javax.servlet.Servlet
> Indeed, looking at the TradeWSAction.java class reveals that it's not a Servlet and thus it shouldn't be declared as a Servlet in web.xml:
>       <servlet id="Servlet_30">
>          <display-name>org_apache_geronimo_samples_daytrader_TradeWSAction</display-name>
>          <servlet-name>org_apache_geronimo_samples_daytrader_TradeWSAction</servlet-name>
>          <servlet-class>org.apache.geronimo.samples.daytrader.TradeWSAction</servlet-class>
>       </servlet>

-- 
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


RE: [jira] Commented: (GERONIMO-1621) [Daytrader] TradeWSAction class is not a Servlet

Posted by Vincent Massol <vm...@pivolis.com>.
Thanks David.

So you're saying that it's allowed to declare a <servlet> in web.xml that is
a POJO class and not extending the Servlet interface?

Thanks
-Vincent

PS: I wanted to answer in JIRA but apparently it's busted on
issues.apache.org/jira. I'm seeing a weird thing: the HTML rendering is
wrong and I'm seeing a jira issue displayed nested within the same jira
issue... (not sure how to express this ;-)).

> -----Original Message-----
> From: David Jencks (JIRA) [mailto:dev@geronimo.apache.org]
> Sent: samedi 11 février 2006 18:34
> To: dev@geronimo.apache.org
> Subject: [jira] Commented: (GERONIMO-1621) [Daytrader] TradeWSAction class
> is not a Servlet
> 
>     [ http://issues.apache.org/jira/browse/GERONIMO-
> 1621?page=comments#action_12366013 ]
> 
> David Jencks commented on GERONIMO-1621:
> ----------------------------------------
> 
> The web.xml is correct, this is a POJO web service.    Apparently there is
> a problem, but this is not it.
> 
> > [Daytrader] TradeWSAction class is not a Servlet
> > ------------------------------------------------
> >
> >          Key: GERONIMO-1621
> >          URL: http://issues.apache.org/jira/browse/GERONIMO-1621
> >      Project: Geronimo
> >         Type: Bug
> >   Components: sample apps
> >     Versions: 1.0
> >     Reporter: Vincent Massol
> 
> >
> > When trying to deploy the web/ module to Jetty I'm getting the following
> error:
> > Servlet class org.apache.geronimo.samples.daytrader.TradeWSAction is not
> a javax.servlet.Servlet
> > Indeed, looking at the TradeWSAction.java class reveals that it's not a
> Servlet and thus it shouldn't be declared as a Servlet in web.xml:
> >       <servlet id="Servlet_30">
> >          <display-
> name>org_apache_geronimo_samples_daytrader_TradeWSAction</display-name>
> >          <servlet-
> name>org_apache_geronimo_samples_daytrader_TradeWSAction</servlet-name>
> >          <servlet-
> class>org.apache.geronimo.samples.daytrader.TradeWSAction</servlet-class>
> >       </servlet>
> 
> --
> 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


[jira] Commented: (GERONIMO-1621) [Daytrader] TradeWSAction class is not a Servlet

Posted by "David Jencks (JIRA)" <de...@geronimo.apache.org>.
    [ http://issues.apache.org/jira/browse/GERONIMO-1621?page=comments#action_12366013 ] 

David Jencks commented on GERONIMO-1621:
----------------------------------------

The web.xml is correct, this is a POJO web service.    Apparently there is a problem, but this is not it.

> [Daytrader] TradeWSAction class is not a Servlet
> ------------------------------------------------
>
>          Key: GERONIMO-1621
>          URL: http://issues.apache.org/jira/browse/GERONIMO-1621
>      Project: Geronimo
>         Type: Bug
>   Components: sample apps
>     Versions: 1.0
>     Reporter: Vincent Massol

>
> When trying to deploy the web/ module to Jetty I'm getting the following error:
> Servlet class org.apache.geronimo.samples.daytrader.TradeWSAction is not a javax.servlet.Servlet
> Indeed, looking at the TradeWSAction.java class reveals that it's not a Servlet and thus it shouldn't be declared as a Servlet in web.xml:
>       <servlet id="Servlet_30">
>          <display-name>org_apache_geronimo_samples_daytrader_TradeWSAction</display-name>
>          <servlet-name>org_apache_geronimo_samples_daytrader_TradeWSAction</servlet-name>
>          <servlet-class>org.apache.geronimo.samples.daytrader.TradeWSAction</servlet-class>
>       </servlet>

-- 
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


[jira] Commented: (GERONIMO-1621) [Daytrader] TradeWSAction class is not a Servlet

Posted by "Matt Hogstrom (JIRA)" <de...@geronimo.apache.org>.
    [ http://issues.apache.org/jira/browse/GERONIMO-1621?page=comments#action_12366054 ] 

Matt Hogstrom commented on GERONIMO-1621:
-----------------------------------------

Section 7.1 of JSR-181 defines Pojo WebServices are defined as servlets or stateless session beans.  Was this error from your JBoss deployment in the verification step?

If you agree that the deployment is corrrect as is can you close this JIRA?

> [Daytrader] TradeWSAction class is not a Servlet
> ------------------------------------------------
>
>          Key: GERONIMO-1621
>          URL: http://issues.apache.org/jira/browse/GERONIMO-1621
>      Project: Geronimo
>         Type: Bug
>   Components: sample apps
>     Versions: 1.0
>     Reporter: Vincent Massol

>
> When trying to deploy the web/ module to Jetty I'm getting the following error:
> Servlet class org.apache.geronimo.samples.daytrader.TradeWSAction is not a javax.servlet.Servlet
> Indeed, looking at the TradeWSAction.java class reveals that it's not a Servlet and thus it shouldn't be declared as a Servlet in web.xml:
>       <servlet id="Servlet_30">
>          <display-name>org_apache_geronimo_samples_daytrader_TradeWSAction</display-name>
>          <servlet-name>org_apache_geronimo_samples_daytrader_TradeWSAction</servlet-name>
>          <servlet-class>org.apache.geronimo.samples.daytrader.TradeWSAction</servlet-class>
>       </servlet>

-- 
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


[jira] Closed: (GERONIMO-1621) [Daytrader] TradeWSAction class is not a Servlet

Posted by "Vamsavardhana Reddy (JIRA)" <ji...@apache.org>.
     [ http://issues.apache.org/jira/browse/GERONIMO-1621?page=all ]

Vamsavardhana Reddy closed GERONIMO-1621.
-----------------------------------------

    Resolution: Won't Fix

Closed as requested by the reporter.  One less JIRA to worry about :o)

> [Daytrader] TradeWSAction class is not a Servlet
> ------------------------------------------------
>
>                 Key: GERONIMO-1621
>                 URL: http://issues.apache.org/jira/browse/GERONIMO-1621
>             Project: Geronimo
>          Issue Type: Bug
>      Security Level: public(Regular issues) 
>          Components: sample apps
>    Affects Versions: 1.0
>            Reporter: Vincent Massol
>
> When trying to deploy the web/ module to Jetty I'm getting the following error:
> Servlet class org.apache.geronimo.samples.daytrader.TradeWSAction is not a javax.servlet.Servlet
> Indeed, looking at the TradeWSAction.java class reveals that it's not a Servlet and thus it shouldn't be declared as a Servlet in web.xml:
>       <servlet id="Servlet_30">
>          <display-name>org_apache_geronimo_samples_daytrader_TradeWSAction</display-name>
>          <servlet-name>org_apache_geronimo_samples_daytrader_TradeWSAction</servlet-name>
>          <servlet-class>org.apache.geronimo.samples.daytrader.TradeWSAction</servlet-class>
>       </servlet>

-- 
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

        

[jira] Commented: (GERONIMO-1621) [Daytrader] TradeWSAction class is not a Servlet

Posted by "Vincent Massol (JIRA)" <de...@geronimo.apache.org>.
    [ http://issues.apache.org/jira/browse/GERONIMO-1621?page=comments#action_12366460 ] 

Vincent Massol commented on GERONIMO-1621:
------------------------------------------

Matt, you can close this issue with "won't fix", thanks. I got the answer from the Jetty team (Jan Bartel). Here it is for the record:

"
Jetty concentrates on the servlet spec, which doesn't provide for POJOs declared as <servlet> in the web.xml.  However, when Jetty is deployed in Geronimo, Geronimo provides the webservices integration that lets (eg) the daytrader app be deployed into Jetty. 

JBoss has done a similar integration for Tomcat and we plan to do it too as part of the jetty-jboss.sar that we provide for people who want to run JBoss with Jetty.

With Jetty6 we also plan to make it an optional feature, using maybe XFire. It's on the todo list for Jetty5.x:
  http://docs.codehaus.org/display/JETTY/Jetty5+ToDo

and now your email has prompted me to put it in the jetty6 JIRA too:
  http://jira.codehaus.org/browse/JETTY-14
"


> [Daytrader] TradeWSAction class is not a Servlet
> ------------------------------------------------
>
>          Key: GERONIMO-1621
>          URL: http://issues.apache.org/jira/browse/GERONIMO-1621
>      Project: Geronimo
>         Type: Bug
>   Components: sample apps
>     Versions: 1.0
>     Reporter: Vincent Massol

>
> When trying to deploy the web/ module to Jetty I'm getting the following error:
> Servlet class org.apache.geronimo.samples.daytrader.TradeWSAction is not a javax.servlet.Servlet
> Indeed, looking at the TradeWSAction.java class reveals that it's not a Servlet and thus it shouldn't be declared as a Servlet in web.xml:
>       <servlet id="Servlet_30">
>          <display-name>org_apache_geronimo_samples_daytrader_TradeWSAction</display-name>
>          <servlet-name>org_apache_geronimo_samples_daytrader_TradeWSAction</servlet-name>
>          <servlet-class>org.apache.geronimo.samples.daytrader.TradeWSAction</servlet-class>
>       </servlet>

-- 
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