Update the SGML changelog
[privoxy.git] / doc / source / changelog.sgml
index ebd270b..2381d86 100644 (file)
@@ -1,11 +1,9 @@
 <!--
- 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.10 2014/11/18 14:22:17 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.23</application> stable is a bug-fix release,
-  some of the fixed bugs are security issues (CVE requests pending):
-</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>
+    General improvements:
+    <itemizedlist>
+    <listitem>
+     <para>
+      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>
+      Add a listen-backlog directive which specifies the backlog
+      value passed to listen().
+      Sponsored by: Robert Klemme
+     </para>
+    </listitem>
+    <listitem>
+     <para>
+      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>
+      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>
+      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>
+      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>
+      Consistently use the U(ngreedy) flag in the 'img-reorder' filter.
+     </para>
+    </listitem>
+    <listitem>
+     <para>
+      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>
+      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>
+      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>
-      Fixed a DoS issue in case of client requests with incorrect
-      chunk-encoded body. When compiled with assertions enabled
-      (the default) they could previously cause Privoxy to abort().
-      Reported by Matthew Daley.
+      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 multiple segmentation faults and memory leaks in the
-      pcrs code. This fix also increases the chances that an invalid
-      pcrs command is rejected as such. Previously some invalid commands
-      would be loaded without error. Note that Privoxy's pcrs sources
-      (action and filter files) are considered trustworthy input and
-      should not be writable by untrusted third-parties.
+      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>
-      Fixed an 'invalid read' bug which could at least theoretically
-      cause Privoxy to crash. So far, no crashes have been observed.
+      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>
-      Compiles with --disable-force again. Reported by Kay Raven.
+      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>
-      Client requests with body that can't be delivered no longer
-      cause pipelined requests behind them to be rejected as invalid.
-      Reported by Basil Hussain.
+      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>
-      If a pcrs command is rejected as invalid, Privoxy now logs
-      the cause of the problem as text. Previously the pcrs error
-      code was logged.
+      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>
+      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>
+      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>
+      Unblock 'adventofcode.com/'.
+      Reported by Clint Adams in Debian bug #848211.
+      Fixes Roland's AF#937.
+     </para>
+    </listitem>
+    <listitem>
+     <para>
+      Unblock 'adlibris.com'.
+      Reported by Wyrex in #935
      </para>
     </listitem>
     <listitem>
      <para>
-      The tests are less likely to cause false positives.
+      Unblock .golang.org/
+     </para>
+    </listitem>
+    <listitem>
+     <para>
+      Add fast-redirects exception for '.youtube.com/.*origin=http'
      </para>
      </listitem>
     </itemizedlist>
   </listitem>
   <listitem>
    <para>
-    Action file improvements:
+    Privoxy-Log-Parser:
     <itemizedlist>
     <listitem>
      <para>
-      '.sify.com/' is no longer blocked. Apparently it is not actually
-      a pure tracking site (anymore?). Reported by Andrew on ijbswa-users@.
+      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>
-      Unblock banners on .amnesty.de/ which aren't ads.
+      Bump version as the behaviour (slightly) changed.
+     </para>
+    </listitem>
+    <listitem>
+     <para>
+      Count connection failures as well in statistics mode.
+      Sponsored by: Robert Klemme
+     </para>
+    </listitem>
+    <listitem>
+     <para>
+      Count connection timeouts as well in statistics mode.
+      Sponsored by: Robert Klemme
+     </para>
+    </listitem>
+    <listitem>
+     <para>
+      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>
     <itemizedlist>
     <listitem>
      <para>
-      The 'Would you like to donate?' section now also contains
-      a "Paypal" address.
+      Refer to the git sources instead of CVS.
+     </para>
+    </listitem>
+    <listitem>
+     <para>
+      Use GNU/Linux when referring to the OS instead of the kernel.
+     </para>
+    </listitem>
+    <listitem>
+     <para>
+      Add FAQ entry for what to do if editing the config file is access denied.
+     </para>
+    </listitem>
+    <listitem>
+     <para>
+      Add brief HTTP/2 FAQ.
+     </para>
+    </listitem>
+    <listitem>
+     <para>
+      Add a small fuzzing section to the developer documentation.
+     </para>
+    </listitem>
+    <listitem>
+     <para>
+      Add a client-header-tagger{client-ip-address} example.
+     </para>
+    </listitem>
+    <listitem>
+     <para>
+      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>
+      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>
+      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>
+   </para>
+  </listitem>
+  <listitem>
+   <para>
+    Templates:
+    <itemizedlist>
+    <listitem>
+     <para>
+      Consistently use https:// when linking to the Privoxy website.
+     </para>
+    </listitem>
+    <listitem>
+     <para>
+      Remove SourceForge references in Copyright header.
+     </para>
+    </listitem>
+    <listitem>
+     <para>
+      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>
+   </para>
+  </listitem>
+  <listitem>
+   <para>
+    Build system improvements:
+    <itemizedlist>
+    <listitem>
+     <para>
+      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>
-      The list of supported operating systems has been updated.
+      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>
-      The existence of the SF support and feature trackers has been
-      deemphasized because they have been broken for months.
-      Most of the time the mailing lists still work.
+      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>
-      The claim that default.action updates are sometimes released
-      on their own has been removed. It hasn't happened in years.
+      Warn when still using select().
      </para>
     </listitem>
     <listitem>
      <para>
-      Explicitly mention that Tor's port may deviate from the default
-      when using a bundle. Requested by Andrew on ijbswa-users@.
+      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>
+      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>
+