Changeset 475 for trunk/lib/App.inc.php

Timestamp:

Apr 26, 2014 3:04:02 AM (10 years ago)

Author:

anonymous

Message:

Logging longer messages when possible. Fixed compatibility regression with Nav::currentPage(); may break compatability with codebase v2.1.6 only.

File:

• trunk/lib/App.inc.php (modified) (4 diffs)

trunk/lib/App.inc.php

@@ -311 +311 @@
          */
 
-        // Skip session for some user agents.
-        if (preg_match('/Atomz|ApacheBench|Wget/i', getenv('HTTP_USER_AGENT'))) {
-            $this->setParam(array('enable_session' => false));
-        }
-
         // Skip sessions if disabled or automatically skip if run in a CLI script.
         if (true === $this->getParam('enable_session') && !defined('_CLI')) {
@@ -657 +652 @@
             'type'      => $this->logPriorityToString($priority),
             'file:line' => "$file : $line",
-            'url'       => mb_substr(isset($_SERVER['REQUEST_URI']) ? $_SERVER['REQUEST_URI'] : '', 0, 128),
+            'url'       => mb_substr(isset($_SERVER['REQUEST_URI']) ? $_SERVER['REQUEST_URI'] : '', 0, 1024),
             'message'   => $message
         );
+        // Here's a shortened version of event data.
+        $event_short = $event;
+        $event_short['url'] = truncate($event_short['url'], 120);
+
 
         // FILE ACTION
         if (false !== $this->getParam('log_file_priority') && $priority <= $this->getParam('log_file_priority')) {
-            $event_str = '[' . join('] [', $event) . ']';
+            $event_str = '[' . join('] [', $event_short) . ']';
             error_log(mb_substr($event_str, 0, 1024) . "\n", 3, $this->getParam('log_directory') . '/' . $this->getParam('log_filename'));
         }
@@ -683 +682 @@
             $hostname = (isset($_SERVER['HTTP_HOST']) && '' != $_SERVER['HTTP_HOST']) ? $_SERVER['HTTP_HOST'] : php_uname('n');
             $subject = sprintf('[%s %s]', $hostname, $priority);
-            $sms_msg = sprintf('%s [%s:%s]', mb_substr($event['message'], 0, 64), basename($file), $line);
+            $sms_msg = sprintf('%s [%s:%s]', mb_substr($event_short['message'], 0, 64), basename($file), $line);
             $headers = 'From: ' . $this->getParam('site_email');
             mb_send_mail($this->getParam('log_to_sms_address'), $subject, $sms_msg, $headers);
@@ -695 +694 @@
         // Restore original locale.
         setlocale(LC_TIME, $locale);
+
+        unset($event, $event_short);
 
         return true;