[jira] Created: (OFBIZ-1791) Update Jetty Container to use Jetty 6

[Nicolas Malin (Jira)]

Update Jetty Container to use Jetty 6
-------------------------------------

                 Key: OFBIZ-1791
                 URL: https://issues.apache.org/jira/browse/OFBIZ-1791
             Project: OFBiz
          Issue Type: Improvement
          Components: framework
    Affects Versions: SVN trunk
            Reporter: Joe Eckard
            Priority: Minor
             Fix For: SVN trunk




--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

[Nicolas Malin (Jira)]

     [ https://issues.apache.org/jira/browse/OFBIZ-1791?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Joe Eckard updated OFBIZ-1791:
------------------------------

    Attachment: jetty.patch

This patch includes the new JettyContainer and configuration. It shouldn't be committed as-is, since it removes the catalina config and makes jetty the default, but it can be used to test the updated container. The newer SSL certificate handling in the catalina container aren't supported yet, but persistent sessions are, along with new NIO listeners.

> Update Jetty Container to use Jetty 6
> -------------------------------------
>
>                 Key: OFBIZ-1791
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-1791
>             Project: OFBiz
>          Issue Type: Improvement
>          Components: framework
>    Affects Versions: SVN trunk
>            Reporter: Joe Eckard
>            Priority: Minor
>             Fix For: SVN trunk
>
>         Attachments: jetty.patch
>
>

--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

[Nicolas Malin (Jira)]

    [ https://issues.apache.org/jira/browse/OFBIZ-1791?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12596441#action_12596441 ]

Joe Eckard commented on OFBIZ-1791:
-----------------------------------

In addition to this patch, you will also need to replace the jars in framework/jetty/lib with the following jars included the jetty 6.1.x distribution:

ant-1.6.5.jar
jasper-compiler-5.5.15.jar
jasper-compiler-jdt-5.5.15.jar
jasper-runtime-5.5.15.jar
jcl104-over-slf4j-1.3.1.jar
jetty-6.1.9.jar
jetty-ajp-6.1.9.jar
jetty-sslengine-6.1.9.jar
jetty-util-6.1.9.jar
slf4j-api-1.3.1.jar
xmlParserAPIs-2.6.2.jar

and also the following jar which tells jetty to use log4j:

slf4j-log4j12-1.3.1.jar

> Update Jetty Container to use Jetty 6
> -------------------------------------
>
>                 Key: OFBIZ-1791
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-1791
>             Project: OFBiz
>          Issue Type: Improvement
>          Components: framework
>    Affects Versions: SVN trunk
>            Reporter: Joe Eckard
>            Priority: Minor
>             Fix For: SVN trunk
>
>         Attachments: jetty.patch
>
>

--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

[Nicolas Malin (Jira)]

    [ https://issues.apache.org/jira/browse/OFBIZ-1791?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12596439#action_12596439 ]

eckardjf edited comment on OFBIZ-1791 at 5/13/08 9:58 AM:
------------------------------------------------------------

This patch includes the new JettyContainer and configuration. It shouldn't be committed as-is, since it removes the catalina config and makes jetty the default, but it can be used to test the updated container. The newer SSL certificate handling features in the catalina container aren't supported yet, but persistent sessions are, along with new NIO listeners.

      was (Author: eckardjf):
    This patch includes the new JettyContainer and configuration. It shouldn't be committed as-is, since it removes the catalina config and makes jetty the default, but it can be used to test the updated container. The newer SSL certificate handling in the catalina container aren't supported yet, but persistent sessions are, along with new NIO listeners.
 

> Update Jetty Container to use Jetty 6
> -------------------------------------
>
>                 Key: OFBIZ-1791
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-1791
>             Project: OFBiz
>          Issue Type: Improvement
>          Components: framework
>    Affects Versions: SVN trunk
>            Reporter: Joe Eckard
>            Priority: Minor
>             Fix For: SVN trunk
>
>         Attachments: jetty.patch
>
>

--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

[Nicolas Malin (Jira)]

    [ https://issues.apache.org/jira/browse/OFBIZ-1791?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12596533#action_12596533 ]

Joe Eckard commented on OFBIZ-1791:
-----------------------------------

Also, the jetty identifier string in UtilJ2eeCompat will need to be changed from "Jetty" to "jetty" since it reports in lowercase now.

> Update Jetty Container to use Jetty 6
> -------------------------------------
>
>                 Key: OFBIZ-1791
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-1791
>             Project: OFBiz
>          Issue Type: Improvement
>          Components: framework
>    Affects Versions: SVN trunk
>            Reporter: Joe Eckard
>            Priority: Minor
>             Fix For: SVN trunk
>
>         Attachments: jetty.patch
>
>

--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

[Nicolas Malin (Jira)]

     [ https://issues.apache.org/jira/browse/OFBIZ-1791?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Joe Eckard updated OFBIZ-1791:
------------------------------

    Attachment: utilj2eecompat.patch

> Update Jetty Container to use Jetty 6
> -------------------------------------
>
>                 Key: OFBIZ-1791
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-1791
>             Project: OFBiz
>          Issue Type: Improvement
>          Components: framework
>    Affects Versions: SVN trunk
>            Reporter: Joe Eckard
>            Priority: Minor
>             Fix For: SVN trunk
>
>         Attachments: jetty.patch, utilj2eecompat.patch
>
>

--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

[Nicolas Malin (Jira)]

     [ https://issues.apache.org/jira/browse/OFBIZ-1791?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Joe Eckard updated OFBIZ-1791:
------------------------------

    Attachment: jetty.patch

New patch with "want" and "need" settings for SSL client auth.

> Update Jetty Container to use Jetty 6
> -------------------------------------
>
>                 Key: OFBIZ-1791
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-1791
>             Project: OFBiz
>          Issue Type: Improvement
>          Components: framework
>    Affects Versions: SVN trunk
>            Reporter: Joe Eckard
>            Priority: Minor
>             Fix For: SVN trunk
>
>         Attachments: jetty.patch, utilj2eecompat.patch
>
>

--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

[Nicolas Malin (Jira)]

     [ https://issues.apache.org/jira/browse/OFBIZ-1791?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Joe Eckard updated OFBIZ-1791:
------------------------------

    Attachment:     (was: jetty.patch)

> Update Jetty Container to use Jetty 6
> -------------------------------------
>
>                 Key: OFBIZ-1791
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-1791
>             Project: OFBiz
>          Issue Type: Improvement
>          Components: framework
>    Affects Versions: SVN trunk
>            Reporter: Joe Eckard
>            Priority: Minor
>             Fix For: SVN trunk
>
>         Attachments: jetty.patch, utilj2eecompat.patch
>
>

--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

[Nicolas Malin (Jira)]

    [ https://issues.apache.org/jira/browse/OFBIZ-1791?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12602185#action_12602185 ]

Jacopo Cappellato commented on OFBIZ-1791:
------------------------------------------

Should we consider to commit the upgrade to Jetty 6 before the upcoming framework release?

> Update Jetty Container to use Jetty 6
> -------------------------------------
>
>                 Key: OFBIZ-1791
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-1791
>             Project: OFBiz
>          Issue Type: Improvement
>          Components: framework
>    Affects Versions: SVN trunk
>            Reporter: Joe Eckard
>            Priority: Minor
>             Fix For: SVN trunk
>
>         Attachments: jetty.patch, utilj2eecompat.patch
>
>

--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

[Nicolas Malin (Jira)]

    [ https://issues.apache.org/jira/browse/OFBIZ-1791?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12602196#action_12602196 ]

David E. Jones commented on OFBIZ-1791:
---------------------------------------

I agree it would be good to do... the Jetty stuff hasn't been updated in a LONG time. I would still vote for keeping Tomcat as the default, but having Jetty in there is still great.

> Update Jetty Container to use Jetty 6
> -------------------------------------
>
>                 Key: OFBIZ-1791
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-1791
>             Project: OFBiz
>          Issue Type: Improvement
>          Components: framework
>    Affects Versions: SVN trunk
>            Reporter: Joe Eckard
>            Priority: Minor
>             Fix For: SVN trunk
>
>         Attachments: jetty.patch, utilj2eecompat.patch
>
>

--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

[Nicolas Malin (Jira)]

    [ https://issues.apache.org/jira/browse/OFBIZ-1791?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12602213#action_12602213 ]

Jacques Le Roux commented on OFBIZ-1791:
----------------------------------------

+1 for me too

> Update Jetty Container to use Jetty 6
> -------------------------------------
>
>                 Key: OFBIZ-1791
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-1791
>             Project: OFBiz
>          Issue Type: Improvement
>          Components: framework
>    Affects Versions: SVN trunk
>            Reporter: Joe Eckard
>            Priority: Minor
>             Fix For: SVN trunk
>
>         Attachments: jetty.patch, utilj2eecompat.patch
>
>

--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

[Nicolas Malin (Jira)]

     [ https://issues.apache.org/jira/browse/OFBIZ-1791?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Jacopo Cappellato reassigned OFBIZ-1791:
----------------------------------------

    Assignee: Jacopo Cappellato

> Update Jetty Container to use Jetty 6
> -------------------------------------
>
>                 Key: OFBIZ-1791
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-1791
>             Project: OFBiz
>          Issue Type: Improvement
>          Components: framework
>    Affects Versions: SVN trunk
>            Reporter: Joe Eckard
>            Assignee: Jacopo Cappellato
>            Priority: Minor
>             Fix For: SVN trunk
>
>         Attachments: jetty.patch, utilj2eecompat.patch
>
>

--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

[Nicolas Malin (Jira)]

    [ https://issues.apache.org/jira/browse/OFBIZ-1791?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12605537#action_12605537 ]

Jacopo Cappellato commented on OFBIZ-1791:
------------------------------------------

Hi Joe,

I would like to commit your work soon; could you please confirm that all we have to do is to:

1) add the following jars:
ant-1.6.5.jar
jasper-compiler-5.5.15.jar
jasper-compiler-jdt-5.5.15.jar
jasper-runtime-5.5.15.jar
jcl104-over-slf4j-1.3.1.jar
jetty-6.1.9.jar
jetty-ajp-6.1.9.jar
jetty-sslengine-6.1.9.jar
jetty-util-6.1.9.jar
slf4j-api-1.3.1.jar
xmlParserAPIs-2.6.2.jar
slf4j-log4j12-1.3.1.jar

2) commit the patch jetty.patch

3) do we really need to commit all the changes in utilj2eecompat.patch? It seems to me that the only required one is this:
@@ -64,7 +64,7 @@
             boolean usestream = true;
             boolean nestjspexception = true;
             // if context is null use an empty string here which will cause the defaults to be used
-            String serverInfo = context == null ? "" : context.getServerInfo();
+            String serverInfo = context == null ? "" : context.getServerInfo().toLowerCase();
 
             Debug.logInfo("serverInfo: " + serverInfo, module);
 
4) why are you saying that the patch removes the catalina config and makes jetty the default? I don't see this code change and I guess it was in your first patch (that you replaced later). Am I wrong?

Thanks,

Jacopo

> Update Jetty Container to use Jetty 6
> -------------------------------------
>
>                 Key: OFBIZ-1791
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-1791
>             Project: OFBiz
>          Issue Type: Improvement
>          Components: framework
>    Affects Versions: SVN trunk
>            Reporter: Joe Eckard
>            Assignee: Jacopo Cappellato
>            Priority: Minor
>             Fix For: SVN trunk
>
>         Attachments: jetty.patch, utilj2eecompat.patch
>
>

--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

[Nicolas Malin (Jira)]

    [ https://issues.apache.org/jira/browse/OFBIZ-1791?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12605627#action_12605627 ]

Joe Eckard commented on OFBIZ-1791:
-----------------------------------

1) Correct. All of the jars are included in the jetty distribution except for slf4j-log4j12-1.3.1.jar, which you need to get from the slf4j project. (I had problem with the newest slf4j, but 1.3.1 works great)

3) The problem is that jetty now reports itself as "jetty" instead of "Jetty" - the minimum required change would be that one line and also the static strings at the beginning of the file.

4) Yes, that was in the first patch - you can disregard that.

> Update Jetty Container to use Jetty 6
> -------------------------------------
>
>                 Key: OFBIZ-1791
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-1791
>             Project: OFBiz
>          Issue Type: Improvement
>          Components: framework
>    Affects Versions: SVN trunk
>            Reporter: Joe Eckard
>            Assignee: Jacopo Cappellato
>            Priority: Minor
>             Fix For: SVN trunk
>
>         Attachments: jetty.patch, utilj2eecompat.patch
>
>

--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

[Nicolas Malin (Jira)]

    [ https://issues.apache.org/jira/browse/OFBIZ-1791?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12606788#action_12606788 ]

Jacopo Cappellato commented on OFBIZ-1791:
------------------------------------------

Joe,

about #2: do we have to change all the strings from uppercase to lowercase or just the one for Jetty?

Thanks for your help,

Jacopo

> Update Jetty Container to use Jetty 6
> -------------------------------------
>
>                 Key: OFBIZ-1791
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-1791
>             Project: OFBiz
>          Issue Type: Improvement
>          Components: framework
>    Affects Versions: SVN trunk
>            Reporter: Joe Eckard
>            Assignee: Jacopo Cappellato
>            Priority: Minor
>             Fix For: SVN trunk
>
>         Attachments: jetty.patch, utilj2eecompat.patch
>
>

--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

[Nicolas Malin (Jira)]

    [ https://issues.apache.org/jira/browse/OFBIZ-1791?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12606793#action_12606793 ]

Joe Eckard commented on OFBIZ-1791:
-----------------------------------

Yes, you could just switch "Jetty" to "jetty" - I was just attempting to make the code more tolerant of changing product names in the future by lowercasing everything before comparing strings.

> Update Jetty Container to use Jetty 6
> -------------------------------------
>
>                 Key: OFBIZ-1791
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-1791
>             Project: OFBiz
>          Issue Type: Improvement
>          Components: framework
>    Affects Versions: SVN trunk
>            Reporter: Joe Eckard
>            Assignee: Jacopo Cappellato
>            Priority: Minor
>             Fix For: SVN trunk
>
>         Attachments: jetty.patch, utilj2eecompat.patch
>
>

--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

[Nicolas Malin (Jira)]

    [ https://issues.apache.org/jira/browse/OFBIZ-1791?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12607506#action_12607506 ]

Jacopo Cappellato commented on OFBIZ-1791:
------------------------------------------

Joe,

could you please resubmit the jetty.patch with the "grant ASF lincense" set to true?

Thanks so much,

Jacopo

> Update Jetty Container to use Jetty 6
> -------------------------------------
>
>                 Key: OFBIZ-1791
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-1791
>             Project: OFBiz
>          Issue Type: Improvement
>          Components: framework
>    Affects Versions: SVN trunk
>            Reporter: Joe Eckard
>            Assignee: Jacopo Cappellato
>            Priority: Minor
>             Fix For: SVN trunk
>
>         Attachments: jetty.patch, utilj2eecompat.patch
>
>

--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

[Nicolas Malin (Jira)]

     [ https://issues.apache.org/jira/browse/OFBIZ-1791?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Jacopo Cappellato updated OFBIZ-1791:
-------------------------------------

    Attachment: jetty-tmp.patch

Joe,

I tried to run the new Jetty using your patch but I'm getting errors when the system attempts to load the web applications:

2008-06-24 11:37:22,971 (main) [            UtilXml.java:241:INFO ] XML Read 0.022s: file:/Users/jacopoc/ofbiz/ofbiz-trunk/applications/ecommerce/webapp/ecommerce/WEB-INF/controller.xml
2008-06-24 11:37:22,973 (main) [                       ?:?  :ERROR] Failed startup of context org.mortbay.jetty.webapp.WebAppContext@a355d0{/ncr,/Users/jacopoc/ofbiz/ofbiz-trunk/hot-deploy/ncr/webapp/ncr}
java.lang.NullPointerException
        at org.mortbay.jetty.webapp.WebAppContext.dumpUrl(WebAppContext.java:468)
        at org.mortbay.jetty.webapp.WebAppContext.doStart(WebAppContext.java:451)
        at org.mortbay.component.AbstractLifeCycle.start(AbstractLifeCycle.java:39)
        at org.mortbay.jetty.handler.HandlerCollection.doStart(HandlerCollection.java:152)
        at org.mortbay.component.AbstractLifeCycle.start(AbstractLifeCycle.java:39)
        at org.mortbay.jetty.handler.HandlerWrapper.doStart(HandlerWrapper.java:130)
        at org.mortbay.jetty.Server.doStart(Server.java:222)
        at org.mortbay.component.AbstractLifeCycle.start(AbstractLifeCycle.java:39)
        at org.ofbiz.jetty.container.JettyContainer.start(JettyContainer.java:402)
        at org.ofbiz.base.container.ContainerLoader.start(ContainerLoader.java:101)
        at org.ofbiz.base.start.Start.startStartLoaders(Start.java:263)
        at org.ofbiz.base.start.Start.startServer(Start.java:312)
        at org.ofbiz.base.start.Start.start(Start.java:316)
        at org.ofbiz.base.start.Start.main(Start.java:399)


I'm sure I am missing something obvious... do you have some hints?

Please find the attached patch that shows all the changes I did.

Thanks,

Jacopo

> Update Jetty Container to use Jetty 6
> -------------------------------------
>
>                 Key: OFBIZ-1791
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-1791
>             Project: OFBiz
>          Issue Type: Improvement
>          Components: framework
>    Affects Versions: SVN trunk
>            Reporter: Joe Eckard
>            Assignee: Jacopo Cappellato
>            Priority: Minor
>             Fix For: SVN trunk
>
>         Attachments: jetty-tmp.patch, jetty.patch, utilj2eecompat.patch
>
>

--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

[Nicolas Malin (Jira)]

    [ https://issues.apache.org/jira/browse/OFBIZ-1791?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12607545#action_12607545 ]

jacopoc edited comment on OFBIZ-1791 at 6/24/08 3:15 AM:
-------------------------------------------------------------------

Joe,

I tried to run the new Jetty using your patch but I'm getting errors when the system attempts to load the web applications:

2008-06-24 11:37:19,549 (main) [                       ?:?  :ERROR] Failed startup of context org.mortbay.jetty.webapp.WebAppContext@c39fe4{/ecommerce,/Users/jacopoc/ofbiz/ofbiz-trunk/applications/ecommerce/webapp/ecommerce}
java.lang.NullPointerException
        at org.mortbay.jetty.webapp.WebAppContext.dumpUrl(WebAppContext.java:468)
        at org.mortbay.jetty.webapp.WebAppContext.doStart(WebAppContext.java:451)
        at org.mortbay.component.AbstractLifeCycle.start(AbstractLifeCycle.java:39)
        at org.mortbay.jetty.handler.HandlerCollection.doStart(HandlerCollection.java:152)
        at org.mortbay.component.AbstractLifeCycle.start(AbstractLifeCycle.java:39)
        at org.mortbay.jetty.handler.HandlerWrapper.doStart(HandlerWrapper.java:130)
        at org.mortbay.jetty.Server.doStart(Server.java:222)
        at org.mortbay.component.AbstractLifeCycle.start(AbstractLifeCycle.java:39)
        at org.ofbiz.jetty.container.JettyContainer.start(JettyContainer.java:402)
        at org.ofbiz.base.container.ContainerLoader.start(ContainerLoader.java:101)
        at org.ofbiz.base.start.Start.startStartLoaders(Start.java:263)
        at org.ofbiz.base.start.Start.startServer(Start.java:312)
        at org.ofbiz.base.start.Start.start(Start.java:316)
        at org.ofbiz.base.start.Start.main(Start.java:399)



I'm sure I am missing something obvious... do you have some hints?

Please find the attached patch that shows all the changes I did.

Thanks,

Jacopo



      was (Author: jacopoc):
    Joe,

I tried to run the new Jetty using your patch but I'm getting errors when the system attempts to load the web applications:

2008-06-24 11:37:22,971 (main) [            UtilXml.java:241:INFO ] XML Read 0.022s: file:/Users/jacopoc/ofbiz/ofbiz-trunk/applications/ecommerce/webapp/ecommerce/WEB-INF/controller.xml
2008-06-24 11:37:22,973 (main) [                       ?:?  :ERROR] Failed startup of context org.mortbay.jetty.webapp.WebAppContext@a355d0{/ncr,/Users/jacopoc/ofbiz/ofbiz-trunk/hot-deploy/ncr/webapp/ncr}
java.lang.NullPointerException
        at org.mortbay.jetty.webapp.WebAppContext.dumpUrl(WebAppContext.java:468)
        at org.mortbay.jetty.webapp.WebAppContext.doStart(WebAppContext.java:451)
        at org.mortbay.component.AbstractLifeCycle.start(AbstractLifeCycle.java:39)
        at org.mortbay.jetty.handler.HandlerCollection.doStart(HandlerCollection.java:152)
        at org.mortbay.component.AbstractLifeCycle.start(AbstractLifeCycle.java:39)
        at org.mortbay.jetty.handler.HandlerWrapper.doStart(HandlerWrapper.java:130)
        at org.mortbay.jetty.Server.doStart(Server.java:222)
        at org.mortbay.component.AbstractLifeCycle.start(AbstractLifeCycle.java:39)
        at org.ofbiz.jetty.container.JettyContainer.start(JettyContainer.java:402)
        at org.ofbiz.base.container.ContainerLoader.start(ContainerLoader.java:101)
        at org.ofbiz.base.start.Start.startStartLoaders(Start.java:263)
        at org.ofbiz.base.start.Start.startServer(Start.java:312)
        at org.ofbiz.base.start.Start.start(Start.java:316)
        at org.ofbiz.base.start.Start.main(Start.java:399)


I'm sure I am missing something obvious... do you have some hints?

Please find the attached patch that shows all the changes I did.

Thanks,

Jacopo


 

> Update Jetty Container to use Jetty 6
> -------------------------------------
>
>                 Key: OFBIZ-1791
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-1791
>             Project: OFBiz
>          Issue Type: Improvement
>          Components: framework
>    Affects Versions: SVN trunk
>            Reporter: Joe Eckard
>            Assignee: Jacopo Cappellato
>            Priority: Minor
>             Fix For: SVN trunk
>
>         Attachments: jetty-tmp.patch, jetty.patch, utilj2eecompat.patch
>
>

--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

[Nicolas Malin (Jira)]

     [ https://issues.apache.org/jira/browse/OFBIZ-1791?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Joe Eckard updated OFBIZ-1791:
------------------------------

    Attachment: jetty20080702.patch

Here is a complete patch against the latest SVN

> Update Jetty Container to use Jetty 6
> -------------------------------------
>
>                 Key: OFBIZ-1791
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-1791
>             Project: OFBiz
>          Issue Type: Improvement
>          Components: framework
>    Affects Versions: SVN trunk
>            Reporter: Joe Eckard
>            Assignee: Jacopo Cappellato
>            Priority: Minor
>             Fix For: SVN trunk
>
>         Attachments: jetty-tmp.patch, jetty.patch, jetty20080702.patch, utilj2eecompat.patch
>
>

--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.