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

Bump generated Firefox version to 14.0.1

Block g.adspeed.net/

Accept and highlight: Closing client socket 5. Keep-alive: 0, Socket alive: 1. Data available: 0

Remove #101: Forward known client headers in a predefined order

The "predefined order" part is implemented and the randomization
no longer strikes me as useful.

Bump copyright

Mention the shiny new LOG_LEVEL_ACTIONS

Accept the shiny new LOG_LEVEL_ACTIONS

Add LOG_LEVEL_ACTIONS to log the applying actions

Unlike in the "Final results", the effect of tags is taken into account.

Add #106: actionlist.h should be embedded in a way that causes less text bloat

Let get_percentage() show 0 of x as 0% even if x is 0 as well

Accept and ignore: Enlisting (sorted|left-over) header ...

Implement the client-header-order directive

It can be used to forward client headers in a different
order than they arrived.

In serve(), log the client socket and its condition before closing it

This makes it more obvious that the socket actually gets closed
and should help when diagnosing problems like #3464439.

Add debug_level_is_enabled()

Block rotation.linuxnewmedia.com/

Block reklamy.sfd.pl/

Reported by kacperdominik in #3399948.

Revert 1.381 which wasn't a style change after all

Add #105: Add support for socks authentication.

Add test URLs for .freebsd.org and .watson.org

Unblock .urbandictionary.com/popular

Block requests to .adnxs.com/

Block farm.plista.com/widgetdata.php

Update copyright range

In ssplit(), assert() that the last substring already is nul-terminated

Use an enum to replace a couple of magic numbers in ssplit()

Remove two pointless casts in ssplit()

Relocate a comment in ssplit()

Bump copyright year

Start using malloc_or_die()

Add malloc_or_die() which will allow to simplify code paths were malloc() failures don't need to be handled gracefully

Merge two if conditions in main()

Bump copyright to 2012

Properly highlight the new log messages from get_request_line().

Replace strdup() calls with strdup_or_die() calls were it's safe and simplifies the code

Do not confuse a client hanging up with a connection time out

If a client closes its side of the connection without sending
a request line, do not send the CLIENT_CONNECTION_TIMEOUT_RESPONSE,
but report the condition properly.

This was reproducible with (for example):
false | nc privoxy-jail 8118

Fix a clang warning about integers and enums being mixed

Update Junkbuster-related FAQ sections

Remove the dead links and use the past tense as
the fork happened years ago.

junkbusters.com has been shut down, update the history section accordingly

Remove two links to junkbusters.com which is no longer maintained

We already link to Wikipedia's Cookie description elsewhere and
Privoxy's origins are sufficiently documented in history.sgml,
so the links don't need replacements here.

Reported by Angelina Matson.

Prevent the CVS substitution in the generated example comment as well

Remove pointless 'debugging support' in ssplit()

gdb offers the same information without recompiling.

Shorten ssplit()'s prototype by removing the last two arguments

We always want to skip empty fields and ignore leading
delimiters, so having parameters for this only complicates
the API.

Let load_one_actions_file() use ssplit() like every other function

It doesn't matter whether or not ignore_leading is enabled.

Remove an unreasonable parameter limit in parse_cgi_parameters()

The new parameter limit depends on the memory available and
is currently unlikely to be reachable, due to other limits in
both Privoxy and common clients.

So the limit is actually still there, but at least it's no
longer unreasonable, is properly handled, and should be high
enough for anybody.

Reported by Andrew on ijbswa-users@.