308439 - XForms Submits will not work with ASP.NET V2 web services

Status: RESOLVED FIXED

(Core Graveyard :: XForms, defect)

Description

[Peter Nunn]

User-Agent:       Mozilla/4.0 (compatible; MSIE 6.0; Windows NT 5.1; formsPlayer 1.3; .NET CLR 2.0.50215; .NET CLR 1.1.4322)
Build Identifier: 

Minor issues with submission in xforms.
Cannot set mediatype to application/soap+xml as the submission looks for an 
attribute called mediaType
Cannot parse returned media type of application/soap+xml

Reproducible: Always

Comment 1

[Peter Nunn]

Attachment: Fix for nsDOMParser to allow application/soap+xml

Allows the XML parser to handle soap returns when interfacing to web services

Comment 2

[Peter Nunn]

Attachment: patch to allow for mediatype

Incorrect spelling of mediatype.  Will always default to application/xml if the
xforms spec spelling of mediatype is used

Comment 3

[Allan Beaufour]

(In reply to comment #2)
> Created an attachment (id=196021) [edit]
> patch to allow for mediatype
> 
> Incorrect spelling of mediatype.  Will always default to application/xml if the
> xforms spec spelling of mediatype is used

Absolutely correct. "mediaType" is wrong.

Comment 4

[Allan Beaufour]

(In reply to comment #1)
> Created an attachment (id=196020) [edit]
> Fix for nsDOMParser to allow application/soap+xml
> 
> Allows the XML parser to handle soap returns when interfacing to web services

It seems reasonable to me to let the DOMParser parse this too. I'm moving the
bug to Core/XML for that (the XForms part is a simple fix).

Comment 5

[Allan Beaufour]

Checked attachment 196021 [details] [diff] [review] in to trunk

Comment 6

[Jonas Sicking (:sicking) No longer reading bugmail consistently]

Comment on attachment 196020 [details] [diff] [review]
Fix for nsDOMParser to allow application/soap+xml

Shouldn't you add this mimetype to the list of mimetypes treated as xml by the
content sinks too?

Comment 7

[Allan Beaufour]

(In reply to comment #6)
> (From update of attachment 196020 [details] [diff] [review] [edit])
> Shouldn't you add this mimetype to the list of mimetypes treated as xml by the
> content sinks too?

I talked with peterv yesterday, and the resolution is that we force-feed
"application/xml" as the content type to the DOMParser. That should work.

So I'm taking the bug back to XForms-land.

Comment 8

[Allan Beaufour]

Attachment: Patch

Here's a simple fix that just sets the content type to "application/xml". If
it's not XML, the parser bails and we'll catch it.

Comment 9

[Olli Pettay [:smaug][[e-mail]]]

Comment on attachment 198286 [details] [diff] [review]
Patch

Changing review request.
Doron is still on vacation.

Comment 10

[Allan Beaufour]

(In reply to comment #8)
> Created an attachment (id=198286) [edit]
> Patch
> 
> Here's a simple fix that just sets the content type to "application/xml". If
> it's not XML, the parser bails and we'll catch it.

Checked in to trunk.

Comment 11

[Asa Dotzler [:asa]]

a checkin comment says this also landed on the 1.8 branch. If that's the case,
can you please add the "fixed1.8" keyword? thanks.

Comment 12

[Allan Beaufour]

(In reply to comment #11)
> a checkin comment says this also landed on the 1.8 branch. If that's the case,
> can you please add the "fixed1.8" keyword? thanks.

Only attachment 196021 [details] [diff] [review] is on branch. Attachment 198286 [details] [diff] is still waiting.

Comment 13

[Doron Rosenberg (IBM)]

checked into branch.