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