From 29d387e7f0d5f060aca5dcb549fdca7037f98ff2 Mon Sep 17 00:00:00 2001 From: tokul Date: Sat, 22 Jul 2006 17:24:33 +0000 Subject: [PATCH] Location: header automatically adds 302 Status header. If extra Status header is used, PHP bug (#36705) causes errors in fastcgi setups. RFC 2616 recommends use of 302 headers for agents that don't support HTTP 1.1. Status header was used when code was introduced in SM 1.1.1. There is no explanation why 303 status is needed instead of standard 302. git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@11424 7612ce4b-ef26-0410-bec9-ea0150e637f0 --- ChangeLog | 1 + src/signout.php | 3 ++- 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/ChangeLog b/ChangeLog index 9ac593a5..c5a5eb63 100644 --- a/ChangeLog +++ b/ChangeLog @@ -107,6 +107,7 @@ Version 1.5.2 - CVS - Removed conf.pl dependency on Perl IO::Socket module. Automatic detection of supported authentication mechanisms is disabled, if IO::Socket is not available. + - Removed HTTP Status header from signout page (#1424748). Version 1.5.1 (branched on 2006-02-12) -------------------------------------- diff --git a/src/signout.php b/src/signout.php index 7b54784b..181434fd 100644 --- a/src/signout.php +++ b/src/signout.php @@ -40,7 +40,8 @@ do_hook('logout'); sqsession_destroy(); if ($signout_page) { - header('Status: 303 See Other'); + // Status 303 header is disabled. PHP fastcgi bug. See 1.91 changelog. + //header('Status: 303 See Other'); header("Location: $signout_page"); exit; /* we send no content if we're redirecting. */ } -- 2.25.1