From 3b4662a58e0673de4172e5b0e964c53a0a67af59 Mon Sep 17 00:00:00 2001 From: Dan McGee Date: Mon, 29 Dec 2008 20:46:06 -0600 Subject: Use non-persistant connections There is really no need to use persistent connections to the database in this day and age. Most PHP development guides recommend against it, and the new mysqli interface doesn't even include the functionality. Add a matching but currently unused db_disconnect() function while we are at it. Reference counting will cover us for the most part, however. Signed-off-by: Dan McGee Signed-off-by: Loui Chang --- web/lib/aur.inc | 19 +++++++++++++++---- 1 file changed, 15 insertions(+), 4 deletions(-) (limited to 'web/lib') diff --git a/web/lib/aur.inc b/web/lib/aur.inc index 22bd4e7..d4ac772 100644 --- a/web/lib/aur.inc +++ b/web/lib/aur.inc @@ -261,8 +261,7 @@ function uid_from_sid($sid="") { # connect to the database # function db_connect() { - - $handle = mysql_pconnect(AUR_db_host, AUR_db_user, AUR_db_pass); + $handle = mysql_connect(AUR_db_host, AUR_db_user, AUR_db_pass); if (!$handle) { die("Error connecting to AUR database: " . mysql_error()); } @@ -273,15 +272,27 @@ function db_connect() { return $handle; } +# disconnect from the database +# this won't normally be needed as PHP/reference counting will take care of +# closing the connection once it is no longer referenced +# +function db_disconnect($db_handle="") { + if ($db_handle) { + mysql_close($db_handle); + return TRUE; + } + return FALSE; +} + # wrapper function around db_query in case we want to put -# query logging/debuggin in. +# query logging/debugging in. # function db_query($query="", $db_handle="") { if (!$query) { return FALSE; } if (!$db_handle) { - $db_handle = db_connect(); + die("DB handle was not provided to db_query"); } $result = @mysql_query($query, $db_handle); return $result; -- cgit v1.2.3-54-g00ecf