You are viewing a plain text version of this content. The canonical link for it is here.
Posted to java-dev@axis.apache.org by Isuru Suriarachchi <is...@gmail.com> on 2011/07/11 11:48:07 UTC

Issue in AbstractTransportListenerEx

Hi Andreas,

Following check is done in the getEPRsForService method of the above class..

//Strip out the operation name
        if (serviceName.indexOf('/') != -1) {
            serviceName = serviceName.substring(0,
serviceName.indexOf('/'));
        }

Due to this, endpoints are not calculated for hierarchical services (because
we use '/' to separate hierarchical parts). When I get rid of the above
check everything works fine.

So can there be situations where service-name/operation-name is passed for
the serviceName parameter of the above method?

Thanks,
~Isuru

-- 
Technical Lead,
WSO2 Inc. http://wso2.org/
Blog : http://isurues.wordpress.com/

Re: Issue in AbstractTransportListenerEx

Posted by Isuru Suriarachchi <is...@gmail.com>.
Ok :). I'll proceed with the fix..

Thanks,
~Isuru

On Tue, Jul 12, 2011 at 12:00 AM, Andreas Veithen <andreas.veithen@gmail.com
> wrote:

> That is definitely not my handwriting ;-)
>
> That appears to be a very old piece of code introduced in r447298 and
> that I recovered from one of the preexisting transport implementations
> (probably JMS). I remember that I wondered what the purpose of this
> code is...
>
> Andreas
>
> On Mon, Jul 11, 2011 at 11:48, Isuru Suriarachchi <is...@gmail.com>
> wrote:
> > Hi Andreas,
> > Following check is done in the getEPRsForService method of the above
> class..
> > //Strip out the operation name
> >         if (serviceName.indexOf('/') != -1) {
> >             serviceName = serviceName.substring(0,
> > serviceName.indexOf('/'));
> >         }
> > Due to this, endpoints are not calculated for hierarchical services
> (because
> > we use '/' to separate hierarchical parts). When I get rid of the above
> > check everything works fine.
> > So can there be situations where service-name/operation-name is passed
> for
> > the serviceName parameter of the above method?
> > Thanks,
> > ~Isuru
> >
> > --
> > Technical Lead,
> > WSO2 Inc. http://wso2.org/
> > Blog : http://isurues.wordpress.com/
> >
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: java-dev-unsubscribe@axis.apache.org
> For additional commands, e-mail: java-dev-help@axis.apache.org
>
>


-- 
Technical Lead,
WSO2 Inc. http://wso2.org/
Blog : http://isurues.wordpress.com/

Re: Issue in AbstractTransportListenerEx

Posted by Andreas Veithen <an...@gmail.com>.
That is definitely not my handwriting ;-)

That appears to be a very old piece of code introduced in r447298 and
that I recovered from one of the preexisting transport implementations
(probably JMS). I remember that I wondered what the purpose of this
code is...

Andreas

On Mon, Jul 11, 2011 at 11:48, Isuru Suriarachchi <is...@gmail.com> wrote:
> Hi Andreas,
> Following check is done in the getEPRsForService method of the above class..
> //Strip out the operation name
>         if (serviceName.indexOf('/') != -1) {
>             serviceName = serviceName.substring(0,
> serviceName.indexOf('/'));
>         }
> Due to this, endpoints are not calculated for hierarchical services (because
> we use '/' to separate hierarchical parts). When I get rid of the above
> check everything works fine.
> So can there be situations where service-name/operation-name is passed for
> the serviceName parameter of the above method?
> Thanks,
> ~Isuru
>
> --
> Technical Lead,
> WSO2 Inc. http://wso2.org/
> Blog : http://isurues.wordpress.com/
>

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