Revert "Rework of worker threads' start and exit + slight changes in cleanup code"

This reverts commit 4f3aafa1e422e9aa005b74c8a1d40cc3e9143346.

Once merged upstream, the CI failed here with:

Warning: Failed to confirm worker threads' start up after timeout.
Warning: Bug:     /builds/tpo/core/tor/build/src/app/tor(_start+0x2a) [0x56404d21ddaa] (on Tor 0.4.9.0-alpha-dev 4f3aafa1e422e9aa)
Warning: Bug:     /builds/tpo/core/tor/build/src/app/tor(cpuworker_init+0x53) [0x56404d373d53] (on Tor 0.4.9.0-alpha-dev 4f3aafa1e422e9aa)
Warning: Bug:     /builds/tpo/core/tor/build/src/app/tor(log_backtrace_impl+0x57) [0x56404d29e1f7] (on Tor 0.4.9.0-alpha-dev 4f3aafa1e422e9aa)
Warning: Bug:     /builds/tpo/core/tor/build/src/app/tor(main+0x19) [0x56404d21dd59] (on Tor 0.4.9.0-alpha-dev 4f3aafa1e422e9aa)
Warning: Bug:     /builds/tpo/core/tor/build/src/app/tor(run_tor_main_loop+0xce) [0x56404d22188e] (on Tor 0.4.9.0-alpha-dev 4f3aafa1e422e9aa)
Warning: Bug:     /builds/tpo/core/tor/build/src/app/tor(threadpool_new+0x295) [0x56404d3e28f5] (on Tor 0.4.9.0-alpha-dev 4f3aafa1e422e9aa)
Warning: Bug:     /builds/tpo/core/tor/build/src/app/tor(tor_assertion_failed_+0x148) [0x56404d2a9248] (on Tor 0.4.9.0-alpha-dev 4f3aafa1e422e9aa)
Warning: Bug:     /builds/tpo/core/tor/build/src/app/tor(tor_main+0x49) [0x56404d21e179] (on Tor 0.4.9.0-alpha-dev 4f3aafa1e422e9aa)
Warning: Bug:     /builds/tpo/core/tor/build/src/app/tor(tor_run_main+0x1e5) [0x56404d221db5] (on Tor 0.4.9.0-alpha-dev 4f3aafa1e422e9aa)
Warning: Bug:     /lib/x86_64-linux-gnu/libc.so.6(__libc_start_main+0xea) [0x7f6aa3b1cd7a] (on Tor 0.4.9.0-alpha-dev 4f3aafa1e422e9aa)
Warning: Bug: Tor 0.4.9.0-alpha-dev (git-4f3aafa1e422e9aa): Assertion 0 failed in threadpool_new at /builds/tpo/core/tor/src/lib/evloop/workqueue.c:641: . Stack trace: (on Tor 0.4.9.0-alpha-dev 4f3aafa1e422e9aa)

We are figuring it out but revert this so we can release an alpha.

Rework of worker threads' start and exit + slight changes in cleanup code

See issue #40991

Merge branch 'maint-0.4.8'

test-network-run: refactor away test_network_ipv6

This should be a pure refactor to simplify the logic a bit now that we
no longer probe for IPv6 support.

test-network: include IPv6 tests unconditionally

Previously we would incorrectly detect that ipv6 isn't supported if the
ping binary isn't present (as it may not be in a relatively stripped
down container image), or if ICMP packets aren't permitted (as they
often aren't in containers).

We don't really have a need to run these network tests on non-IPv6
systems, so it makes more sense to just run them unconditionally.

Merge branch 'maint-0.4.8'

debian-i386-minimal: explicitly set platform

This is to work around
https://github.com/docker-library/official-images/issues/17896

CI: Use TPA-based images

debian-gpl: Run on debian:bullseye instead of buster

* buster is now very old
* all the other jobs run bullseye
* This will facilitate switching to a TPA-maintained image (there is no
  buster image).

Merge branch 'tpa-images' into 'main'

Update images

See merge request tpo/core/tor!846

debian-i386-minimal: explicitly set platform

This is to work around
https://github.com/docker-library/official-images/issues/17896

CI: Use TPA-based images

debian-gpl: Run on debian:bullseye instead of buster

