Thank you.

Could you please change the message to be "SOAP body is missing or
something" and "SOAP" has to be upper case I guess.

Deepal
> Hi Deepal, 
>
> I have pointed out the relevant part of specs, Please have a look. 
>
> SOAP 1.1 - 
> http://www.w3.org/TR/2000/NOTE-SOAP-20000508/#_Toc478383494
> SOAP 1.2 - http://www.w3.org/TR/2007/REC-soap12-part1-20070427/#soapenv
>
> Regards,
> Shameera.
>
> On Sun, May 5, 2013 at 8:16 PM, Deepal Jayasinghe
> <[email protected] <mailto:[email protected]>> wrote:
>
>
>     > Author: shameera
>     > Date: Sun May  5 14:33:41 2013
>     > New Revision: 1479332
>     >
>     > URL: http://svn.apache.org/r1479332
>     > Log:
>     > Fixed AXIS2-4221, check for soap body after build soap envelop
>     >
>     > Modified:
>     >    
>     
> axis/axis2/java/core/trunk/modules/kernel/src/org/apache/axis2/transport/TransportUtils.java
>     >
>     > Modified:
>     
> axis/axis2/java/core/trunk/modules/kernel/src/org/apache/axis2/transport/TransportUtils.java
>     > URL:
>     
> http://svn.apache.org/viewvc/axis/axis2/java/core/trunk/modules/kernel/src/org/apache/axis2/transport/TransportUtils.java?rev=1479332&r1=1479331&r2=1479332&view=diff
>     >
>     
> ==============================================================================
>     > ---
>     
> axis/axis2/java/core/trunk/modules/kernel/src/org/apache/axis2/transport/TransportUtils.java
>     (original)
>     > +++
>     
> axis/axis2/java/core/trunk/modules/kernel/src/org/apache/axis2/transport/TransportUtils.java
>     Sun May  5 14:33:41 2013
>     > @@ -169,7 +169,7 @@ public class TransportUtils {
>     >          return createSOAPEnvelope(documentElement);
>     >      }
>     >
>     > -    public static SOAPEnvelope createSOAPEnvelope(OMElement
>     documentElement) {
>     > +    public static SOAPEnvelope createSOAPEnvelope(OMElement
>     documentElement) throws AxisFault {
>     >          SOAPEnvelope envelope;
>     >          // Check whether we have received a SOAPEnvelope or not
>     >          if (documentElement instanceof SOAPEnvelope) {
>     > @@ -183,6 +183,10 @@ public class TransportUtils {
>     >                  envelope.getBody().addChild(documentElement);
>     >              }
>     >          }
>     > +        //According to both SOAP 1.1 and 1.2 specification SOAP
>     body is mandatory element for a valid soap envelop
>     > +        if (envelope.getBody() == null) {
>     > +            throw new AxisFault("soap body element is null, but
>     it is mandatory for a valid soap envelop");
>     What do you mean by 'SOAP body element is null' ? did you mean to
>     say it
>     is missing ?. And I am not 100% sure whether the SOAP body is
>     mandatory
>     (need to read the spec :)).
>
>
>     Deepal
>     > +        }
>     >          return envelope;
>     >      }
>     >
>     >
>     >
>     >
>
>     --
>     Blog - http://blogs.deepal.org/
>
>     ---------------------------------------------------------------------
>     To unsubscribe, e-mail: [email protected]
>     <mailto:[email protected]>
>     For additional commands, e-mail: [email protected]
>     <mailto:[email protected]>
>
>
>
>
> -- 
> Best Regards,
> Shameera Rathnayaka.
>
> email: shameera AT apache.org <http://apache.org> , shameerainfo AT
> gmail.com <http://gmail.com>
> Blog : http://shameerarathnayaka.blogspot.com/

Reply via email to