php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login

go to bug id or search bugs for

Bug #38210 oci8.ping_interval interpreted incorrectly
Submitted: 2006-07-25 17:46 UTC Modified: 2006-07-26 07:01 UTC
From: Paul dot Merchant at dartmouth dot edu Assigned:
Status: Closed Package: OCI8 related
PHP Version: 5.1.4 OS: Any
Private report: No CVE-ID: None
View Add Comment Developer Edit
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:
32 - 11 = ?
Subscribe to this entry?

 
 [2006-07-25 17:46 UTC] 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) &&....

Patches

Add a Patch

Pull Requests

Add a Pull Request

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2006-07-26 07:01 UTC] tony2001@php.net 
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.
 
PHP Copyright © 2001-2024 The PHP Group
All rights reserved. 		Last updated: Fri Apr 19 23:01:28 2024 UTC