* buster is now very old
* all the other jobs run bullseye
* This will facilitate switching to a TPA-maintained image (there is no
  buster image).

Merge branch 'maint-0.4.8'

Fix: Use FREE_AND_NULL() for releasing workqueue resources

See issue #40989

Merge branch 'pin-chutney' into 'main'

CI: use a fixed version of chutney

See merge request tpo/core/tor!843

CI: use a fixed version of chutney

While chutney currently runs tor's chutney test in its own CI,
it's difficult to guarantee the two won't accidentally diverge.
Probably best to use a fixed version here so that we can control
chutney version bumps and avoid surprise breakage in tor's CI.

This will also free us to intentionally make breaking changes in
chutney (though I don't have any immediate plans for any).

Merge branch 'whitespace' into 'main'

whitespace fix while debugging #40981

See merge request tpo/core/tor!841

whitespace fix while debugging #40981

Merge branch 'maint-0.4.8'

Merge branch 'tor-gitlab/mr/840' into maint-0.4.8

Use pip to install chutney's python dependencies

Merge branch 'maint-0.4.8'

version: Bump version to 0.4.8.13-dev

Re-format CHUTNEY installation block into multiple lines

We're about to add a few more steps, which would make the single-line
format a bit unwieldy.

Merge branch 'maint-0.4.8'

version: Bump version to 0.4.8.13

Merge branch 'maint-0.4.8'

fallbackdir: Update list generated on October 24, 2024

Update geoip files to match ipfire location db, 2024/10/24.

Revert "Handle empty DNS reply without error as NOERROR"

Closes #40984

This reverts commit 3d2f7c3467e7aff545d8e22ffd4153d7598c6d7c.

Merge branch 'maint-0.4.8'

Merge branch 'tor-resolve-bugfix' into 'main'

Create socket with correct family as given by sockshost

See merge request tpo/core/tor!836

Add changes file

don't build preemptive conflux circuits if no predicted ports

Conflux circuit building was ignoring the "predicted ports" feature,
which aims to make Tor stop building circuits if there have been
no user requests lately. This bug led to every idle Tor on the
network building and discarding circuits every 30 seconds, which
added overall load to the network, used bandwidth and battery from
clients that weren't actively using their Tor, and kept sockets open
on guards which added connection padding essentially forever.

Bug went in on commit 39c2927d when we added preemptive conflux circuit
pools.

Fixes bug 40981; bugfix on 0.4.8.1-alpha.

Create socket with correct family as given by sockshost

Merge branch 'changelogs' into 'main'

forward-port the 0.4.8.12 changes

See merge request tpo/core/tor!834

Merge branch 'maint-0.4.8'

Fix: Memory leaks in cpuworker on shutdown

Resources allocated by cpuworker weren't being freed on clean shutdown.
This applies for worker threads, worker thread pool, reply queue, reply
event, ...

changes: Add file for ticket 40248

Signed-off-by: David Goulet <dgoulet@torproject.org>

Merge branch 'empty-dns' into 'main'

Handle empty DNS reply without error as NOERROR

Closes #40248

See merge request tpo/core/tor!830

forward-port the 0.4.8.12 changes

Merge branch 'fix-unittest' into 'main'

Fix unit test in test_entrynodes.c

See merge request tpo/core/tor!831

Fix unit test in test_entrynodes.c

Handle empty DNS reply without error as NOERROR

Merge branch 'typo-socks-proxy' into 'main'

Fix a typo in the HTTP proxy error message.

See merge request tpo/core/tor!829

test: Add unit tests for prop351

Signed-off-by: David Goulet <dgoulet@torproject.org>

Fix a typo in the HTTP proxy error message.

Implement proposal 351

This proposal adds new syntax to the SOCKS5 username/password extension scheme,
so that requests with usernames starting with <torS0X> are now reserved.

For C tor, all we need to do is reject every username starting with <torS0X>
unless it is exactly "<torS0X>0".

Merge branch 'maint-0.4.8'

dirvote: Fix memleak when computing consensus

Fixes #40966

Signed-off-by: David Goulet <dgoulet@torproject.org>

Merge branch 'manpage-typos' into 'main'

two small presentation issues in man page

See merge request tpo/core/tor!824

two small presentation issues in man page

Merge branch 'tap-out-phase-1' into 'main'

Implement phase 1 of Proposal 350 (remove TAP onion keys)

See merge request tpo/core/tor!822

Changes file for proposal 350 phase 1

Do not publish TAP key when publish-dummy-tap-key is 0.

Update supported protovers for prop350

Relay=1 is no longer supported; it corresponds to TAP.

Microdesc=3 and Desc=3 are now supported; they correspond to the ability to
handle (micro)descriptors without TAP onion keys.

Add a test for parsing a routerdesc with no TAP key.

Add a test for a microdesc with no onion key.

Remove a now-unused variable.

process_descs: Authorities require TAP keys for now.

Routerparse: accept routerdescs without TAP keys.

routerparse: Simplify checking for now-mandatory elements

All of these elements are now mandatory, so we can now simplify
our logic for making sure that they are all present or all not-present.

Rename "onion_pkey" fields in routerinfo_t, and make them optional.

(Renaming them has forced me to look at every place where they are used, so I
can make sure that they are really optional now.)

Remove node_get_rsa_onion_key

Remove TAP key from extend_info_t

Make onion-key body optional in microdescs

Also, stop storing onion keys in microdesc_t.

(In prop350, for microdescs, we are making the body optional; the "onion-key"
entry is still mandatory, so that we can tell where microdescs begin.)

Remove support for deciding to use CREATE/EXTEND/TAP.

prop350: Remove all support for TAP, CREATE, and EXTEND.

prop350: Stop accepting CREATE and EXTEND.

Merge branch 'maint-0.4.8'

bridge: Always put transport-info line

Signed-off-by: David Goulet <dgoulet@torproject.org>

Clean-up Managed Proxy state in PT tests.

This patch makes sure we clean up our version and implementation fields
in our Managed Proxy struct after each test run.

This was detected by LeakSanitizer.

See: tpo/core/tor#11101.

Add changes file for tpo/core/tor#11101.

Add implementation and version metadata to bridge extra-info.

This patch adds two new keys to bridges' extra-info document:
"transport-version" and "transport-implementation".

These two new values always appear together (if one is missing, the
other one will be missing too) and is parsed from PT's STATUS
TYPE=version messages.

See: tpo/core/tor#11101.

Include "IMPLEMENTATION" parameter to STATUS TYPE=version PT messages.

Drop requirement for TRANSPORT being present in STATUS messages.

Add support for PT STATUS TYPE=version messages.

This patch adds support for handling the version status message. Once we
receive such message, we add the given version string to the
managed_proxy_t instance. Note this value can be NULL and the value can
change throughout the lifetime of the PT as multiple status version
messages are handled.

See: tpo/core/tor#11101

Merge branch 'sandbox-bwauth' into 'main'

fix sandbox for bandwidth authority

Closes #40933

See merge request tpo/core/tor!821

fix sandbox for bandwidth authority

Merge branch 'maint-0.4.8'

version: Bump version to 0.4.8.12-dev

Merge branch 'maint-0.4.8'

version: Bump version to 0.4.8.12

Merge branch 'maint-0.4.8'

fallbackdir: Update list generated on June 06, 2024

Update geoip files to match ipfire location db, 2024/06/06.

Merge branch 'maint-0.4.8'

Update OverloadStatistics to match prop 328

Closes: #40940.

Merge branch 'maint-0.4.8'

Merge branch 'ticket40689_048_01' into maint-0.4.8

Merge branch 'ticket40689_048_01' into 'main'

dirauth: Add new faravahar

Closes #40689

See merge request tpo/core/tor!819

Merge branch 'maint-0.4.8'

scripts: Fix Python3 warnings

Signed-off-by: David Goulet <dgoulet@torproject.org>

dirauth: Add new faravahar

Brand new address, brand new keys, brand new day.

Closes #40689

Signed-off-by: David Goulet <dgoulet@torproject.org>

Merge branch 'typos' into 'main'

fix a few comment typos

See merge request tpo/core/tor!818

fix a few comment typos

Merge branch 'maint-0.4.8'

Fix typo in tor-relays list address

Merge branch 'maint-0.4.8'