#38210 [Opn->Csd]: oci8.ping_interval interpreted incorrectly

ID: 38210 Updated by: tony2001@php.net Reported By: Paul dot Merchant at dartmouth dot edu -Status: Open +Status: Closed Bug Type: OCI8 related Operating System: Any PHP Version: 5.1.4 New Comment: This bug has been fixed in CVS. Snapshots of the sources are packaged every three hours; this change will be in the next snapshot. You can grab the snapshot at http://snaps.php.net/. Thank you for the report, and for helping us make PHP better. Previous Comments: ------------------------------------------------------------------------ [2006-07-25 17:46:02] Paul dot Merchant at dartmouth dot edu Description: ------------ A ping interval of 0 should cause a persistent connection to be polled each time php_oci_do_connect is called. It does this by adding ping_interval seconds to the timestamp of the last operation to compute a next ping time; however, php_oci_do_connect only performs a ping when the current timestamp is greater than the next ping time, guaranteeing that at least 1 second must pass before the next ping occurs. Effectively, php_oci_connection_ping is called every ping_interval+1 seconds. The comparison on php_oci_do_connect should read if ( (connection->next_ping > 0) && (timestamp >= connection- >next_ping) &&.... ------------------------------------------------------------------------ -- Edit this bug report at http://bugs.php.net/?id=38210&edit=1