[AXISCPP-1059] HTTPTransport generates duplicate "Authentication: " headers

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: current (nightly)
• Fix Version/s: current (nightly)
• Component/s: Transport (axis3)
• Environment: Any
• Assignee: Nadir K. Amra
• Reporter: Michael Chen
• Created: 14/Oct/08 18:32
• Updated: 08/Nov/10 05:24
• Resolved: 08/Nov/10 05:23

Description

Source file "axis/trunk/c/src/transport/axis3/HTTPTransport.cpp", line 360.
When generateHTTPHeaders() calls setTransportProperty("Authorization", ...), the later does NOT replace existing "Authentication: " header but add another one. My application uses the same stub to call the same service repeatedly. This resulted in increasingly more and more duplicate "Authentication: " header being sent to the service. After a while, the service refuses the request.

The common practice is that "set" replaces duplicate key/value pairs, while "add" implies the acceptance of duplicate keys. setTransportProperty(const char*, const char*) only remove duplicates of 3 know headers, "Connection", "Content-Length" and "SOAPAction". This is a design flaw. There should be two sibling member functions:

setTransportProperty(const char*, const char*)
addTransportProperty(const char*, const char*)