Description: ------------ The OCI8 module will always attempt to connect to port 1521, even if a different port is specified in the DSN. Test script: --------------- $dsn = 'oci8://user:pass@10.10.10.1:1522/dbname'; $db =& DB::connect($dsn); if (DB::isError($db)) { echo "Failed connection\n"; } Expected result: ---------------- No error message. Actual result: -------------- Failed connection. A tcpdump shows that it is trying to connect on the default port (1521)

Hmm .. specifying a port is only allowed with the easy connect syntax, or am I mistaken? so in that case the it should use: //[hostspec]:[port] if no port is specified it would simply use [hostspec] so if you want to use the easy connect syntax with the default port then you woulld need to include the "//" in the hostspec .. (actually one would need "///") can you verify this analysis?

I'm not sure I understand what you're asking us to do to verify this bug and/or a workaround. You say "if no port is specified it would simply use [hostspec]", but the whole point is that we are trying to use a port. Can you advise what string to pass as a $dsn to connect using an instantclient when using an alternate port? Thanks, Tom

I am proposing the following change in the code: Today if you want to use the easy connect syntax you would specifiy the entire easy connect string in the hostspec or database ('//hostname:port' or '//hostname'). With the change you could alternatively set the hostname either in the hostspec or database and if you also specify a port then an easy connect string would be assembled as you can see above.

I can confirm that: $dbtype="oci8"; $dbuser="user"; $dbpass="pass"; $dbhost="///host"; $dbport="port"; $dbname="dbname"; works fine. I'm appending the port as well, as in my case, I'm using a non-standard port. $oradsn="$dbtype://$dbuser:$dbpass@$dbhost:$dbport/$dbname"; Thanks, Tom

I have commited the change in MDB2 .. here is the proposed patch for DB retrieving revision 1.103 diff -u -r1.103 oci8.php --- oci8.php 11 Apr 2005 15:10:22 -0000 1.103 +++ oci8.php 30 Mar 2006 19:28:11 -0000 @@ -232,6 +232,9 @@ } else { $db = $dsn['database']; } + if (isset($this->dsn['port'])) { + $db = '//'.$db.':'.$this->dsn['port']; + }

actually the following is propably right + if (isset($this->dsn['port']) && $this->dsn['port']) { + $db = '//'.$db.':'.$this->dsn['port']; + }

This bug has been fixed in CVS. If this was a documentation problem, the fix will appear on pear.php.net by the end of next Sunday (CET). If this was a problem with the pear.php.net website, the change should be live shortly. Otherwise, the fix will appear in the package's next release. Thank you for the report and for helping us make PEAR better.