Reduce server_last_modified()'s stack size

Shorten get_http_time() by using strftime()

In chat(), properly report missing server responses as such instead of calling them empty

Let load_one_actions_file() properly complain about a missing '{' at the beginning of the file

Simply stating that a line is invalid isn't particularly helpful.

In forwarded_connect(), fix a log message nobody should ever see

Rephrase #12 as pipelined client requests aren't completely unsupported

Add a link to the latest version

Note that #97 is mostly done

Fix a log message in socks5_connect(), a failed write operation was logged as failed read operation

Ditch trailing whitespace

Fix the dok-tidy target to actually suppress backup file creation

Constify the known_http_methods pointers in unknown_method()

Constify the time_formats pointers in parse_header_time()

Constify the formerly_valid_actions pointers in action_used_to_be_valid()

Fix spelling in a comment

In html_code_map[], use a numeric character reference instead of ' which wasn't standardized before XHTML 1.0

Oops.

In parse_header_time(), zero out gmt before each strptime() just to be sure

Let parse_header_time() look for two-digit years first

Should work around GNU libc's strptime() reporting negative
years when parsing a two-digit year with %Y.

Reported by Väinö in #3403560

In parse_header_time(), add a sanity check for GNU libc's strptime() which apparently occasionally returns negative years.

Reported by Väinö in #3403560

Dijkstraize parse_header_time(): two or more, use a for

Adjust a comment in parse_header_time(). Zeroing out gmt is documented to be required for GNU libc

Ditch trailing whitespace

Rename the tidy target to clean-editor-files so it can't be confused with the dok-tidy target

Remove the obsolete browser support matrix. Minor cosmetics.

The first result of the shiny-new dok-tidy target.

Bump copyright.

Add a dok-tidy target to clean up the messy HTML generated by the other dok targets

Not yet connected to any other target,
but probably should be after some testing.

Ditch a duplicated dot in a comment

Change the HTTP version in the snipplets to 1.1

The snipplets are compliant, so there's no reason not to.

Rebuild with updated copyright

Rebuild with updated copyright and minor changes

Rebuild after ditching GNUisms in GNUMakefile.in

Deduplicate a line in the dok-release target

Add #97: Add support for Tor's 'optimistic data' socks extension.

Once more, with feeling: GNUisms be gone

This time the dok target has actually been tested on
a GNU/Linux system using GNU make.

When sending out the "no-server-data" response, do not log two LOG_LEVEL_CLF messages

send_crunch_response() already takes care of emitting the
message, so there's no need for chat() to emit another one.

Noticed while looking into #3394462 reported by Andy.

Don't use unspecified MSN sites as examples for User-Agent-based descrimination

Without knowing the URLs, nobody can easily verify it and it could
be mistaken as FUD. I also assume that it's no longer an issue anyway.

Bump some more documentation copyright ranges.

Also highlight the compression level, now that it is logged, too

Rebuild README

Index updates.

Update the entities.

This commit is chronological out of order.

Rebuild user-manual, developer-manual and faq.

Looks like the code status UNRELEASED isn't properly dealt with everywhere,
but as this build is not going to get released, I think we can live with
it for now.

Also ampersands in URLs weren't properly escaped, but hopefully
none of the broken links slipped into this commit. This needs to
be addressed before the next release anyway.

Formattting fixes missed in the previous commit and forgotten to squash

Rebuild config with the recently added compression directives

Bumpy copyright year

Revert the previous commit for now.

Turns out that it broke a the dok targets. Ooops.

In compress_buffer(), let zlib figure out the proper length for the new buffer

Simply adding 30 bytes is somewhat "suboptimal".

Fix a compiler warning on GNU/Linux i386

When loading action sections, verify that the referenced filters exist

Currently missing filters only result in an error message,
but eventually the severity should be upgraded to fatal.

Add #96: Enabled filters should be easier to look up

Get the language for #42 right.

While at it, remove the bible reference as it arrived a while ago.

Sync with upstream 2011-06-03/5669ec08b530b9bbed5848ec414f8c029e41ab02

Add tests for missing socks4 and socks4a forwarders

The code paths are different than for socks5 and the recent
memory leak in socks5_connect() was only detected due to the
already existing test.

Currently no such leaks exist in socks4_connect() but I may
add some in the future and would prefer to know about it.

