Package home | Report new bug | New search | Development Roadmap Status: Open | Feedback | All | Closed Since Version 1.4.4

Bug #1037 Cannot connect to port 80 through HTTPS
Submitted: 2004-03-19 14:44 UTC
From: jausions Assigned: avb
Status: Closed Package: HTTP_Request
PHP Version: Irrelevant OS: Any
Roadmaps: (Not assigned)    
Subscription  
Comments Add Comment Add patch


Anyone can comment on a bug. Have a simpler test case? Does it work for you on a different platform? Let us know! Just going to say 'Me too!'? Don't clutter the database with that please !
Your email address:
MUST BE VALID
Solve the problem : 48 + 50 = ?

 
 [2004-03-19 14:44 UTC] jausions
Description: ------------ Due to a bug in Net_URL that HTTP_Request::setURL() attempts to correct, it is not possible to connect to port 80 through HTTPS (i.e. https://somewhere.com:80/) (Net_URL assigns port 80 to an URL when no specific port number is present) Created relevant bug #1036 for Net_URL package. Submitted patch for HTTP_Request to pear-dev@lists.php.net Reproduce code: --------------- <?php require_once('HTTP/Request.php'); $oHTTP = new HTTP_Request('https://somewhere.com:80/'); print_r($oHTTP->_url); ?> Expected result: ---------------- net_url Object ( [url] => https://somewhere.com:80/ [protocol] => https [username] => [password] => [host] => somewhere.com [port] => 80 [path] => / [querystring] => Array ( ) [anchor] => [useBrackets] => [user] => [pass] => ) Actual result: -------------- net_url Object ( [url] => https://somewhere.com/ [protocol] => https [username] => [password] => [host] => somewhere.com [port] => 443 [path] => / [querystring] => Array ( ) [anchor] => [useBrackets] => [user] => [pass] => )

Comments

 [2004-03-20 20:52 UTC] avb
A link to the patch: http://marc.theaimsgroup.com/?l=pear-dev&m=107970924225879&w=2 Please do not mix the whitespace related fixes and behaviour fixes in one patch. It is quite difficult to review it.
 [2004-03-20 21:30 UTC] Phiippe dot Jausions at 11abacus dot com
Sorry about the whitespaces, most likely some tabs that got converted to spaces or something like that. I thought I configured my editor properly to match PEAR indentation standard though... I'll double check. Anyway, the change for this package is merely commenting lines 309-314 in HTTP/Request.php. This change MUST be made in relation to bug #1036 correction. I hope this helps. -Philippe
 [2004-03-23 15:16 UTC] avb
Marking this Suspended till bug #1036 is fixed.
 [2004-05-08 19:00 UTC] avb
Looks like the bug #1036 is fixed, will apply the patch soon.
 [2004-05-14 22:02 UTC] jausions
(e-mail address correction) By the way, Net_URL fixed the bug #1036. -Philippe
 [2004-05-19 10:57 UTC] avb
Thanks, fixed in release 1.2.2