Error in: SoapSampler.setPostHeaders(PostMethod post) in the else branch

[asfimport]

Carol Gagne (Bug 48451):
In class: org.apache.jmeter.protocol.http.sampler.SoapSampler
Method: setPostHeaders(PostMethod post)
Source file line: 143
....
} else {
// otherwise we use "text/xml" as the default
post.addParameter(HEADER_CONTENT_TYPE, DEFAULT_CONTENT_TYPE);
}...

I believe it should be something like

    } else {
        // otherwise we use "text/xml" as the default
        post.setRequestHeader(HEADER_CONTENT_TYPE, DEFAULT_CONTENT_TYPE);
    }

How to reproduce: Make sure no header manager is null (not set).
Capture the content of the call (with TCPMon for exemple). You will see the default content type does not appear in HTTP Header but in the content.

Regards

Severity: normal
OS: Windows XP

[asfimport]

Sebb (migrated from Bugzilla):
Thanks for the report and suggested fix.

This has been applied to SVN:

URL: http://svn.apache.org/viewvc?rev=894535&view=rev
Log:
#2323 - Error in: SoapSampler.setPostHeaders(PostMethod post) in the else branch

and will be in nightly builds after r894535