Deduplicate the INADDR_NONE definition for Solaris by moving it to jbsockets.h

While at it, add a comment about whether or not the workaround is still needed.

In block_url(), ditch the obsolete workaround for ancient Netscape versions that supposedly couldn't properly deal with status code 403.

White-space fixes for add_loader() and run_loader()

Remove a useless NULL pointer check in load_trustfile()

Remove two useless NULL pointer checks in load_one_re_filterfile().

csp has already been dereferenced earlier and there's
no reason why it should be NULL anyway.

Set socket_error to errno if connecting fails in rfc2553_connect_to()

Previously rejected direct connections could be incorrectly reported as DNS issues.

Slightly restructure the error handling in socks5_connect().

No (intended) change of behaviour.

Add the filter-file loaders before the action-file loaders

This way all filters are loaded by the time the action files
are parsed and filter{} actions specifying non-existent filters
can be detected and rejected (with code not yet committed).

Disable banners-by-size filters for '.thinkgeek.com/'

The filter only seems to catch pictures of the inventory.

Block requests for 'go.idmnet.bbelements.com/please/showit/'

Reported by kacperdominik in #3372959.

Add #95: Support a non-standard client header in CONNECT requests that contains the URL of the requested ressource

Remove #71 as it has been implemented a few days ago

Minor style fix for load_config()

Make sure --privoxy-address works with IPv6 addresses containing brackets, too

Minor style fixes for bind_port_helper()

Reduce code duplication in bind_port_helper()

Minor style fix for unload_configfile()

Add a comment about possibly questionable behaviour in accept_connection()

Style and comment fixes for accept_connection()

Restructure bind_ports_helper() a bit

Setting config->need_bind isn't necessary.

Do not claim to listen on a socket until we actually do.

Patch submitted by Petr Písař in #3354485

Allow to bind to multiple separate addresses.

Patch set submitted by Petr Písař in #3354485.

Fix a memory leak if the socks5 server is unreachable.

I introduced the leak after the last release so this
fix is irrelevant as far as the ChangeLog is concerned.

Minor style fix for listen_loop().

Remove #13 which is already implemented.

Also log the client socket when dropping a connection.

Document compression-level

Document enable-compression

Add a 'to' that was missing in the FEATURE_COMPRESSION description

Add an enable-compression directive and disable compression by default

parse_toggle_state() only returns 0 or 1, so check for 1 directly.

Also let parse_toggle_state() reject values like '1[insert garbage here]'

Add a dedicated function to parse the values of toggles

Reduces duplicated code in load_config() and provides
better error handling. Invalid or missing toggle values
are now a fatal error instead of being silently ignored.

If compressing the buffer failed, include buffer size and compression level in the error message

In compress_buffer(), add some room for the zlib overhead

If the entropy is high enough, the data may not be compressable,
or at least not compressable enough to make room for the header.

If the compression level is 0, the "compressed" is guaranteed
to take more room than the "uncompressed" data.

Add the config directive compression-level

There isn't a single best compression level for
all situations, so hard-coding one makes no sense.

#94: Add a config directive to let Privoxy prefer either IPv4 (or IPv6) addresses

#93: Add a config directive to let Privoxy explicitly request either IPv4 (or IPv6) addresses

In bind_port(), stop hinting AI_ADDRCONFIG to getaddrinfo()

It's not strictly needed there and causes GNU libc to fail to
return loopback addresses if no non-loopback IP address is
configured on the system. Apparently this is the result of a
somewhat strict but valid interpretation of RFC 3493 and
considered a feature. Sad but true.

This is mainly an issue if the system is using DHCP and Privoxy
is started before the network is completely configured.

Reported by Raphael Marichez in #3349356.
Additional insight from Petr Pisar.

Move LOWER_LENGTH_LIMIT_FOR_COMPRESSION into cgi.[ch] where it belongs

While at it, turn it into a proper variable so it can be changed from the debugger.

Fix spelling of LOWER_LENGTH_LIMIT_FOR_COMPRESSION

Fix the FEATURE_ZLIB description. It's mainly used for decompression.

Unbreak execute_dumb_fetch_test().

Apparently 1.73 wasn't regression-tested. How ironic.

Remove a stray exit(0)

Cosmetics for generate_creation_time()

Remove a couple of unnecessary variables in generate_creation_time()