Axis-C++
  1. Axis-C++
  2. AXISCPP-1059

HTTPTransport generates duplicate "Authentication: " headers

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: current (nightly)
    • Fix Version/s: current (nightly)
    • Component/s: Transport (axis3)
    • Labels:
      None
    • Environment:
      Any

      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*)

        Activity

        Michael Chen created issue -
        nadir amra made changes -
        Field Original Value New Value
        Assignee nadir amra [ nadiramra ]
        nadir amra made changes -
        Status Open [ 1 ] Closed [ 6 ]
        Fix Version/s current (nightly) [ 10476 ]
        Resolution Fixed [ 1 ]
        Mark Thomas made changes -
        Workflow jira [ 12444129 ] Default workflow, editable Closed status [ 12556681 ]
        Mark Thomas made changes -
        Workflow Default workflow, editable Closed status [ 12556681 ] jira [ 12579942 ]

          People

          • Assignee:
            nadir amra
            Reporter:
            Michael Chen
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development