Bug #13038 Updated: Cookie Error in Net/Curl.php

From: Date: Mon, 22 Oct 2001 18:53:24 +0000
Subject: Bug #13038 Updated: Cookie Error in Net/Curl.php
References: 1  Groups: php.dev 
Request: Send a blank email to php-dev+get-68722@lists.php.net to get a copy of this message 
ID: 13038
Updated by: sterling
Reported By: olaf@sevenval.de
Old Status: Open
Status: Closed
Bug Type: HTTP related
Operating System: all
PHP Version: 4.0.6
New Comment:

fixed in cvs, thanks.

Previous Comments:
------------------------------------------------------------------------

[2001-08-30 08:06:47] olaf@sevenval.de

line 319:       
$ret = curl_setopt($ch, CURLOPT_COOKIE, $cookie_data);       
may cause a warning, if cookies is an empty array.       
Better:       
if(!empty($cookie_data){       
 $ret = curl_setopt($ch, CURLOPT_COOKIE, $cookie_data);   }


------------------------------------------------------------------------

[2001-08-29 13:26:03] olaf@sevenval.de

In Curl.php around line 316 multiple cookies are concatenated with newline. Shold be semicolon:
if (isset($this->cookies)) {
  foreach ($this->cookies as $name => $value) {
  $cookie_data .= urlencode($name) . ": " .\ urlencode($value);
 $cookie_data .= ";"; // This is correct, before it was "\n"
}


------------------------------------------------------------------------



Edit this bug report at http://bugs.php.net/?id=13038&edit=1

Thread (3 messages)

« previous php.dev (#68722) next »