Fix sorting

Set the SERVER_CONTENT_LENGTH_SET flag after creating a new Content-Length header

This allows to keep the client connection alive after
filtering a chunk-encoded response.

Let server_set_cookie() stay silent when the cookie isn't modified

It previously would claim to have created a session cookie
after letting a session cookie pass unmodified.

Allow HTTP/1.0 clients to signal interest in keep-alive through the Proxy-Connection header

While such client are rare in the real world, it doesn't
hurt and couple of curl tests rely on it.

Stop considering client sockets tainted after receving a request with body

It hasn't been necessary for a while now and unnecessarily
causes test failures when using curl's test suite.

Fix a harmless logic bug in mark_server_socket_tainted()

The log message was never printed while the intention was
to only suppress it if the socket was tainted already.

Copy&paste man struck again.

Allow serve() to keep client connections open if the response content length is zero

Allow closing curly braces as part of action values as long as they are escaped

Move string_move() into miscutil.c and make it extern

It's not really a parser.

Reduce indirection in parse_header_time()

Increase MAX_AF_FILES to 30

An unused file slot is pretty inexpensive.

Consistently put the tests between the tested actions and the TAG or URL pattern

It makes it more obvious which test belongs to which action section
and we already use this style in default.action.master.

Remove a TODO comment about expiring cookies before the end of the session

Disable limit-cookie-lifetime{} in two tests

Add limit-cookie-lifetime{} support for the CGI editor

Bump copyright

Document limit-cookie-lifetime{}

Add limit-cookie-lifetime{} action

It can be used to let cookies expire before the end of the session.

limit-cookie-lifetime{0} creates session cookies and can thus
replace the session-cookies-only action in the future.

Only call time() in server_set_cookie() if the time is actually needed

Fix a function description

When parsing action files, use strdup_or_die() instead of strdup()

It simplifies the code, the input is trusted and errors
are already fatal anyway.

Fix grammar

Style fix for the previous commit

Rebuild after documenting --config-test

Document --config-test option

Regenerate man page

Sort options alphabetically

Document the --config-test option

Add a --config-test option

It lets Privoxy exit after checking whether or not the
configuration seems valid. The limitations noted in
TODO #22 and #23 still apply.

Logging the problem to the logfile if one is configured
and --no-daemon isn't used could be considered non-intuitive
and might also clobber messages emitted by an already running
instance.

Based on a patch by Ramkumar Chinchani.

Let finish_http_response() advertise that the Content-Length header has been set

This allows jcc.c v1.412's serve() to keep the client connection open.

Remove an obsolete master remark

Disable fast-redirects for .googleapis.com/

Remove an obsolete debug message in decompress_iob()

Add a couple of hide-if-modified-since{} tests with different date formats

Block /openx/www/delivery/

Do not keep the client connection open if it has to be closed to mark the end of the response

Clarify that the destination-change detection works for intercepted requests as well

For some values of "clarify".

Fix an assertion in pcrs_compile_replacement(), nul bytes are actually fine

In verify_request_length(), express 'unread client data available' in a more reasonable way

Fix an assertion in verify_request_length()

Update verify_request_length()'s description

Accept and highlight: Tainting client socket 7 due to unread data.

Unblock websupport.wdc.com/

Reported by Adam Piggot in #3577851.

Make a log message in serve() more consistent with the rest

Unbreak compilation without FEATURE_CONNECTION_KEEP_ALIVE

Make a couple of csp members visible when compiling without FEATURE_CONNECTION_KEEP_ALIVE

They are unconditionally used in a couple of log messages
and ifdef protecting them would be a mess.

Hide get_expected_content_length() when compiling without FEATURE_CONNECTION_KEEP_ALIVE

It's not called and references functions that aren't available
in that scenario.

Make socket_is_still_alive() visible when compiling without FEATURE_CONNECTION_KEEP_ALIVE

Nowadays it's used for general timeout detection as well.

Make mark_connection_closed() visible when compiling without FEATURE_CONNECTION_KEEP_ALIVE

It's used all over the place, ifdef-protecting each
occurrence would be a mess, and calling it doesn't hurt.

In serve(), stop considering connection sharing a reason to keep the client socket open

This should have been committed with 1.396.

Add a digit missed in the previous commit

Accept and highlight: Drained 180 bytes before closing socket 6

Accept and highlight the extended log message for client sockets

Bump generated Firefox version to 16

Remember connections in case of destination mismatches in chat()

