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

Remove two checks that were already commented out

Spread the AC_CHECK_HEADERS() check across one line per header to make future diffs more readable

Update server_proxy_connection_adder()'s description to reflect reality

Add DragonFly to the list of BSDs Privoxy is known to run on

Add Haiku to the list of operating systems Privoxy is known to run on

Move a variable declaration in parse_client_header_order() to the beginning of its code block

It's how we usually do it and is apparently also required
when compiling with gcc 2.95.

Initial patch submitted by Simon South in #3564815.

Move a couple of variable declarations in get_last_url() to the beginning of their code block

It's how we usually do it and is apparently also required
when compiling with gcc 2.95.

Initial patch submitted by Simon South in #3564815.

Additional Haiku-specific improvements

- Disable checks intended for multi-user systems
  (as Haiku is presently single-user).
- Group Haiku-specific settings in their own section,
  following the pattern for Solaris, OS/2 and AmigaOS
- Add additional library-related settings to remove
  the need for providing configure with custom LDFLAGS.

Submitted by Simon South in #3574538.

Spell 'refresh-tags' correctly

Reported by Don in #3571927.

Documentation for 3.0.19, regenerated with the modified tidy changes

Bring back the body attributes now that tidy no longer messes them up

Apparently they aren't entirely useless as they prevent
the Docbook mess from inserting other attributes.

Leave it up to the user whether or not smart tags are inserted

Ditch pointless keywords. Search engines no longer need them.

Remove tidy's clean flag as it changes the scope of attributes

Link-specific colors end up being applied to all text.
Reported by Adam Piggott in #3569551.

Ditch obsolete body attributes

The style sheets are sufficient and the body attributes
didn't properly make it through the docbook/tidy chain,
causing link-specific attributes to be applied to all text.

Reported by Adam Piggott in #3569551.

In finish_http_response(), set 'Connection: close' after preventing pipelining

I missed the CGI parts when committing parsers.c v1.250.

Revert an incorrect part of 1.250 that could result in two Connection headers being sent

On Haiku, do not pass -lpthread to the compiler

Haiku's pthreads implementation is contained in its system library,
libroot, so no additional library needs to be searched.

Patch submitted by Simon South in #3564815.

Accept and highlight: Accepted connection from 10.0.0.1 on socket 5

Relocate the connection-accepted message from listen_loop() to serve()

This way it's printed by the thread that is actually serving the
connection which is nice when grepping for thread ids in log files.

While at it, use an upper-case first character which is more
consistent with the other log messages.

Stop suppressing messages about accepted client connections and highlight them properly

Fix a comment in client_connection()

After preventing the client from pipeling, don't signal keep-alive intentions

When looking at the response headers alone, it previously wasn't
obvious from the client's perspective that no additional responses
should be expected.

This might improve compatibility with Opera (#3464439).

Figure out the request length before running sed()

We may want to adjust the Connection header based on the information.

Add get_expected_content_length()

Let get_content_length() use a header value as input instead of the whole header

This makes it easier to use it in combination with get_header_value().

Note that CSP_FLAG_SERVER_SOCKET_TAINTED is poorly named