Mailing-List: contact dev-help@geronimo.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@geronimo.apache.org
Message-ID: <23759682.1172771150938.JavaMail.jira@brutus>
Date: Thu, 1 Mar 2007 09:45:50 -0800 (PST)
From: "Lin Sun (JIRA)" <jira@apache.org>
To: dev@geronimo.apache.org
Subject: [jira] Created: (GERONIMO-2913) minor changes to
 JettyEJBWebServiceContext and TomcatEJBWebServiceContext for Axis2
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit

minor changes to JettyEJBWebServiceContext and TomcatEJBWebServiceContext for Axis2
-----------------------------------------------------------------------------------

                 Key: GERONIMO-2913
                 URL: https://issues.apache.org/jira/browse/GERONIMO-2913
             Project: Geronimo
          Issue Type: Improvement
      Security Level: public (Regular issues)
          Components: webservices
    Affects Versions: 2.0
         Environment: winxp + sun 1.5 SDK
            Reporter: Lin Sun
             Fix For: 2.0


This patch contains the following change:

1) In Axis2 integration code, we use request.getURI().getQuery(), however null is returned when running jaxws-ejb test as the queryString wasn't set.   The patch will pass the queryString as the parameter when constructing URI.

2) In our Axis2 integration code, we call request.getContextPath() and it returns null from JettyEJBWebServiceContext and /JAXWS-EJB-2.0-snapshot/JAXWSBEAN from TomcatEJBWebServiceContext when running the jaxws-ejb test case.  To me, the tomcat behavior is right.  (See dev list on some discussion of this prob.)  There seems to be a bug in Jetty in request.getContextPath() so I propose to use request.getRequestURI() instead to temply work around the prob.

Patch has been tested and able to do a full build with it.

Thanks, Lin


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