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'

Merge branch 'm4-serial-fix' into 'main'

m4: update ax_check_compile_flag.m4 from autoconf-archive

See merge request tpo/core/tor!813

Merge branch 'changelogs' into 'main'

forward-port the 0.4.8.10 and .11 changelogs

See merge request tpo/core/tor!814

fix changelog typos and clean up a bit

m4: update ax_check_compile_flag.m4 from autoconf-archive

I noticed this when doing some M4 macro analysis work, and saw
that the version of `ax_check_compile_flag.m4` in Tor has two serial
lines ('serial 5' and 'serial 6') which is invalid.

We could just fix one of the lines, but it makes more sense to just
sync with upstream, I think.

Import ax_check_compile_flag.m4 from autoconf-archive at latest
commit at time of writing (4e8aab846b0872fba99f1fe02ebcdff178a34c87).

Signed-off-by: Sam James <sam@gentoo.org>

Merge branch 'typos' into 'main'

Typos

See merge request tpo/core/tor!812

forward-port the 0.4.8.10 and .11 changelogs

typos in comments

fix syntax formatting in man page

Merge branch 'typo' into 'main'

typo

See merge request tpo/core/tor!811

typo

circuit: Remove log_warn triggered by a proto violation

Fixes #40932

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

Merge branch 'count-log-bug' into 'main'

also consider LD_BUG logs when counting bug reached

Closes #40924

See merge request tpo/core/tor!805

Merge branch 'maint-0.4.8'

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

conflux: Don't process a closed circuit on get packaged window

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

conflux: Avoid noting a cell was sent on a closed circuit

It turns out that circuit_package_relay_cell() returns 0 in order to drop a
cell but there is a code path, if the circuit queue is full, that also silently
closes the circuit and returns 0.

This lead to Conflux thinking a cell was sent but actually the cell was not and
the circuit was closed leading to the hard assert.

And so this function makes sure that circuit_package_relay_cell() and
append_cell_to_circuit_queue() returns a value that indicate what happened with
the cell and circuit so the caller can make an informed decision with it.

This change makes it that we do NOT enter the Conflux subsystem if the cell is
not queued on the circuit.

Fixes #40921

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

rename metric bug_reached to bug_reached_count

also consider LD_BUG logs when counting bug reached

Merge branch 'maint-0.4.8'

version: Bump version to 0.4.8.11-dev

Merge branch 'maint-0.4.8'

version: Bump version to 0.4.8.11

Merge branch 'maint-0.4.8'

fallbackdir: Update list generated on April 10, 2024

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

Merge branch 'maint-0.4.8'

dirauth: New keys and address for tor26

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

test: Disable unit test causing issue with nightly package

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

Merge branch 'testing-intro-rot' into 'main'

hs_service: don't force high intro point rotation for TestingTorNetwork

Closes #40922

See merge request tpo/core/tor!802

hs_service: don't force high intro point rotation for TestingTorNetwork

This was previously added to facilitate testing intro point rotation
with chutney. However, the implementation is problematic since it forces
excessive rotation whenever TestingTorNetwork is enabled, and can't be
adjusted or disabled.

Alternatives for testing intro point rotation include:
 * Using shadow to "fast forward" time
 * Overriding the consensus parameters hs_intro_min_lifetime and
   hs_intro_max_lifetime.

Fixes #40922