Issue Details (XML | Word | Printable)

Key: CAMEL-727
Type: Bug Bug
Status: Closed Closed
Resolution: Fixed
Priority: Major Major
Assignee: Willem Jiang
Reporter: William Tam
Votes: 0
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
Apache Camel

Propagated CXF header's value should not include quotes

Created: 15/Jul/08 11:35 AM   Updated: 17/Jul/08 06:13 AM
Return to search
Component/s: camel-cxf
Affects Version/s: 1.3.0
Fix Version/s: 1.4.0

Time Tracking:
Not Specified

File Attachments:
  Size
Text File Licensed for inclusion in ASF works test_patch.txt 2008-07-15 01:34 PM William Tam 7 kB


 Description  « Hide
CAMEL-686 fixed the issue of not propagating transport headers (e.g. SOAPAction) but it looks like we added quotes around the value (i.e. getCustomer becomes "getCustomer"). I updated the CustomerServicesTest to illustrate the problem. Also, I cleaned up the client/server-application.xml.

 All   Comments   Work Log   Change History   Subversion Commits   FishEye   Crucible      Sort Order: Ascending order - Click to sort in descending order
William Tam made changes - 15/Jul/08 11:39 AM
Field Original Value New Value
Attachment test_patch.txt [ 16739 ]
William Tam added a comment - 15/Jul/08 12:35 PM - edited
It is not a bug as the quotes came from CXF. CXF puts quotes around the SOAPAction's value. Anyway, I think we should apply the patch to verify header values in the test because activemq does not look for SOAPAction header (like TIBCO EMS) so the old test was not verifying the header propagation functionality.

William Tam made changes - 15/Jul/08 12:35 PM
Resolution Fixed [ 1 ]
Status Open [ 1 ] Resolved [ 5 ]
William Tam made changes - 15/Jul/08 12:38 PM
Attachment test_patch.txt [ 16739 ]
William Tam made changes - 15/Jul/08 12:38 PM
Attachment test_patch.txt [ 16741 ]
William Tam made changes - 15/Jul/08 01:34 PM
Attachment test_patch.txt [ 16741 ]
William Tam made changes - 15/Jul/08 01:34 PM
Attachment test_patch.txt [ 16742 ]
Willem Jiang made changes - 15/Jul/08 09:44 PM
Assignee Willem Jiang [ njiang ]
Willem Jiang added a comment - 15/Jul/08 10:02 PM
Applied test patch with thanks to William.

http://svn.apache.org/viewvc?rev=677156&view=rev


Willem Jiang made changes - 15/Jul/08 10:02 PM
Status Resolved [ 5 ] Closed [ 6 ]
Hadrian Zbarcea made changes - 17/Jul/08 06:13 AM
Fix Version/s 1.5.0 [ 11922 ]
Fix Version/s 1.4.0 [ 11849 ]
Create crucible review for this changeset in