Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: API: C
Submitted By: Stephen Deasey (sdeasey)
Assigned to: Nobody/Anonymous (nobody)
Summary: Invalid response status logged for custom redirects
If notfound.adp exists (which it should) then the
client is served with a 200 response code, which also
end up in the access log. Same for all redirects.
This patch reverts the default status code to 200 in
driver.c when a fresh Conn is set up. It explicitly
sets responseCode in ReturnRedirect. And it explicitly
does not set a response code of 200 when constructing
headers. 200 should be inherited from a fresh Conn,
and we don't want to disturb anything the custom
redirect handler just set.