[ 
https://issues.apache.org/jira/browse/AXIS2C-1393?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12847327#action_12847327
 ] 

S.Uthaiyashankar commented on AXIS2C-1393:
------------------------------------------

I agree with Sebastien. For example, tcp_worker.c line 117 passes hardcoded 
value of "AXIOM_SOAP12_SOAP_ENVELOPE_NAMESPACE_URI" when creating soap_builder. 
If someone send SOAP11 message to TCP server, the request will fail. Ideal case 
would be pass NULL, and get the version from soap message. 

> axiom_soap_builder_identify_soap_version should accept parameter 
> soap_version_uri_from_transport to be NULL
> -----------------------------------------------------------------------------------------------------------
>
>                 Key: AXIS2C-1393
>                 URL: https://issues.apache.org/jira/browse/AXIS2C-1393
>             Project: Axis2-C
>          Issue Type: Bug
>          Components: xml/soap
>    Affects Versions: 1.6.0
>            Reporter: Sebastien Bigot
>            Assignee: S.Uthaiyashankar
>             Fix For: 1.7.0
>
>
> Looking at axiom_soap_builder_identify_soap_version in soap_builder.c,
> NULL soap_version_uri_from_transport means that we don't know the version of 
> the soap message to be decoded prior the call to 
> axiom_soap_builder_identify_soap_version.
> You should remove this 
>     AXIS2_PARAM_CHECK(env->error, soap_version_uri_from_transport, 
> AXIS2_FAILURE); at line 873 
> since later we check that the soap message should match  the specified soap 
> version ( line 914), 
> and if it was not specified just set the soap version to the one decoded in 
> the message (line 925, 940).
> Regards,
> Sebastien

-- 
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: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to