Remove silver sponsor www.top10vpn.com

The sponsorship period ended in September ...

developer-manual: Update paragraph to reflect that Privoxy-Regression-Test now defaults to using 127.0.0.1:8118/ as privoxy address

Add #180: Add support for GnuTLS

privoxy-regression-test: Use 127.0.0.1:8118/ as default privoxy address

... unless http_proxy is set through the environment.

Remove an obsolete comment

Remove pointless redefinition of 'privoxy_mutex_t'

... when compiling with FEATURE_HTTPS_INSPECTION.

Silences warnings when compiling with "-std=c99":

    cc -c -pipe -fstack-protector-all -ggdb -Wshadow  -Wconversion -I/usr/local/include/  -pthread -Wall -std=c99   errlog.c -o errlog.o
    In file included from errlog.c:70:
    ./jcc.h:66:25: warning: redefinition of typedef 'privoxy_mutex_t' is a C11 feature [-Wtypedef-redefinition]
    typedef pthread_mutex_t privoxy_mutex_t;
                            ^
    ./project.h:59:28: note: previous definition is here
       typedef pthread_mutex_t privoxy_mutex_t;
                               ^
    1 warning generated.

Compile-tested on Windows by Lee.

create_server_ssl_connection(): Declare a variable at the beginning of the code block

... to silence:
    cc -c -pipe -fstack-protector-all -ggdb -Wshadow  -Wconversion -I/usr/local/include/  -pthread -Wall -std=c89   openssl.c -o openssl.o
    openssl.c:1144:12: warning: GCC does not allow variable declarations in for loop initializers before C99 [-Wgcc-compat]
          for (int i = 0; i < sk_X509_num(chain); i++)
               ^

ssl_store_cert(): Declare a variable at the beginning of the the code block

... to silence:

    cc -c -pipe -fstack-protector-all -ggdb -Wshadow  -Wconversion -I/usr/local/include/  -pthread -Wall -std=c89   openssl.c -o openssl.o
    openssl.c:408:12: warning: GCC does not allow variable declarations in for loop initializers before C99 [-Wgcc-compat]
          for (int i = 0; i < bs->length; i++)
               ^

Remove 'inline' from a bunch of functions and leave the optimization decision to the compiler

Unbreaks the build with '-std=c89'. Previous failure:
    cc -c -pipe -fstack-protector-all -ggdb -Wshadow  -Wconversion -I/usr/local/include/  -pthread -Wall -std=c89   errlog.c -o errlog.o
    errlog.c:95:8: error: unknown type name 'inline'
    static inline void lock_logfile(void)
           ^
    errlog.c:99:8: error: unknown type name 'inline'
    static inline void unlock_logfile(void)
           ^
    errlog.c:103:8: error: unknown type name 'inline'
    static inline void lock_loginit(void)
           ^
    errlog.c:107:8: error: unknown type name 'inline'
    static inline void unlock_loginit(void)
           ^
    errlog.c:447:8: error: unknown type name 'inline'
    static inline size_t get_log_timestamp(char *buffer, size_t buffer_size)
           ^
    errlog.c:447:21: error: expected ';' after top level declarator
    static inline size_t get_log_timestamp(char *buffer, size_t buffer_size)
                        ^
                        ;

action_render_string_filters_template(): Declare a variable at the beginning

... of the function to silence a compiler warning when building with -std=c89:

   cc -c -pipe -fstack-protector-all -ggdb -Wshadow  -Wconversion -I/usr/local/include/  -pthread -Wall -std=c89   cgiedit.c -o cgiedit.o
   cgiedit.c:4436:9: warning: GCC does not allow variable declarations in for loop initializers before C99 [-Wgcc-compat]
      for (int i=0; i < SZ(desc); ++i)
           ^
   1 warning generated.

Add #178: Add a add-server-header{} action

Add #178: Warn on config.privoxy.org/client-tags

... if a Tag name has not at least one matching action section.

Add #177: Support https-inspection for intercepted requests

Add #176: Find a new fiduciary sponsor as a replacement for Zwiebelfreunde e.V.

Add #175: Add more screenshots to the documentation and website

Remove #137

A logo has been added recently to the website.

