Last Comment Bug 385806 - SOAPAction Header on submission is not being set
: SOAPAction Header on submission is not being set
: fixed1.8.1.5
Product: Core Graveyard
Classification: Graveyard
Component: XForms (show other bugs)
: Trunk
: x86 Windows XP
-- major (vote)
: ---
Assigned To: aaronr
Depends on:
  Show dependency treegraph
Reported: 2007-06-25 14:14 PDT by Keith
Modified: 2016-07-15 14:46 PDT (History)
3 users (show)
See Also:
QA Whiteboard:
Iteration: ---
Points: ---

Web Service submission from XForms (3.50 KB, application/xhtml+xml)
2007-06-25 14:17 PDT, Keith
no flags Details
patch (1.33 KB, patch)
2007-06-25 14:26 PDT, aaronr
doronr: review+
bugs: review+
Details | Diff | Splinter Review

Description User image Keith 2007-06-25 14:14:31 PDT
User-Agent:       Mozilla/4.0 (compatible; MSIE 6.0; Windows NT 5.1; Sidewinder 1.0; formsPlayer 2.0; formsPlayer 1.4; SV1; formsPlayer 1.5; .NET CLR 1.1.4322; .NET CLR 2.0.50727)
Build Identifier: 

Reference  The code to support the SOAPAction header based upon the XForms 1.1 spec. was added to xforms.xpi  Using a SOAP monitor and the test case from 309442, the SOAPAction header was not being viewed in the http headers.  Discussed the problem with Aaron, who subsequently debugged the problem. He said "we are testing the DOMDocument.  We should be checking against the document's documentElement".  Please reference the SerializeDataXML method where the nodeNS and nodeName are tested to determine if the node of the instance is in the SOAP Namespace and is the SOAP "Envelope" element. 

Reproducible: Always

Steps to Reproduce:
1.See Test case
Comment 1 User image Keith 2007-06-25 14:17:06 PDT
Created attachment 269735 [details]
Web Service submission from XForms
Comment 2 User image aaronr 2007-06-25 14:26:00 PDT
Created attachment 269738 [details] [diff] [review]

Simple fix as it turns out.  We were incorrectly looking for SOAPEnvelope on the the document instead of the documentElement.  Doh!
Comment 3 User image Olli Pettay [:smaug] 2007-06-26 05:00:43 PDT
Comment on attachment 269738 [details] [diff] [review]

oops. Regression from Bug 338314 ?
Comment 4 User image aaronr 2007-06-27 12:19:00 PDT
fix committed to trunk and 1.8 branch

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