<!--
- File : $Source: /cvsroot/ijbswa/current/doc/source/changelog.sgml,v $
+ File : doc/source/changelog.sgml
Purpose : Entity included in other project documents.
- $Id: changelog.sgml,v 2.15 2016/01/21 15:57:16 fabiankeil Exp $
-
- Copyright (C) 2013 Privoxy Developers http://www.privoxy.org/
+ Copyright (C) 2013-2018 Privoxy Developers https://www.privoxy.org/
See LICENSE.
======================================================================
user-manual
-->
-<para>
- <application>Privoxy 3.0.24</application> stable contains a couple
- of new features but is mainly a bug-fix release. Two of the fixed
- bugs are security issues and may be used to remotely trigger crashes
- on platforms that carefully check memory accesses (most don't).
-</para>
<!--
The SGML ChangeLog can be generated with: utils/changelog2doc.pl ChangeLog
-->
-<para>
<para>
+ <application>Privoxy 3.0.27</application> stable scales better
+ in multi-user environments and brings a couple of tuning directives.
+</para>
+
<itemizedlist>
<listitem>
<para>
- Security fixes (denial of service):
+ General improvements:
<itemizedlist>
<listitem>
<para>
- Prevent invalid reads in case of corrupt chunk-encoded content.
- CVE-2016-1982. Bug discovered with afl-fuzz and AddressSanitizer.
+ Add a receive-buffer-size directive which can be used to
+ set the size of the previously statically allocated buffer
+ in handle_established_connection().
+ Increasing the buffer size increases Privoxy's memory usage but
+ can lower the number of context switches and thereby reduce the
+ CPU usage and potentially increase the throughput.
+ This is mostly relevant for fast network connections and
+ large downloads that don't require filtering.
+ Sponsored by: Robert Klemme
</para>
</listitem>
<listitem>
<para>
- Remove empty Host headers in client requests.
- Previously they would result in invalid reads. CVE-2016-1983.
- Bug discovered with afl-fuzz and AddressSanitizer.
+ Add a listen-backlog directive which specifies the backlog
+ value passed to listen().
+ Sponsored by: Robert Klemme
</para>
- </listitem>
- </itemizedlist>
- </para>
- </listitem>
- <listitem>
- <para>
- Bug fixes:
- <itemizedlist>
+ </listitem>
<listitem>
<para>
- When using socks5t, send the request body optimistically as well.
- Previously the request body wasn't guaranteed to be sent at all
- and the error message incorrectly blamed the server.
- Fixes #1686 reported by Peter Müller and G4JC.
+ Add an enable-accept-filter directive which allows to
+ toggle accept filter support at run time when compiled
+ with FEATURE_ACCEPT_FILTER support.
+ It makes testing more convenient and now that it's
+ optional we can emit an error message if enabling
+ the accept filter fails.
+ Sponsored by: Robert Klemme
</para>
</listitem>
<listitem>
<para>
- Fixed buffer scaling in execute_external_filter() that could lead
- to crashes. Submitted by Yang Xia in #892.
+ Add a delay-response{} action.
+ This is useful to tar pit JavaScript requests that
+ are endlessly retried in case of blocks. It can also
+ be used to simulate a slow Internet connection.
+ Sponsored by: Robert Klemme
</para>
</listitem>
<listitem>
<para>
- Fixed crashes when executing external filters on platforms like
- Mac OS X. Reported by Jonathan McKenzie on ijbswa-users@.
+ Add a 'trusted-cgi-referrer' directive.
+ It allows to configure another page or site that can be used
+ to reach sensitive CGI resources.
+ Sponsored by: Robert Klemme
</para>
</listitem>
<listitem>
<para>
- Properly parse ACL directives with ports when compiled with HAVE_RFC2553.
- Previously the port wasn't removed from the host and in case of
- 'permit-access 127.0.0.1 example.org:80' Privoxy would try (and fail)
- to resolve "example.org:80" instead of example.org.
- Reported by Pak Chan on ijbswa-users@.
+ Add a --fuzz mode which exposes Privoxy internals to input
+ from files or stdout.
+ Mainly tested with American Fuzzy Lop. For details see:
+ https://www.fabiankeil.de/talks/fuzzing-on-freebsd/
+ This work was partially funded with donations and done
+ as part of the Privoxy month in 2015.
</para>
</listitem>
<listitem>
<para>
- Check requests more carefully before serving them forcefully
- when blocks aren't enforced. Privoxy always adds the force token
- at the beginning of the path, but would previously accept it anywhere
- in the request line. This could result in requests being served that
- should be blocked. For example in case of pages that were loaded with
- force and contained JavaScript to create additionally requests that
- embed the origin URL (thus inheriting the force prefix).
- The bug is not considered a security issue and the fix does not make
- it harder for remote sites to intentionally circumvent blocks if
- Privoxy isn't configured to enforce them.
- Fixes #1695 reported by Korda.
+ Consistently use the U(ngreedy) flag in the 'img-reorder' filter.
</para>
</listitem>
<listitem>
<para>
- Normalize the request line in intercepted requests to make rewriting
- the destination more convenient. Previously rewrites for intercepted
- requests were expected to fail unless $hostport was being used, but
- they failed "the wrong way" and would result in an out-of-memory
- message (vanilla host patterns) or a crash (extended host patterns).
- Reported by "Guybrush Threepwood" in #1694.
+ listen_loop(): Reuse a single thread attribute object
+ The object doesn't change and creating a new one for
+ every thread is a waste of (CPU) time.
+ Sponsored by: Robert Klemme
</para>
</listitem>
<listitem>
<para>
- Enable socket lingering for the correct socket.
- Previously it was repeatedly enabled for the listen socket
- instead of for the accepted socket. The bug was found by
- code inspection and did not cause any (reported) issues.
+ Free csp resources in the thread that belongs to the csp instead
+ of the main thread which has enough on its plate already.
+ Sponsored by: Robert Klemme
</para>
</listitem>
<listitem>
<para>
- Detect and reject parameters for parameter-less actions.
- Previously they were silently ignored.
+ Improve 'socket timeout reached' message.
+ Log the timeout that was triggered and downgrade the
+ log level to LOG_LEVEL_CONNECT to reduce the log noise
+ with common debug settings.
+ The timeout isn't necessary the result of an error and
+ usually merely indicates that Privoxy's socket timeout
+ is lower than the relevant timeouts used by client and
+ server.
+ Sponsored by: Robert Klemme
+ </para>
+ </listitem>
+ <listitem>
+ <para>
+ Explicitly taint the server socket in case of CONNECT requests.
+ This doesn't fix any known problems, but makes
+ some log messages less confusing.
+ </para>
+ </listitem>
+ <listitem>
+ <para>
+ Let write_pid_file() terminate if the pid file can't be opened.
+ Logging the issue at info level is unlikely to help.
+ </para>
+ </listitem>
+ <listitem>
+ <para>
+ log_error(): Reduce the mutex-protected area by not using a
+ heap-allocated buffer that is shared between all threads.
+ This increases performance and reduces the latency with
+ verbose debug settings and multiple concurrent connections.
+ Sponsored by: Robert Klemme
+ </para>
+ </listitem>
+ <listitem>
+ <para>
+ Let zalloc() use calloc() if it's available.
+ In some situations using calloc() can be faster than
+ malloc() + memset() and it should never be slower.
+ In the real world the impact of this change is not
+ expected to be noticeable.
+ Sponsored by: Robert Klemme
+ </para>
+ </listitem>
+ <listitem>
+ <para>
+ Never use select() when poll() is available.
+ On most platforms select() is limited by FD_SETSIZE while
+ poll() is not. This was a scaling issue for multi-user setups.
+ Using poll() has no downside other than the usual risk
+ that code modifications may introduce new bugs that have
+ yet to be found and fixed.
+ At least in theory this commit could also reduce the latency
+ when there are lots of connections and select() would use
+ "bit fields in arrays of integers" to store file descriptors.
+ Another side effect is that Privoxy no longer has to stop
+ monitoring the client sockets when pipelined requests are
+ waiting but can't be read yet.
+ This code keeps the select()-based code behind ifdefs for
+ now but hopefully it can be removed soonish to make the
+ code more readable.
+ Sponsored by: Robert Klemme
+ </para>
+ </listitem>
+ <listitem>
+ <para>
+ Add a 'reproducible-tarball-dist' target.
+ It's currently separate from the "tarball-dist" target
+ because it requires a tar implementation with mtree spec
+ support.
+ It's far from being perfect and does not enforce a
+ reproducible mode, but it's better than nothing.
+ </para>
+ </listitem>
+ <listitem>
+ <para>
+ Use arc4random() if it's available.
+ While Privoxy doesn't need high quality pseudo-random numbers
+ there's no reason not to use them when we can and this silences
+ a warning emitted by code checkers that can't tell whether or not
+ the quality matters.
+ </para>
+ </listitem>
+ <listitem>
+ <para>
+ Show the FEATURE_EXTERNAL_FILTERS status on the status page.
+ Better late than never. Previously a couple of tests weren't
+ executed as Privoxy-Regression-Test couldn't detect that the
+ FEATURE_EXTERNAL_FILTERS dependency was satisfied.
+ </para>
+ </listitem>
+ <listitem>
+ <para>
+ Ditch FEATURE_IMAGE_DETECT_MSIE.
+ It's an obsolete workaround we inherited from Junkbuster
+ and was already disabled by default.
+ Users that feel the urge to work around issues with
+ image requests coming from an Internet Explorer version
+ from more than 15 years ago can still do this using tags.
+ </para>
+ </listitem>
+ <listitem>
+ <para>
+ Consistently use strdup_or_die() instead of strdup() in
+ cases where allocation failures aren't expected.
+ Using strdup_or_die() allows to remove a couple of explicit
+ error checks which slightly reduces the size of the binary.
+ </para>
+ </listitem>
+ <listitem>
+ <para>
+ Insert a refresh tag into the /client-tags CGI page when
+ serving it while a client-specific tag is temporarily enabled.
+ This makes it less likely that the user ends up
+ looking at tag state that is out of date.
+ </para>
+ </listitem>
+ <listitem>
+ <para>
+ Use absolute URLs in the client-tag forms.
+ It's more consistent with the rest of the CGI page
+ URLs and makes it more convenient to copy the forms
+ to external pages.
+ </para>
+ </listitem>
+ <listitem>
+ <para>
+ cgi_error_disabled(): Use status code 403 and an appropriate response line
+ </para>
+ </listitem>
+ <listitem>
+ <para>
+ Use a dedicated CGI handler to deal with tag-toggle requests
+ As a result the /client-tags page is now safe to reach without
+ trusted Referer header which makes bookmarking or linking to
+ it more convenient.
+ Finally, refreshing the /client-tags page to show the
+ current state can no longer unintentionally repeat the
+ previous toggle request.
+ </para>
+ </listitem>
+ <listitem>
+ <para>
+ Don't add a "Connection" header for CONNECT requests.
+ Explicitly sending "Connection: close" is not necessary and
+ apparently it causes problems with some forwarding proxies
+ that will close the connection prematurely.
+ Reported by Marc Thomas.
+ </para>
+ </listitem>
+ <listitem>
+ <para>
+ Fix compiler warnings.
+ </para>
+ </listitem>
+ </itemizedlist>
+ </para>
+ </listitem>
+ <listitem>
+ <para>
+ Bug fixes:
+ <itemizedlist>
+ <listitem>
+ <para>
+ rfc2553_connect_to(): Properly detect and log when poll()
+ reached the time out. Previously this was logged as:
+ Could not connect to [...]: No error: 0.
+ which isn't very helpful.
+ Sponsored by: Robert Klemme
</para>
</listitem>
<listitem>
<para>
- Fixed invalid reads in internal and outdated pcre code.
- Found with afl-fuzz and AddressSanitizer.
+ add_tag_for_client(): Set time_to_live properly.
+ Previously the time_to_live was always set for the first tag.
+ Attempts to temporarily enable a tag would result in enabling
+ it permanently unless no tag was enabled already.
</para>
</listitem>
<listitem>
<para>
- Prevent invalid read when loading invalid action files.
- Found with afl-fuzz and AddressSanitizer.
+ Revert r1.165 which didn't perform as advertised.
+ While the idea was to use "https:// when creating links
+ for the user manual on the website", the actual effect
+ was to use "https://" when Privoxy was supposed to serve
+ the user manual itself.
+ Reported by Yossi Zahn on Privoxy-devel@.
</para>
</listitem>
<listitem>
<para>
- Windows build: Use the correct function to close the event handle.
- It's unclear if this bug had a negative impact on Privoxy's behaviour.
- Reported by Jarry Xu in #891.
+ socks5_connect(): Fail in case of unsupported address types.
+ Previously they would not be detected right away and
+ Privoxy would fail later on with an error message that
+ didn't make it obvious that the problem was socks-related.
+ So far, no such problems have actually been reported.
</para>
</listitem>
<listitem>
<para>
- In case of invalid forward-socks5(t) directives, use the
- correct directive name in the error messages. Previously they
- referred to forward-socks4t failures.
- Reported by Joel Verhagen in #889.
+ socks5_connect(): Properly deal with socks replies that
+ contain IPv6 addresses.
+ Previously parts of the reply were left unread and
+ later on treated as invalid HTTP response data.
+ Fixes #904 reported by Danny Goossen who also provided
+ the initial version of this patch.
</para>
</listitem>
</itemizedlist>
</listitem>
<listitem>
<para>
- General improvements:
+ Action file improvements:
<itemizedlist>
<listitem>
<para>
- Set NO_DELAY flag for the accepting socket. This significantly reduces
- the latency if the operating system is not configured to set the flag
- by default. Reported by Johan Sintorn in #894.
+ Unblock 'msdn.microsoft.com/'.
+ It (presumably) isn't used to serve the kind of ads Privoxy should
+ block by default but happens to serve lots of pages with URLs that
+ are likely to result in false positives.
+ Reported by bugreporter1694 in AF#939.
</para>
</listitem>
<listitem>
<para>
- Allow to build with mingw x86_64. Submitted by Rustam Abdullaev in #135.
+ Disable gif deanimation for requests tagged with CSS-REQUEST.
+ The action will ignore content that isn't considered text
+ anyway and explicitly disabling it makes this more obvious
+ if "action" debugging (debug 65536) is enabled while
+ "gif deanimation" debugging (debug 256) isn't.
</para>
</listitem>
<listitem>
<para>
- Introduce the new forwarding type 'forward-webserver'.
- Currently it is only supported by the forward-override{} action and
- there's no config directive with the same name. The forwarding type
- is similar to 'forward', but the request line only contains the path
- instead of the complete URL.
+ Explicitly disable HTML filters for requests with CSS-REQUEST tag.
+ The filters are unlikely to break CSS files but executing
+ them without (intentionally) getting any hits is a waste of
+ cpu time and makes the log more noisy when running with
+ "debug 64".
</para>
</listitem>
<listitem>
<para>
- The CGI editor no longer treats 'standard.action' special.
- Nowadays the official "standards" are part of default.action
- and there's no obvious reason to disallow editing them through
- the cgi editor anyway (if the user decided that the lack of
- authentication isn't an issue in her environment).
+ Unblock 'adventofcode.com/'.
+ Reported by Clint Adams in Debian bug #848211.
+ Fixes Roland's AF#937.
</para>
</listitem>
<listitem>
<para>
- Improved error messages when rejecting intercepted requests
- with unknown destination.
+ Unblock 'adlibris.com'.
+ Reported by Wyrex in #935
</para>
</listitem>
<listitem>
<para>
- A couple of log messages now include the number of active threads.
+ Unblock .golang.org/
</para>
</listitem>
<listitem>
<para>
- Removed non-standard Proxy-Agent headers in HTTP snipplets
- to make testing more convenient.
+ Add fast-redirects exception for '.youtube.com/.*origin=http'
+ </para>
+ </listitem>
+ </itemizedlist>
+ </para>
+ </listitem>
+ <listitem>
+ <para>
+ Privoxy-Log-Parser:
+ <itemizedlist>
+ <listitem>
+ <para>
+ Don't gather host and resource statistics if they aren't requested.
+ While the performance impact seems negligible this significantly
+ reduces the memory usage if there are lots of requests.
</para>
</listitem>
<listitem>
<para>
- Include the error code for pcre errors Privoxy does not recognize.
+ Bump version as the behaviour (slightly) changed.
</para>
</listitem>
<listitem>
<para>
- Config directives with numerical arguments are checked more carefully.
+ Count connection failures as well in statistics mode.
+ Sponsored by: Robert Klemme
</para>
</listitem>
<listitem>
<para>
- Privoxy's malloc() wrapper has been changed to prevent zero-size
- allocations which should only occur as the result of bugs.
+ Count connection timeouts as well in statistics mode.
+ Sponsored by: Robert Klemme
</para>
</listitem>
<listitem>
<para>
- Various cosmetic changes.
+ Fix an 'uninitialized value' warning when generating
+ statistics for a log file without response headers.
+ While privoxy-log-parser was supposed to detect this already,
+ the check was flawed and the message the user didn't see was
+ somewhat confusing anyway.
+ Now the message is less confusing, more helpful and actually printed.
+ Reported by: Robert Klemme
</para>
</listitem>
</itemizedlist>
</listitem>
<listitem>
<para>
- Action file improvements:
+ Documentation improvements:
<itemizedlist>
<listitem>
<para>
- Unblock ".deutschlandradiokultur.de/".
- Reported by u302320 in #924.
+ Refer to the git sources instead of CVS.
</para>
</listitem>
<listitem>
<para>
- Add two fast-redirect exceptions for "yandex.ru".
+ Use GNU/Linux when referring to the OS instead of the kernel.
</para>
</listitem>
<listitem>
<para>
- Disable filter{banners-by-size} for ".plasmaservice.de/".
+ Add FAQ entry for what to do if editing the config file is access denied.
</para>
</listitem>
<listitem>
<para>
- Unblock "klikki.fi/adv/".
+ Add brief HTTP/2 FAQ.
</para>
</listitem>
<listitem>
<para>
- Block requests for "resources.infolinks.com/".
- Reported by "Black Rider" on ijbswa-users@.
+ Add a small fuzzing section to the developer documentation.
</para>
</listitem>
<listitem>
<para>
- Block a bunch of criteo domains.
- Reported by Black Rider.
+ Add a client-header-tagger{client-ip-address} example.
</para>
</listitem>
<listitem>
<para>
- Block "abs.proxistore.com/abe/".
- Reported by Black Rider.
+ Stop suggesting that Privoxy is an anonymizing proxy.
+ The term could lead to Privoxy users overestimating
+ what it can do on its own (without Tor).
</para>
</listitem>
<listitem>
<para>
- Disable filter{banners-by-size} for ".black-mosquito.org/".
+ Make it more obvious that SPI accepts Paypal, too.
+ Currently most donations are made through the Paypal account
+ managed by Zwiebelfreunde e.V. and a more even distribution
+ would be useful.
</para>
</listitem>
<listitem>
<para>
- Disable fast-redirects for "disqus.com/".
+ Suggest to log applying actions as well when reproducing problems.
+ </para>
+ </listitem>
+ <listitem>
+ <para>
+ Explicitly mention that Privoxy binaries are built by individuals
+ on their own systems. Buyer beware!
+ </para>
+ </listitem>
+ <listitem>
+ <para>
+ Mention the release feed on the homepage.
+ </para>
+ </listitem>
+ <listitem>
+ <para>
+ Remove a mysterious comment with a GNU FDL link as it isn't
+ useful and could confuse license scanners.
+ In May 2002 it was briefly claimed that "this document" was covered
+ by the GNU FDL. The commit message (r1.5) doesn't explain the motivation
+ or whether all copyright holders were actually asked and agreed to the
+ declared license change.
+ It's thus hard to tell whether or not the license change was legit,
+ but luckily two days later the "doc license" was "put" "back to GPL"
+ anyway (r1.6).
+ At the same time the offending comment with a link to the FDL
+ (not the GPL) was added for no obvious reason.
+ Now it's gone again.
+ </para>
+ </listitem>
+ </itemizedlist>
+ </para>
+ </listitem>
+ <listitem>
+ <para>
+ Regression tests:
+ <itemizedlist>
+ <listitem>
+ <para>
+ Bump for-privoxy-version to 3.0.27 as we now rely on untrusted
+ CGI request being rejected with status code 403 (instead of 200).
+ </para>
+ </listitem>
+ <listitem>
+ <para>
+ Update test for /send-stylesheet and add another one
</para>
</listitem>
</itemizedlist>
</listitem>
<listitem>
<para>
- Documentation improvements:
+ Templates:
<itemizedlist>
<listitem>
<para>
- FAQ: Explicitly point fingers at ASUS as an example of a
- company that has been reported to force malware based on
- Privoxy upon its customers.
+ Consistently use https:// when linking to the Privoxy website.
</para>
</listitem>
<listitem>
<para>
- Correctly document the action type for a bunch of "multi-value"
- actions that were incorrectly documented to be "parameterized".
- Reported by Gregory Seidman on ijbswa-users@.
+ Remove SourceForge references in Copyright header.
</para>
</listitem>
<listitem>
<para>
- Fixed the documented type of the forward-override{} action
- which is obviously 'parameterized'.
+ Remove a couple of SourceForge references in a comment.
+ While at it, fix the grammar.
+ </para>
+ </listitem>
+ <listitem>
+ <para>
+ Move the site-specific documentation block before the generic one.
+ While most Privoxy installations don't have a site-specific
+ documentation block, in cases were it exists it's likely to
+ be more relevant than the generic one.
+ Showing it first makes it less likely that users stop reading
+ before they reach it, especially on pages that don't fit on
+ the screen.
</para>
</listitem>
</itemizedlist>
</listitem>
<listitem>
<para>
- Website improvements:
+ Build system improvements:
<itemizedlist>
<listitem>
<para>
- Users who don't trust binaries served by SourceForge
- can get them from a mirror. Migrating away from SourceForge
- is planned for 2016 (TODO list item #53).
+ Prefer openjade to jade. On some systems Jade produces
+ HTML with unescaped ampersands in URLs.
+ </para>
+ </listitem>
+ <listitem>
+ <para>
+ Prefer OpenSP to SP to be consistent.
+ </para>
+ </listitem>
+ <listitem>
+ <para>
+ Have Docbook generated HTML files be straight ASCII.
+ Dealing with a mixture of ISO-8859 and UTF-8 files is problematic.
+ </para>
+ </listitem>
+ <listitem>
+ <para>
+ Echo the filename to stderr for 'make dok-tidy'.
+ Make it a bit easier to find errors in docbook generated HTML.
+ </para>
+ </listitem>
+ <listitem>
+ <para>
+ Warn when still using select().
+ </para>
+ </listitem>
+ <listitem>
+ <para>
+ Warn when compiling without calloc().
+ </para>
+ </listitem>
+ <listitem>
+ <para>
+ Make it more obvious that the --with-fdsetsize configure switch
+ is pointless if poll() is available.
</para>
</listitem>
<listitem>
<para>
- The website is now available as onion service
- (http://jvauzb4sb3bwlsnc.onion/).
+ Remove support for AmigaOS.
+ </para>
+ </listitem>
+ <listitem>
+ <para>
+ Update windows build system to use supported software.
+ The cygwin gcc -mno-cygwin option is no longer supported, so
+ convert the windows build system to use the cygwin cross-compiler
+ to build "native" code.
+ </para>
+ </listitem>
+ <listitem>
+ <para>
+ Add --enable-static-linking option for configure
+ does the same thing as LDFLAGS=-static; ./configure
+ but nicer than mixing evars and configure options.
</para>
</listitem>
</itemizedlist>
</para>
</listitem>
</itemizedlist>
-</para>
+
projects / privoxy.git / blobdiff