Synchronize no-cache headers. This reverses revision 13940 because no-cahce headers...

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@13945 7612ce4b-ef26-0410-bec9-ea0150e637f0

diff --git a/doc/ChangeLog b/doc/ChangeLog
index f9b4d179657ff144081941776538656114a99b6c..50f062af4017da336e77558397aeb845be7f915a 100644 (file)
--- a/doc/ChangeLog
+++ b/doc/ChangeLog
@@ -337,7 +337,6 @@ Version 1.5.2 - SVN
     leakage when Firefox does DNS prefetching for URLs contained in emails.
   - Added the ability to configure Google Mail (Gmail) as the mail server
     behind SquirrelMail.
-  - Explicitly disable caching for left_main and right_main pages (#2983134).
   - Fix error with SpamCop reporting plugin not being able to send report as
     emails (#1795310).
   - Fix typo in SpamCop plugin.

diff --git a/functions/page_header.php b/functions/page_header.php
index 744b26d6717df4e01eb2180174c20b57efc21ad6..f6d1e80a30f16e9fc2bd9f85b660b0ae389bf4d8 100644 (file)
--- a/functions/page_header.php
+++ b/functions/page_header.php
@@ -24,9 +24,12 @@ include_once(SM_PATH . 'functions/imap_mailbox.php');
  * @param string xtra extra HTML to insert into the header
  * @param bool do_hook whether to execute hooks, default true
  * @param bool frames generate html frameset doctype (since 1.5.1)
+ * @param bool $browser_cache_ok When TRUE, it's OK to leave out the
+ *                               no-cache browser headers (OPTIONAL;
+ *                               default = FALSE, send no-cache headers)
  * @return void
  */
-function displayHtmlHeader( $title = 'SquirrelMail', $xtra = '', $do_hook = TRUE, $frames = FALSE ) {
+function displayHtmlHeader( $title = 'SquirrelMail', $xtra = '', $do_hook = TRUE, $frames = FALSE, $browser_cache_ok=FALSE ) {
     global $squirrelmail_language, $sTemplateID, $oErrorHandler, $oTemplate;
 
     if ( !sqgetGlobalVar('base_uri', $base_uri, SQ_SESSION) ) {
@@ -36,10 +39,15 @@ function displayHtmlHeader( $title = 'SquirrelMail', $xtra = '', $do_hook = TRUE
         $default_fontset, $chosen_fontset, $default_fontsize, $chosen_fontsize, 
         $chosen_theme, $chosen_theme_path, $user_themes, $user_theme_default;
 
-    /* add no cache headers here */
+    // add no cache headers here
+    //
+    if (!$browser_cache_ok) {
 //FIXME: should change all header() calls in SM core to use $oTemplate->header()!!
-    $oTemplate->header('Pragma: no-cache'); // http 1.0 (rfc1945)
-    $oTemplate->header('Cache-Control: private, no-cache, no-store'); // http 1.1 (rfc2616)
+        $oTemplate->header('Pragma: no-cache'); // http 1.0 (rfc1945)
+        $oTemplate->header('Cache-Control: private, no-cache, no-store, must-revalidate, max-age=0'); // http 1.1 (rfc2616)
+        $oTemplate->header('Expires: Sat, 1 Jan 2000 00:00:00 GMT');
+//TODO: is this needed? $oTemplate->header('Last-Modified: ' . gmdate('D, d M Y H:i:s') . 'GMT');
+    }
     /* prevent information leakage about read emails by forbidding Firefox
      * to do preemptive DNS requests for any links in the message body. */
     $oTemplate->header('X-DNS-Prefetch-Control: off');

diff --git a/src/left_main.php b/src/left_main.php
index 24f32253c2e55dad2d73e1e58bb1e6450411f81f..64324429d280ad68114b37d18f5af62f9e2141b0 100644 (file)
--- a/src/left_main.php
+++ b/src/left_main.php
@@ -15,11 +15,6 @@
 /** This is the left_main page */
 define('PAGE_NAME', 'left_main');
 
-/* Disable browser caching */
-header('Cache-Control: no-cache, no-store, must-revalidate');
-header('Pragma: no-cache');
-header('Expires: ' . gmdate(DATE_RFC1123, time()-1));
-
 /**
  * Include the SquirrelMail initialization file.
  */

diff --git a/src/mailto.php b/src/mailto.php
index 3981c5bd8db33f6df30dce010d223d9b34636a0c..edbd5d71b22733c01274a3005a544cba8ec4f710 100644 (file)
--- a/src/mailto.php
+++ b/src/mailto.php
@@ -64,7 +64,11 @@ $force_login = FALSE;
 $compose_only = FALSE;
 
 
+// Disable browser caching
+//
+header('Cache-Control: no-cache, no-store, must-revalidate, max-age=0');
 header('Pragma: no-cache');
+header('Expires: Sat, 1 Jan 2000 00:00:00 GMT');
 
 $trtable = array('cc'           => 'cc',
                  'bcc'          => 'bcc',

diff --git a/src/redirect.php b/src/redirect.php
index fdf8d0967479714c365a03d761af199517a2f399..d8253f638c8f2e2c4f3fb6a67a3bb557d342df38 100644 (file)
--- a/src/redirect.php
+++ b/src/redirect.php
@@ -23,7 +23,11 @@ require('../include/init.php');
 require_once(SM_PATH . 'functions/imap_general.php');
 require_once(SM_PATH . 'functions/strings.php');
 
+// Disable browser caching
+//
+header('Cache-Control: no-cache, no-store, must-revalidate, max-age=0');
 header('Pragma: no-cache');
+header('Expires: Sat, 1 Jan 2000 00:00:00 GMT');
 $location = get_location();
 
 // session_set_cookie_params (0, $base_uri);

diff --git a/src/right_main.php b/src/right_main.php
index ef2797da4c20444e47852c86ac43245e438485a8..94bd618ea01aa0babdaa388c8d21d6a964214d48 100644 (file)
--- a/src/right_main.php
+++ b/src/right_main.php
@@ -17,11 +17,6 @@ define('PAGE_NAME', 'right_main');
 
 //xdebug_start_profiling("/var/spool/xdebug/right_main.txt");
 
-/* Disable browser caching */
-header('Cache-Control: no-cache, no-store, must-revalidate');
-header('Pragma: no-cache');
-header('Expires: ' . gmdate(DATE_RFC1123, time()-1));
-
 /**
  * Include the SquirrelMail initialization file.
  */

diff --git a/src/squirrelmail_rpc.php b/src/squirrelmail_rpc.php
index 787b7d28bb61a5bcd7915fa47cc6575cf4660c36..651718d1da57380cd7ab03627a840d89e8ebb3dd 100644 (file)
--- a/src/squirrelmail_rpc.php
+++ b/src/squirrelmail_rpc.php
@@ -100,12 +100,10 @@ if (!sqGetGlobalVar('rpc_action', $rpc_action, SQ_FORM)) {
   */
 $oTemplate->header('Content-Type: text/xml');
 $oTemplate->header('Content-Type: application/xml'); // required by IE
-//FIXME: which anti-cache headers do we want to use?
-$oTemplate->header('Cache-Control: no-cache');
-// $oTemplate->header("Expires: Sat, 1 Jan 2000 00:00:00 GMT");
-// $oTemplate->header("Last-Modified: " . gmdate("D, d M Y H:i:s") . "GMT");
-// $oTemplate->header("Cache-Control: no-cache, must-revalidate");
-// $oTemplate->header("Pragma: no-cache");
+$oTemplate->header('Pragma: no-cache');
+$oTemplate->header('Cache-Control: no-cache, no-store, must-revalidate, max-age=0');
+$oTemplate->header('Expires: Sat, 1 Jan 2000 00:00:00 GMT');
+//TODO: is this needed? $oTemplate->header('Last-Modified: ' . gmdate('D, d M Y H:i:s') . 'GMT');