Item9045: Should we really allow the web server to fill in a 200 Status?
Priority: Normal
Current State: Closed
Released In: 1.1.0
Target Release: minor
Applies To: Engine
Component:
Branches:
by not setting the status explicitly (and defaulting to an error condition), we run the 'risk' of sending a 200 code even when there is an unexpected error.
I propose to change this
only in
DEBUG
mode -
500
perhaps.
--
SvenDowideit - 21 May 2010
sub new {
my $proto = shift;
my $class = ref($proto) || $proto;
my $this = {
status => undef, #web server fills in 200 if status is not set
headers => {},
body => undef,
charset => 'ISO-8859-1',
cookies => [],
startedPrinting => 0,
};
#SMELL: default to 500 in debug modeso we can shake out the missing Status's
$this->{status} = 500 if DEBUG;
return bless $this, $class;
}
--
SvenDowideit - 21 May 2010
Agreed.
--
CrawfordCurrie - 22 May 2010