From 871f598dcc81d7018f25590d767591722bb977f2 Mon Sep 17 00:00:00 2001
From: Evan Prodromou <evan@controlyourself.ca>
Date: Sat, 27 Jun 2009 08:11:09 -0700
Subject: [PATCH] debug flag for sessions

---
 README              | 2 ++
 classes/Session.php | 4 +++-
 lib/common.php      | 3 ++-
 3 files changed, 7 insertions(+), 2 deletions(-)

diff --git a/README b/README
index 7f8748b3fd..c8c529ed86 100644
--- a/README
+++ b/README
@@ -1287,6 +1287,8 @@ handle: boolean. Whether we should register our own PHP session-handling
 	code (using the database and memcache if enabled). Defaults to false.
 	Setting this to true makes some sense on large or multi-server
 	sites, but it probably won't hurt for smaller ones, either.
+debug: whether to output debugging info for session storage. Can help
+       with weird session bugs, sometimes. Default false.
 
 Troubleshooting
 ===============
diff --git a/classes/Session.php b/classes/Session.php
index 5c48e4aa90..93fd99baa8 100644
--- a/classes/Session.php
+++ b/classes/Session.php
@@ -42,7 +42,9 @@ class Session extends Memcached_DataObject
 
     static function logdeb($msg)
     {
-        common_debug("Session: " . $msg);
+        if (common_config('sessions', 'debug')) {
+            common_debug("Session: " . $msg);
+        }
     }
 
     static function open($save_path, $session_name)
diff --git a/lib/common.php b/lib/common.php
index 3a5913f85c..e2936f0751 100644
--- a/lib/common.php
+++ b/lib/common.php
@@ -255,7 +255,8 @@ $config =
         'search' =>
         array('type' => 'fulltext'),
         'sessions' =>
-        array('handle' => false), // whether to handle sessions ourselves
+        array('handle' => false, // whether to handle sessions ourselves
+              'debug' => false), // debugging output for sessions
         );
 
 $config['db'] = &PEAR::getStaticProperty('DB_DataObject','options');