You are viewing a plain text version of this content. The canonical link for it is here.
Posted to axis-cvs@ws.apache.org by du...@apache.org on 2007/03/01 16:04:26 UTC

svn commit: r513367 - /webservices/axis/trunk/proposals/dug/java/src/org/apache/axis/transport/http/HTTPSender.java

Author: dug
Date: Thu Mar  1 07:04:25 2007
New Revision: 513367

URL: http://svn.apache.org/viewvc?view=rev&rev=513367
Log:
sync with head

Modified:
    webservices/axis/trunk/proposals/dug/java/src/org/apache/axis/transport/http/HTTPSender.java

Modified: webservices/axis/trunk/proposals/dug/java/src/org/apache/axis/transport/http/HTTPSender.java
URL: http://svn.apache.org/viewvc/webservices/axis/trunk/proposals/dug/java/src/org/apache/axis/transport/http/HTTPSender.java?view=diff&rev=513367&r1=513366&r2=513367
==============================================================================
--- webservices/axis/trunk/proposals/dug/java/src/org/apache/axis/transport/http/HTTPSender.java (original)
+++ webservices/axis/trunk/proposals/dug/java/src/org/apache/axis/transport/http/HTTPSender.java Thu Mar  1 07:04:25 2007
@@ -531,26 +531,25 @@
     private void fillHeaders(MessageContext msgContext, String header, StringBuffer otherHeaders) {
         Object ck1 = msgContext.getProperty(header);
         if (ck1 != null) {
+            // Do we have multiple values to set?
             if (ck1 instanceof String[]) {
                 String [] cookies = (String[]) ck1;
+                otherHeaders.append(header).append(": ");
                 for (int i = 0; i < cookies.length; i++) {
-                    addCookie(otherHeaders, header, cookies[i]);
+                    // See bug AXIS-2064: https://issues.apache.org/jira/browse/AXIS-2064
+                    // Append the cookie
+                    otherHeaders.append(cookies[i]);
+                    // Append a ';' if not the last cookie
+                    if (i < cookies.length -1)
+                        otherHeaders.append(";");
                 }
+                otherHeaders.append("\r\n");
             } else {
-                addCookie(otherHeaders, header, (String) ck1);
+                // Simple string value
+                otherHeaders.append(header).append(": ")
+                        .append((String) ck1).append("\r\n");
             }
         }
-    }
-
-    /**
-     * add cookie to headers
-     * @param otherHeaders
-     * @param header
-     * @param cookie
-     */
-    private void addCookie(StringBuffer otherHeaders, String header, String cookie) {
-        otherHeaders.append(header).append(": ")
-                .append(cookie).append("\r\n");
     }
 
     private InputStream readHeadersFromSocket(SocketHolder sockHolder,



---------------------------------------------------------------------
To unsubscribe, e-mail: axis-cvs-unsubscribe@ws.apache.org
For additional commands, e-mail: axis-cvs-help@ws.apache.org