From: Bjoern Jacke <bjoern@j3e.de>
Date: Tue, 22 Mar 2011 23:55:34 +0000 (-0700)
Subject: Bug 643925: When shutdownhtml sends an HTTP 503, also include a Retry-After
X-Git-Tag: bugzilla-4.1.2~43
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=a011cac4d115c49f5cc2742027a0009aa3bc4722;p=thirdparty%2Fbugzilla.git

Bug 643925: When shutdownhtml sends an HTTP 503, also include a Retry-After
header.
r=mkanat, a=mkanat
---

diff --git a/Bugzilla.pm b/Bugzilla.pm
index 2f657e3ded..65ddcc2bce 100644
--- a/Bugzilla.pm
+++ b/Bugzilla.pm
@@ -76,6 +76,10 @@ use constant SHUTDOWNHTML_EXIT_SILENTLY => qw(
     whine.pl
 );
 
+# shutdownhtml pages are sent as an HTTP 503. After how many seconds
+# should search engines attempt to index the page again?
+use constant SHUTDOWNHTML_RETRY_AFTER => 3600;
+
 #####################################################################
 # Global Code
 #####################################################################
@@ -170,9 +174,12 @@ sub init_page {
         else {
             $extension = 'txt';
         }
-        # Set the HTTP status to 503 when Bugzilla is down to avoid pages
-        # from being indexed.
-        print Bugzilla->cgi->header(-status=>503) if i_am_cgi();
+        if (i_am_cgi()) {
+            # Set the HTTP status to 503 when Bugzilla is down to avoid pages
+            # being indexed by search engines.
+            print Bugzilla->cgi->header(-status => 503, 
+                -retry_after => SHUTDOWNHTML_RETRY_AFTER);
+        }
         my $t_output;
         $template->process("global/message.$extension.tmpl", $vars, \$t_output)
             || ThrowTemplateError($template->error);