You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@servicemix.apache.org by "stefan klinger (JIRA)" <ji...@apache.org> on 2006/04/25 12:31:28 UTC

[jira] Created: (SM-409) Do not assume that first part of multi mime message is xml

Do not assume that first part of multi mime message is xml 
-----------------------------------------------------------

         Key: SM-409
         URL: https://issues.apache.org/activemq/browse/SM-409
     Project: ServiceMix
        Type: Improvement

  Components: servicemix-soap  
    Versions: incubation    
    Reporter: stefan klinger
    Priority: Minor
     Fix For: incubation


Currently, the SoapReader expects the start part to be of xml format. If that part does not exist, it assumes it's the first part. I would prefer that it checks for xml parts and uses the first one it finds in that case. If none are found, insert a dummy source e.g. <payload/> similar to the lightweight HttpMarshaler.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   https://issues.apache.org/activemq/secure/Administrators.jspa
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira

[jira] Resolved: (SM-409) Do not assume that first part of multi mime message is xml

Posted by "Guillaume Nodet (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/activemq/browse/SM-409?page=all ]
     
Guillaume Nodet resolved SM-409:
--------------------------------

    Fix Version: 3.0-M2
                     (was: incubation)
     Resolution: Fixed
      Assign To: Guillaume Nodet

Author: gnodet
Date: Thu Apr 27 02:43:07 2006
New Revision: 397492

URL: http://svn.apache.org/viewcvs?rev=397492&view=rev
Log:
SM-409: Do not assume that first part of multi mime message is xml
Patch provided by Stefan Klinger, thx !

Modified:
    incubator/servicemix/trunk/servicemix-soap/src/main/java/org/apache/servicemix/soap/marshalers/SoapReader.java
    incubator/servicemix/trunk/servicemix-soap/src/test/java/org/apache/servicemix/soap/marshalers/SoapMessageMarshalerTest.java



> Do not assume that first part of multi mime message is xml
> ----------------------------------------------------------
>
>          Key: SM-409
>          URL: https://issues.apache.org/activemq/browse/SM-409
>      Project: ServiceMix
>         Type: Improvement

>   Components: servicemix-soap
>     Versions: incubation
>     Reporter: stefan klinger
>     Assignee: Guillaume Nodet
>     Priority: Minor
>      Fix For: 3.0-M2
>  Attachments: servicemix-soap-diff.txt
>
>
> Currently, the SoapReader expects the start part to be of xml format. If that part does not exist, it assumes it's the first part. I would prefer that it checks for xml parts and uses the first one it finds in that case. If none are found, insert a dummy source e.g. <payload/> similar to the lightweight HttpMarshaler.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   https://issues.apache.org/activemq/secure/Administrators.jspa
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira

[jira] Updated: (SM-409) Do not assume that first part of multi mime message is xml

Posted by "stefan klinger (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/activemq/browse/SM-409?page=all ]

stefan klinger updated SM-409:
------------------------------

    Attachment: servicemix-soap-diff.txt

This diff file fixes the problem by checking for the start tag of a multipart mime message , if it does exist then use the xml as the message source. If it doesn't exist, it checks for all other mime parts for xml, and uses the first xml it comes across as the message source. if no xml exist, a dummy payload message is inserted. Also includes a test case.

> Do not assume that first part of multi mime message is xml
> ----------------------------------------------------------
>
>          Key: SM-409
>          URL: https://issues.apache.org/activemq/browse/SM-409
>      Project: ServiceMix
>         Type: Improvement

>   Components: servicemix-soap
>     Versions: incubation
>     Reporter: stefan klinger
>     Priority: Minor
>      Fix For: incubation
>  Attachments: servicemix-soap-diff.txt
>
>
> Currently, the SoapReader expects the start part to be of xml format. If that part does not exist, it assumes it's the first part. I would prefer that it checks for xml parts and uses the first one it finds in that case. If none are found, insert a dummy source e.g. <payload/> similar to the lightweight HttpMarshaler.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   https://issues.apache.org/activemq/secure/Administrators.jspa
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira