axis-java-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Saminda Wishwajith Abeyruwan (JIRA)" <j...@apache.org>
Subject [jira] Updated: (AXIS2-3710) fixes for backward compatibility
Date Mon, 07 Apr 2008 10:44:24 GMT

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

Saminda Wishwajith Abeyruwan updated AXIS2-3710:
------------------------------------------------

    Attachment: axisservlet.patch

fix 

> fixes for backward compatibility 
> ---------------------------------
>
>                 Key: AXIS2-3710
>                 URL: https://issues.apache.org/jira/browse/AXIS2-3710
>             Project: Axis 2.0 (Axis2)
>          Issue Type: Bug
>          Components: kernel
>    Affects Versions: 1.4
>         Environment: Any
>            Reporter: Saminda Wishwajith Abeyruwan
>            Assignee: Davanum Srinivas
>            Priority: Blocker
>             Fix For: 1.4
>
>         Attachments: axisservlet.patch
>
>
> Hi,
> Is there a particular reason to add the line.
> msgContext.setProperty(HTTPCon
>     stants.MC_HTTP_SERVLETCONTEXT, getServletContext());
> You will be able to access the context from http servlet request object.
> Beside the prior added line will break some of the existing production code.
> getServletContext() will work properly, IFF the user's servlet inherit the init method
or if over-rides, it should called the parent init method.
> In AxisServlet, yes it dose over-rides the init method and calls the parent init.
> But those of us who have extended AxisServlet to work with, where over-rides the init
method  to intialize the AxisConfiguration the way our production system wants  does not call
the parents (AxisServlet) init method.
> ex:
> public class MyServlet extends AxisServlet {
>     ...
>    public void init(ServletConfig config) {
>       // my way of initializing axisconfiguration and listeners rather using parent
>      // init, but inheriting parents do* from AxisServlet.
>   }
> }
> We used AxisServlet as show above for quite a long  time.
> I'm afraid the prior line of code  has broken the backward compatibility of re-using
AxisServlet in production environments.
> Thus, the correct way is to obtain the servlet context from request, rather setting the
property.
> Thus, I would humbly ask to revert the commit to preserve backward compatibility.
> Thank you!

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


---------------------------------------------------------------------
To unsubscribe, e-mail: axis-dev-unsubscribe@ws.apache.org
For additional commands, e-mail: axis-dev-help@ws.apache.org


Mime
View raw message