Sorry guys I didn't check my mail yesterday at all.
And thanks Robert for testing it out for me.
Btw during the BOF Sanjewa & Deepal mentioned that we should not have the servlet context in the AxisConfiguration and should be only in the MessageContext.
Btw I coudln't test 1_1 as the build is broken.
Regards,
Rajith.
Unable to obtain goal [multiproject:install-callback] -- /opt/workspace/axis2/1_1/modules/samples/maven.xml:207:58: <ant:copy> /opt/workspace/axis2/1_1/modules/samples/script/yahooRESTSearch not found.
On 10/12/06, robert lazarski <
[EMAIL PROTECTED]> wrote:
I just tested the SpringServletContextObjectSupplier - the servlet
context one - and its working again. Thanks guys. Any ideas on getting
ServletContext from our testing env ? I sure would like to get a unit
test on SpringServletContextObjectSupplier.
I'm testing the new ServiceLifeCycle class on the 'Spring inside the
aar case' now.
Thanks,
Robert
On 10/12/06, Saminda Abeyruwan <[EMAIL PROTECTED]> wrote:
> Hi Rajith,
>
> Dims and myself have done some improvements to Spring to work in a servlet
> container.
>
> Please try your test. This will be available in the 1.1 branch Revision:
> 463340. Will add the changes to the trunk soon.
>
> Saminda
>
>
> On 10/12/06, Davanum Srinivas < [EMAIL PROTECTED]> wrote:
> > Rajith,
> >
> > Could you please test one more time from 1.1 branch. I have a patch
> > that may work in r463307.
> >
> > http://svn.apache.org/viewvc?view=rev&revision=463307
> >
> > Basically we were setting the servlet context *AFTER* all the services
> > got loaded. I changed it to set the context property as soon as
> > ConfigurationContext is created.
> >
> > thanks,
> > dims
> >
> > On 10/12/06, Davanum Srinivas < [EMAIL PROTECTED]> wrote:
> > > Saminda,
> > >
> > > Can you please try to recreate this problem? This could easily turn
> > > into a showstopper for RC2.
> > >
> > > thanks,
> > > dims
> > >
> > > On 10/11/06, Rajith Attapattu < [EMAIL PROTECTED]> wrote:
> > > > Saminda,
> > > >
> > > > I tried yesterdays nightly and the problem is still there.
> > > > Apparently the servlet config is not in the AxisConfiguration
> > > >
> > > > I will try to debug it tomorrow if I have some time.
> > > >
> > > > Regards,
> > > >
> > > > Rajith
> > > >
> > > > On 10/11/06, Saminda Abeyruwan < [EMAIL PROTECTED]> wrote:
> > > > > Hi,
> > > > >
> > > > > AxisServelet#init(ServletConfig config);
> > > > >
> > > > > Line 309 --> 312 ; ServletConfig has been set, thus, ServletContext
> is
> > > > available.
> > > > >
> > > > > Saminda
> > > > >
> > > > >
> > > > >
> > > > > On 10/11/06, robert lazarski < [EMAIL PROTECTED]> wrote:
> > > > > > Its late here so I'll take a look tomorrow. In the meantime, the
> > > > > > reason why this wasn't caught is because I'm not sure how to get a
> > > > > > ServletContext from our testing environment or if its even
> possible.
> > > > > > There are tests for the non-servlet container config. If someone
> knows
> > > > > > how to test the former case please post to the list. I've been
> mostly
> > > > > > focused on the latter lately, ie, non-servletContext config inside
> the
> > > > > > AAR.
> > > > > >
> > > > > > Thanks for catching that Rajith.
> > > > > >
> > > > > > Robert
> > > > > >
> > > > > > On 10/10/06, Rajith Attapattu < [EMAIL PROTECTED]> wrote:
> > > > > > > Forwarding with the axis2 prefix.
> > > > > > >
> > > > > > >
> > > > > > > ---------- Forwarded message ----------
> > > > > > > From: Rajith Attapattu < [EMAIL PROTECTED] >
> > > > > > > Date: Oct 10, 2006 10:17 PM
> > > > > > > Subject: Re: Spring support broken ?????
> > > > > > > To: axis-dev@ws.apache.org
> > > > > > > Cc: [EMAIL PROTECTED]
> > > > > > >
> > > > > > > Hi All,
> > > > > > >
> > > > > > > The current spring support fails with the following exception
> possibly
> > > > due
> > > > > > > to the new code change of obtaining the servlet config from the
> > > > > > > AxisConfiguration instead from the Message Context.
> > > > > > >
> > > > > > > Caused by: org.apache.axis2.AxisFault: Axis2 Can't find
> > > > ServletConfig;
> > > > > > > nested exception is:
> > > > > > > java.lang.Exception: Axis2 Can't find ServletConfig
> > > > > > > at
> > > > > > > org.apache.axis2.AxisFault.makeFault
> > > > (AxisFault.java:318)
> > > > > > > at
> > > > > > >
> > > >
> org.apache.axis2.extensions.spring.receivers.SpringServletContextObjectSupplier.getServiceObject(SpringServletContextObjectSupplier.java
> :70)
> > > > > > > ... 43 more
> > > > > > > Caused by: java.lang.Exception: Axis2 Can't find ServletConfig
> > > > > > > at
> > > > > > >
> > > >
> org.apache.axis2.extensions.spring.receivers.SpringServletContextObjectSupplier.getServiceObject (SpringServletContextObjectSupplier.java:53)
> > > > > > > ... 43 more
> > > > > > >
> > > > > > > Now when I look at the code I see that at revesion 426884 the
> code was
> > > > > > > obtaining the servlet config from the msgContext.
> > > > > > >
> > > > > > > servletContext servletContext = (ServletContext)
> > > > msgContext.getOptions().
> > > > > > >
> > > > > > >
> getProperty(HTTPConstants.MC_HTTP_SERVLETCONTEXT);
> > > > > > >
> > > > > > > However the head revision was obtaining the servlet config via
> the
> > > > > > > AxisConfiguration.
> > > > > > >
> > > > > > > ServletConfig servletConfig = (ServletConfig)
> > > > > > > axisService.getAxisConfiguration()
> > > > > > >
> > > > > > >
> .getParameter(HTTPConstants.HTTP_SERVLETCONFIG);
> > > > > > > if (servletConfig == null) {
> > > > > > > throw new Exception("Axis2 Can't find
> > > > ServletConfig");
> > > > > > > }
> > > > > > >
> > > > > > > Why did we make the change ??? If we did so are we making sure
> that we
> > > > > > > populate the AxisConfiguration with the servlet config ?
> > > > > > >
> > > > > > >
> > > > > >
> > > > > >
> > > >
> ---------------------------------------------------------------------
> > > > > > To unsubscribe, e-mail:
> > > > [EMAIL PROTECTED]
> > > > > > For additional commands, e-mail: [EMAIL PROTECTED]
> > > > > >
> > > > > >
> > > > >
> > > > >
> > > >
> > > >
> > >
> > >
> > > --
> > > Davanum Srinivas : http://www.wso2.net (Oxygen for Web Service
> Developers)
> > >
> >
> >
> > --
> > Davanum Srinivas : http://www.wso2.net (Oxygen for Web Service Developers)
> >
> >
> ---------------------------------------------------------------------
> > To unsubscribe, e-mail:
> [EMAIL PROTECTED]
> > For additional commands, e-mail: [EMAIL PROTECTED]
> >
> >
>
>
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
