httpd-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "William A. Rowe, Jr." <>
Subject [patch 1.3] Allow keepalives on bogus requests?
Date Wed, 11 May 2005 06:46:30 GMT
The subject is misleading; we do allow keepalives on any bogus
requests, today.  Undoubtedly this is already used in the wild.

Attached is a patch that would drop keepalive for the garbage
requests that never even hit mod_forensics, or any other module,
due to bogus request lines or unparseable header fields.

The argument against, more cycles are wasted in setup and
teardown than simply filtering bogus requests.  Counter arguments,
so does the client; and the bogus connection is dumped and a fresh
one created.  This actually solved many issues with mod_ssl on
Win32, although it was entirely the wrong solution to the issue
I discovered.

FYI yes in spite of ap_set_keepalive() they still are active.  It's
as simple as the fact that the previous request on the connection
is successful, therefore keepalives are alive and well.

Anyways, comments on the patch?

View raw message