History | Log In     View a printable version of the current page.  
Issue Details (XML | Word | Printable)

Key: ZF-23
Type: Bug Bug
Status: Closed Closed
Resolution: Won't Fix
Priority: Major Major
Assignee: Shahar Evron
Reporter: Naoto Ishikawa
Votes: 0
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
Google issue summary
Zend Framework

Zend_Http_Client::get() always build path string with '?'

Created: 19/Jun/06 01:18 AM   Updated: 05/Jul/07 02:44 PM
Component/s: Zend_Http_Client
Affects Version/s: 0.1.3
Fix Version/s: 0.1.4

Time Tracking:
Not Specified

Resolution Date: 23/Jun/06 12:26 PM


 Description  « Hide
Zend_Http_Client builds a request path string like following code.

http://framework.zend.com/developer/browser/trunk/library/Zend/Http/Client.php#L74
74 $request = array_merge(array('GET ' . $this->_uri->getPath() . '?' . $this->_uri->getQuery() . ' HTTP/1.0',

This code always makes a path string with "?" if URL has no querystring.
ex) "/path/to/file?"
But some webpages return other contents if the path string has "?" or not.
ex) http://wordpress.rauru-block.org/index.php/1354 and http://wordpress.rauru-block.org/index.php/1354?

So please change like this.

74 $request = array_merge(array('GET ' . $this->_uri->getPath() . (($this->_uri->getQuery() !== false) ? '?' . $this->_uri->getQuery() : '') . ' HTTP/1.0',



 All   Comments   Work Log   Change History   FishEye   Crucible      Sort Order: Ascending order - Click to sort in descending order
Shahar Evron - 21/Jun/06 07:38 AM
Development of the HTTP client currently concentrated on the incubator version which is a complete rewrite of the client.

Please test if the problem exists in the incubator version.

To learn how to use the incubator components, read this:
http://framework.zend.com/wiki/display/ZFDEV/Zend+Framework+Subversion+%28SVN%29+Standards#ZendFrameworkSubversion%28SVN%29Standards-UsingtheZendFrameworkIncubatorComponents


Naoto Ishikawa - 21/Jun/06 08:04 AM
Ok, I will test the incubator version.
Please close this issue.

Shahar Evron - 23/Jun/06 12:26 PM
Does not happen in the incubator version - will not fix, please use the incubator (will soon replace the old HTTP client anyway).