Reject URLs with invalid port

Previously they were parsed incorrectly and characters between
the port number and the first slash were silently dropped as
shown by curl test 187.

Rebuild docs for +limit-cookie-lifetime{} and forward-socks5t

Ditch $Log$ which broke the dok target due to Docbook being awesome

Document forward-socks5t

Accept and highlight: Optimistically sending 318 bytes of client headers intended for privoxy.org

Add support for Tor's optimistic-data SOCKS extension

.. which can reduce the latency for requests on newly
created connections.

Currently only the headers are sent optimistically and
only if the client request has already been read completely
which rules out requests with large bodies.

Add support for chunk-encoded client request bodies

It's simplistic and requires the whole request body to be buffered
before the request is forwarded. If the buffer limit is reached,
the request is rejected.

Previously chunk-encoded request bodies weren't guaranteed
to be forwarded correctly, so this can also be considered a
bug fix although chunk-encoded request bodies aren't commonly
used in the real world.

Make any_loaded_file_changed() more reliable

Using the timestamps from csp->config->config_file_list
doesn't always work as they are only updated when the
config file is changed.

If an action or filter file is reloaded the timestamps
aren't updated so any_loaded_file_changed() continued
to report file changes.

Use strdup_or_die() instead of strdup()

The strings are known to be short.

Remove remnant of FEATURE_COOKIE_JAR

Add #108: Allow to use a somewhat random string intead of PRIVOXY-FORCE

According to #2963658 this was already on "the TODO list",
but apparently not on this one.

Add #107: Support more pcrs variables

Add Rick Sykes

He suggested something like the +limit-cookie-lifetime{} action in #1049575.

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