configure: Update the link to the 'Removing outdated PCRE version ...' thread

Rebuild docs

Rebuild README

Check the chdir() return code

... to fix the compiler warning:
   jcc.c: In function ‘main’:
   jcc.c:5185:7: warning: ignoring return value of ‘chdir’, declared with attribute warn_unused_result [-Wunused-result]
          chdir("/");
          ^~~~~~~~~~

developer-manual: Mention the directory from which to execute the commands to create Debian packages

Regenerate config file

Mention regression-tests.action in the config file

Improve the message shown when the client-tags CGI page is requested with no tags configued

Use the '/sponsor' redirect for the link to the sponsor page

Explicitly mention that access to the ca key should be limited to Privoxy

Gracefully handle existing website keys without matching certificates

This can happen if Privoxy was previously running with an invalid
TLS configuration that didn't allow it to create a certificate.

The problem can be reproduced manually by removing or renaming a
certificate while keeping the key.

Previously this would result in a confusing client error messages:

    fk@t520 ~ $curl -v --head https://www.electrobsd.org/
    * Uses proxy env variable https_proxy == 'http://127.0.1.1:8118/'
    *   Trying 127.0.1.1:8118...
    * Connected to 127.0.1.1 (127.0.1.1) port 8118 (#0)
    * allocate connect buffer!
    * Establish HTTP proxy tunnel to www.electrobsd.org:443
    > CONNECT www.electrobsd.org:443 HTTP/1.1
    > Host: www.electrobsd.org:443
    > User-Agent: curl/7.72.0
    > Proxy-Connection: Keep-Alive
    >
    < HTTP/1.1 200 Connection established
    HTTP/1.1 200 Connection established
    <

    * Proxy replied 200 to CONNECT request
    * CONNECT phase completed!
    * ALPN, offering h2
    * ALPN, offering http/1.1
    * Cipher selection: ALL:!EXPORT:!EXPORT40:!EXPORT56:!aNULL:!LOW:!RC4:@STRENGTH
    * successfully set certificate verify locations:
    *   CAfile: /usr/local/share/certs/ca-root-nss.crt
      CApath: none
    * TLSv1.2 (OUT), TLS header, Certificate Status (22):
    * TLSv1.2 (OUT), TLS handshake, Client hello (1):
    * OpenSSL SSL_connect: SSL_ERROR_SYSCALL in connection to www.electrobsd.org:443
    * Closing connection 0
    curl: (35) OpenSSL SSL_connect: SSL_ERROR_SYSCALL in connection to www.electrobsd.org:443

While the Privoxy log would say something like:
04:53:53.932 099 Error: Subject key was already created
04:53:53.932 099 Error: Loading webpage certificate /usr/local/etc/privoxy/CA/certs/6db5da8a16c246d1bd8c0fa7cd160a5b.crt failed: error:02001002:system library:fopen:No such file or directory
04:53:53.932 099 Error: Loading webpage certificate /usr/local/etc/privoxy/CA/certs/6db5da8a16c246d1bd8c0fa7cd160a5b.crt failed: error:20074002:BIO routines:file_ctrl:system lib
04:53:53.933 099 Error: Loading webpage certificate /usr/local/etc/privoxy/CA/certs/6db5da8a16c246d1bd8c0fa7cd160a5b.crt failed: error:140AD002:SSL routines:SSL_CTX_use_certificate_file:system lib
04:53:53.933 099 Error: Failed to open a secure connection with the client

Instead of failing, just remove the key and continue.

List more client-specific-tag examples for inspiration

Fix grammar

redirect_url(): Use two seperate variables with limited scope

... instead of having two if blocks share a single variable
for no obvious reason.

Remove reference to 'How to Report Bugs Effectively'

It was only rendered as text without URL in the README anyway
and there's no indication that users read it ...

Remove a comment that claimed that the version number is for RPM

Remove reference to SourceForge

... as users may get official packages from the website too now.

Rephrase readme purpose

Don't repeat the fact that the generated file is named README.

Let the dok-readme target fix the location embedded into the README file

This used to be done by CVS but since the git migration
it has to be done throuhg other means.

Update meaning of debug bit 4

Bump copyright

Mark TODO #14 as work in progress and link to the patch tracker

Clarify that only Privoxy team members can object to new sponsors

... and link to the list of current team members.

Remove 'experimental' warning for client-specific-tag-related directives

They seem to work reliably and there is no obvious reason
why we would change the syntax in the near future.

Rebuild HTML man page

Rebuild man page

Use the new donate link in the TODO list

Rebuild config file

Mention that HTTPS inspection also allows to filter encrypted responses

Turn a reference to the show-status page into a link

... when rendered for the user manual.

Describe how to check if Privoxy has been built with FEATURE_HTTPS_INSPECTION

Add a link to the trusted-cas-file documentation

... that explains how the user can create the file herself.

Update link to the cacert.pem file

Add #174: Let the Tor Onion Service for the privoxy website serve gitweb and the git repository as well

privoxy-log-parser: Add a handler for tagging messages

Convert a couple of additional messages to log level "Tagging"

Don't explicitly mention the license for the code coming from 'Anonymous Coders' and Junkbusters

It's obviously licensed under the GNU GPL like the
rest of Privoxy or we wouldn't be allowed to distribute
it.

privoxy-log-parser: Highlight the new "Tagging" log level in purple

privoxy-log-parser: Bump version to 0.9.2

privoxy-log-parser: Accept and ignore tagging-related log messages for now

Document the new meaning of debug bit 4

Recycle debug bit 4 for Tagging-related messages

configure: Move the comment describing the version number above the variables

Block requests to eu-tlp03.kameleoon.com/

Add another hide-referrer{conditional-block} test

Add another hide-referrer{conditional-forge} test

Fix a hide-referrer{conditional-forge} test

... that expected an acceptable header to be forged.

Fix a hide-referrer{conditional-block} test

... that expected an acceptable Referer to be removed.

Let the hide-referrer code tolerate Referer headers with https:// URLs

Previously they would always be treated like a changed host.

redirect_url(): Check the actual URL when https inspecting requests

Previously we would only check the path which resulted
in rewrite results being rejected as invalid URLs.

Before:
19:37:29.494 014 Error: pcrs command "s@/test@/@" changed "/test" to "/" (1 hit), but the result doesn't look like a valid URL and will be ignored.

After:
19:40:57.857 002 Redirect: pcrs command s@/test@/@ changed https://www.electrobsd.org/test to https://www.electrobsd.org/ (1 hit).

Reported by withoutname in #1736.

Regenerate developer-manual

developer-manual: Remove the packaging instructions for RPM-based systems

They don't work and we don't release RPM packages anymore anyway.

developer-manual: Remove the packaging instructions for Solaris

They don't work and we don't release Solaris packages anymore anyway.

developer-manual: Update the suggested subject for the announce mails

developer-manual: Update upload instructions

ftp://upload.sourceforge.net is no longer functional.

developer-manual: Remove a couple of package-dependend upload instructions

... that don't actually work.

developer-manual: Remove 'cd current' that no longer works

developer-manual: Add regression-tests.action to the list of files that should be installed

developer-manual: Stop claiming that there are text versions of the manuals

We stopped building them in 2008 (9ed36a3c5e6f12).

developer-manual: Note that the 'webserver' target creates the link needed for the user-manual

developer-manual: Suggest to use the master branch as reference when creating the ChangeLog

... so the steps work when the current branch differs
from master which is likely as the developer manual
suggests to use a local branch for development.

Add #173: Document Privoxy's governance model

developer-manual: Add the -s flag to the suggested 'git tag' command

We prefer signed tags.

developer-manual: Regenerate with git steps

developer-manual: Add git commands that should result in a merge-free history

developer-manual: Regenerate

developer-manual: Simplify privoxy-regression-test.pl command in the example

The --debug parameter adds bits without overriding default
flags so it's sufficient to add 4 (LL_PAGE_FETCHING).

developer-manual: Fix typo

Rebuild developer-manual

developer-manual: Add privoxy-regression-test.pl example output

fix compiler warnings about including winsock2.h before windows.h

getting plenty too many of these warnings:
i686-w64-mingw32-gcc -c -pipe -O2 -Wshadow -DWINVER=0x501   -mwindows -Wall -Ipcre  actions.c -o actions.o
In file included from /usr/i686-w64-mingw32/sys-root/mingw/include/ws2tcpip.h:17,
                 from project.h:95,
                 from actions.c:46:
/usr/i686-w64-mingw32/sys-root/mingw/include/winsock2.h:15:2: warning: #warning Please include winsock2.h before windows.h [-Wcpp]
   15 | #warning Please include winsock2.h before windows.h
      |  ^~~~~~~
i686-w64-mingw32-gcc -c -pipe -O2 -Wshadow -DWINVER=0x501   -mwindows -Wall -Ipcre  cgi.c -o cgi.o
In file included from /usr/i686-w64-mingw32/sys-root/mingw/include/ws2tcpip.h:17,
                 from project.h:95,
                 from cgi.c:52:
/usr/i686-w64-mingw32/sys-root/mingw/include/winsock2.h:15:2: warning: #warning Please include winsock2.h before windows.h [-Wcpp]
   15 | #warning Please include winsock2.h before windows.h
      |  ^~~~~~~
i686-w64-mingw32-gcc -c -pipe -O2 -Wshadow -DWINVER=0x501   -mwindows -Wall -Ipcre  cgiedit.c -o cgiedit.o
In file included from /usr/i686-w64-mingw32/sys-root/mingw/include/ws2tcpip.h:17,
                 from project.h:95,
                 from cgiedit.c:57:
/usr/i686-w64-mingw32/sys-root/mingw/include/winsock2.h:15:2: warning: #warning Please include winsock2.h before windows.h [-Wcpp]
   15 | #warning Please include winsock2.h before windows.h
      |  ^~~~~~~

Merge branch 'master' of ssh://git.privoxy.org:23/git/privoxy

Bring back the select()-based fallback code

As reported by Lee it's stille needed on Windows.

This reverts commit 9126f0c935e4bb64a87be7c3a1855b7768791142.

Merge branch 'master' of ssh://git.privoxy.org:23/git/privoxy

Regenerate docs

Add the new action suppress-tag{}

Usage:
in user.filters:
--begin--
CLIENT-HEADER-TAGGER: maximum-url-length Tag for URLS longer than 600 characters.

s@(^GET\s+\/.{600,}\s+HTTP\/\d\.\d\s*$)@MAXIMUM-URL-LENGTH@i
--end--

in user.actions:
--begin--
{+client-header-tagger{maximum-url-length}}
/

{+block{Maximum URL length of 600 bytes reached.}}
TAG:^MAXIMUM-URL-LENGTH

{+suppress-tag{MAXIMUM-URL-LENGTH}}
.google.*
 --end--

will block all URLs with length > 600 bytes except for google.

Currently the online action editor supports modification/removal of any
number of existing tags and the creation of a single suppress tag per
one submit. The submit scheme that is used is similar to the existing
filter one but:

1. It uses 'string_filter[_r|_n|_o|_t][hex_index]' keys for existing
string filter values (id/name(value)/old_name(old value)/filter type)
and 'new_string_filter[_r|_n|_t][hex_index]' for new string filter
values. 'String filter values' here are parameters of the suppress-tag
action that are simple strings rather than parameters of, for example,
the client-header-tagger action that must be described in filters file.

2. String filter values are accessed by the value rather by the
index. Indexes must start from 0 and when there is no key with index+1
in parameters - we've done with existing or new string filters
processing.

Possible further improvements:

1. Extend suppress-tag action edit scheme to add-header action
   edit that is not supported now.
2. If needed, multiple suppress-tag addition can be added with
   some browser JS code.

Sponsored by: Robert Klemme

Rebuild docs for 3.0.29 UNRELEASED

developer-manual: Mention Privoxy-Regression-Test

developer-manual: Add a section id to reduce link churn

developer-manual: Recommend the dok-tidy target when building docs for the webserver

developer-manual: Add another plug for the privoxy-devel mailing list

developer-manual: Mention that merges into 'master' should be avoided

developer-manual: Let the intro link the copyright section in the user manual instead of giving an incomplete summary of the license status

privoxy-regression-test: Replace an XXX comment with a better one