Last Comment Bug 308439 - XForms Submits will not work with ASP.NET V2 web services
: XForms Submits will not work with ASP.NET V2 web services
Status: RESOLVED FIXED
: fixed1.8
Product: Core Graveyard
Classification: Graveyard
Component: XForms (show other bugs)
: Trunk
: All All
: -- normal (vote)
: ---
Assigned To: aaronr
: Stephen Pride
:
Mentors:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2005-09-14 01:36 PDT by Peter Nunn
Modified: 2016-07-15 14:46 PDT (History)
5 users (show)
See Also:
QA Whiteboard:
Iteration: ---
Points: ---


Attachments
Fix for nsDOMParser to allow application/soap+xml (1.19 KB, patch)
2005-09-14 01:54 PDT, Peter Nunn
no flags Details | Diff | Splinter Review
patch to allow for mediatype (1.15 KB, patch)
2005-09-14 01:55 PDT, Peter Nunn
allan: review+
doronr: review+
Details | Diff | Splinter Review
Patch (1.54 KB, patch)
2005-10-03 02:29 PDT, Allan Beaufour
bugs: review+
aaronr: review+
Details | Diff | Splinter Review

Description Peter Nunn 2005-09-14 01:36:36 PDT
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 2005-09-14 01:54:32 PDT
Created attachment 196020 [details] [diff] [review]
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 2005-09-14 01:55:45 PDT
Created attachment 196021 [details] [diff] [review]
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 2005-09-14 02:07:06 PDT
(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 2005-09-14 02:25:29 PDT
(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 2005-09-15 04:38:11 PDT
Checked attachment 196021 [details] [diff] [review] in to trunk
Comment 6 Jonas Sicking (:sicking) No longer reading bugmail consistently 2005-09-27 13:51:59 PDT
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 2005-09-28 04:46:01 PDT
(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 2005-10-03 02:29:33 PDT
Created attachment 198286 [details] [diff] [review]
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] 2005-10-03 13:25:01 PDT
Comment on attachment 198286 [details] [diff] [review]
Patch

Changing review request.
Doron is still on vacation.
Comment 10 Allan Beaufour 2005-10-03 23:38:17 PDT
(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] 2005-10-10 20:54:34 PDT
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 2005-10-11 00:39:08 PDT
(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) 2005-10-21 09:23:34 PDT
checked into branch.

Note You need to log in before you can comment on or make changes to this bug.