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
Welcome! If you don't have a Git account, you can't do anything here.
You can add a comment by following this link or if you reported this bug, you can edit this bug over here.
(description)
Block user comment
Status: Assign to:
Package:
Bug Type:
Summary:
From: Paul dot Merchant at dartmouth dot edu
New email:
PHP Version: OS:

 

 [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-2023 The PHP Group
All rights reserved. 		Last updated: Mon Mar 27 23:03:38 2023 UTC