... when configured to share connection.

Rephrase log messages in serve()

Rebuild documentation for the new config directives in 3.0.20

Document the tolerate-pipelining directive

In the keep-alive section, acknowledge that there are other reasons why clients may hang

Accept and highlight: Client request 4 arrived in time on socket 7.

Accept and highlight: Closing client socket 8. Keep-alive: 1. Socket alive: 0. Data available: 0. Configuration file change detected: 0. Requests received: 11.

Keep track of how many requests have been received on a client socket

In verify_request_length(), use clear_iob() when nuking pipelined data

Rephrase #12 now that pipelining on incoming connections is supported

Replace the IOB_RESET() macro with the new function clear_iob()

Keep track of how many request are sent through a server connection

In cgi_show_request() only restore the client request up to client_iob->cur

Everything beyond that hasn't been touched by get_header()
and thus doesn't need restoring.

Deal with unsupported methods sent to the CGI pages by discarding any data following the headers

Add optional client-side pipelining support

If the new config directive tolerate-pipelining is enabled,
Privoxy will keep pipelined client requests around to deal
with them once the current request has been served.

At the moment this is only done if the current request
is known to have no body.

Let the IOB_PEEK macro take the iob as argument

Let serve() wait for keep-alive-timeout seconds for new requests

The server connection timeout is no longer relevant.

When closing the client socket in serve(), also log if a config file change has been detected

chat() with the client as long as the connection state allows it

Due to various changes in the past, the state of the
server connection is no longer relevant.

Remove a pointless fflush() call in chat()

Flushing all streams pretty much all the time for no obvious
reason is ridiculous.

Set the socket option SO_LINGER for the client socket in accept_connection()

This should further reduce the chances of the client getting
an incomplete response, even though Privoxy wrote the whole
one to the socket.

In chat(), check for failed delivery of request headers and body separately

Use a dedicated iob for the data read from the client

This is necessary to support client-side pipelining.

Use IOB_RESET() in prepare_csp_for_next_request()

Let IOB_RESET() take the iob to reset as argument

This allows it to be used for iobs other than csp->iob.

Remove pre-3.0 compatibility goo which has been obolete for more than 10 years now

Do not promise a logrotate script for Red-Hat-based systems

It has been garbage collected in 2009.

Clarify that the log detail of the Windows GUI log depends on the debug directive

Parse the Windows-specific toggles with parse_toggle_state() as well

In the redirect{} section, refer pcrs newbies to the 'filter file' section

Use the correct type for reusable_connection.forwarder_type

Accept and highlight: Continuing buffering server headers from socket 5. Bytes most recently read: 498.

When logging the buffering of server headers in chat(), include their source

Rename ACTION_NO_COOKIE_KEEP to ACTION_SESSION_COOKIES_ONLY to match the current name of the action

Rename ACTION_NO_COOKIE_READ to ACTION_CRUNCH_OUTGOING_COOKIES to match the current name of the action

Rename ACTION_NO_COOKIE_SET to ACTION_CRUNCH_INCOMING_COOKIES to match the current name of the action

Fix grammar in an error message in remember_connection()

Use strdup_or_die() in remember_connection()

Fix latency calculation for shared connections

It has been broken since I committed it with 1.57 in 2009.

The calculated latency for most connections would be 0 due to
remember_connection() always putting the timestamps in the
first slot. Oops.

Use parse_toggle_state() to get the global_toggle_state

Downgrade two error messages in drain_and_close_socket() to LOG_LEVEL_CONNECT

The errors are expected if the connection is closed on the remote
end after drain_and_close_socket() confirmed that the socket is
still alive, but before it tries to drain it.

Fix a comment typo.

Use the shiny-new drain_and_close_socket() when closing client sockets in serve()

Hopefully this fixes the page truncation issue reported
with Opera in #3464439.

Add drain_and_close_socket()

Apparently on some platforms immediately closing a client socket
with both unread and written-but-not-yet-transmitted data may result
in the connection being teared down prematurely, in which case
the client ends up with an incomplete response.

To prevent this, drain_and_close_socket() shuts down our side of
the connection (on platforms with shutdown()), drains the unread
data on the socket and finally calls close_socket().

I suspect that this was the main cause of the page truncation
issues reported with Opera in #3464439.

Don't bother checking for strstr() as we have no fallback anyway

Spread a AC_CHECK_FUNCS() line across one line per checked function to make future diffs more readable