hs-v3: Report SOCKS ExtendedErrors when all intro timed out

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

hs-v3: Report rendezvous circuit failure SOCKS ExtendedErrors

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

hs-v3: Report introduction failure SOCKS ExtendedErrors

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

address: Simplify tor_addr_is_valid()

And rewrite the function comment.

Part of 33679.

Added tests for tor_addr_is_null/valid()

Added tests for tor_addr_is_valid(),
and added tests for tor_addr_is_null(),
which is not modfied.
Ticket 33679

Merge branch 'pr1838_squashed'

Try rewriting normalize_srcdir to normalize harder.

Make check_subsystem_order work in distcheck builds.

Make check_subsystem_order work in out-of-tree builds.

add changes file for 31634

Add a test script to check subsystem order as part of make check.

channel: Fix a comment typo

channel: Remove a newline at the start of the file

Merge branch 'maint-0.4.3'

doc: Fix another man page typo

The default value of the option is "auto", not "default".

Merge branch 'maint-0.4.3'

doc: Fix a typo in the man page

"proceeding" means starting an action.
"preceding" means the thing before this thing.

In this context, it's a bit ambiguous.

core/or: Fix a comment typo in onion.h

Comment-only change.

Merge branch 'maint-0.4.3'

changes: file for ticket 33782

Avoid conflicts between the fake sockets in tor's unit tests, and real
file descriptors. Resolves issues running unit tests with GitHub
Actions, where the process that embeds or launches the tests has
already opened a large number of file descriptors.

Fixes bug 33782; bugfix on 0.2.8.1-alpha.

Found and fixed by Putta Khunchalee.

doc: Also stop running practracker in hooks

When making a new maint branch, stop running practracker in hooks.

doc: Minor restructure for Release Lifecycle

* make end and beginning of life into second-level sections
* write an intro for beginning
* tweak intro for end

Re-apply: doc: Replace "underline" with "## Section name".

Re-applying changes after merge.

Merge remote-tracking branch 'tor-github/pr/1739'

Ignored conflicting style changes: they will be re-applied in
the next commit.

Change starting file descriptor for tests.

scripts: Ignore editor temporary files

Ignore editor temporary files when autostyling in:
* rectify_include_paths.py
* rename_c_identifier.py

Obviously correct changes to already-reviewed code.

Merge branch 'maint-0.4.3'

doc: Rewrite the approved-routers man page entry

Obviously correct documentation changes.

Merge branch 'tor-github/pr/1851'

Merge branch 'maint-0.4.3'

configure: Fix enabled module variable expansion

Fixes #33646

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

hs-v3: Move to log notice the registration of an OB instance

This is to allow a visual feedback in the logs for operators setting up Onion
Balance so they can confirm they properly configured the instances.

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

Merge branch 'tor-github/pr/1844'

hs-v3: Move ob_subcreds to hs_service_state_t.

It's more natural there since it's runtime state.

hs-v3: Don't crash after SIGHUP in Onionbalance backend mode.

The ob_subcreds array was not copied after SIGHUP, and that left the
post-SIGHUP service with a NULL ob_subcreds pointer (until the next descriptor
gets build where we regenerate ob_subcreds in hs_ob_refresh_keys()).

Fixes bug #33762; not in any released tor version.

Merge branch 'maint-0.4.3'

hs-v3: Relax severity of a log message when decoding descriptors.

Make it LOG_PROTOCOL_WARN and also add the expiration timestamp in there to
ease debugging in the future.

changes: Add file for ticket 33014

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

Merge branch 'tor-github/pr/1775'

Use a macro to make our hs_opts checking terser.

Use SECTION_HEADER, not "HiddenServiceDir".

Add a nonfatal assertion about a branch that should be unreachable.

Update expected log messages in tests to new format.

Derive hidden service configuration from hs_opts_t.

This simplifies our parsing code by about 150 lines, and makes the
functions more straightforward.

Start using confmgt logic to parse HS configuration.

This patch doesn't actually use the results of the parsed object to
create the service configuration: subsequent patches will do
that. This patch just introduces the necessary configuration tables
and starts using them to validate the configuration.

As of this writing, this patch breaks tests.  I'll likely fix that
in a rebase later on: the current error messages for failures to
parse options are a regression, so I've opened #33640 for that.

Use config_lines_partition() to parse hs config sections.

Put *.a wildcard entry into .gitignore instead of listing all .a files

Merge remote-tracking branch 'tor-github/pr/1832/head'

fix typo in struct addr_policy_t comment

correct tor_addr_family() function comment

kind of a losing game to remember to update that comment when
we add another possible family, but hey, maybe it will be a while
until we add one more.

spell consensus more right in comments

fix typo in log message

Teach --check-subsystem-order to take input from stdin

Extend includes.py to compare topology with subsystem init order.

Document the --dbg- prefix in tor.1

Add --dbg-dump-subsystem-list command to list the subsystems.

I'm prefixing this with --dbg-* because it is not meant to be used
externally.

doc: Remove TODO secion in README.1st.md

It seems like this section is not valid anymore.
Most of this topics are already explained in different
files.

Implement a function to list the subsystems to stdout.

Add a SUBSYS_DECLARE_LOCATION() to every subsystem.

Add a "location" field for subsystems to declare which file they are in.

doc: Replace "underline" with "## Section name".

This approach doesn't require variable-length underlines.

doc: Remove first, empty line in the file.

doc: Updates paths to the top of the source tree.

Example:
.../doc/HACKING is now doc/HACKING

Merge remote-tracking branch 'tor-github/pr/1780/head'

Merge branch 'maint-0.4.3'

Merge branch 'tor-github/pr/1794' into maint-0.4.3

Merge branch 'maint-0.4.3'

Merge branch 'maint-0.4.2' into maint-0.4.3

Merge branch 'maint-0.4.1' into maint-0.4.2

Merge branch 'bug33651'

Merge branch 'tor-github/pr/1789'

Merge branch 'maint-0.4.3'

Merge branch 'tor-github/pr/1788' into maint-0.4.3

Edits requested by teor

Merge remote-tracking branch 'tor-github/pr/1797/head'

Update README files with common style.

- external links use markdown's []() format,
- local files are formated with backquotes,
- duplicated new lines were removed,
- unnecesary single spaces were removed,
- broken links and local paths were updated.

practracker: Accept changes

Accept a smaller file and function.
Let router.c grow slightly without warning.
Accept an extra comment line.

Cleanup after 32588.

Add some comments about future relay IPv6 work

Comments about 33681.

parseconf: Add ORPort and DirPort auto tests

These tests don't actually trigger bug 32588, but they do increase
the coverage of the auto port config code.

Tests for 32588.

Merge branch 'bug32588_043' into bug32588_master

Merge branch 'bug32588_042' into bug32588_043

Merge static function declaration deletions from bug32588_042 and
maint-0.4.3 in app/config/config.c.

Merge branch 'bug32588_041' into bug32588_042

Merge branch 'bug32588_035' into bug32588_041

Merge tests from maint-0.4.1 with new tests from bug32588_035
in test_router.c.

Add a test for the localhost case.

Extend test to handle router_get_advertised_ipv6_or_ap

Add a test for router_get_advertised_or_port_by_af().

Add tests for get_first_advertised_{addr,port}_by_type_af()

router: Refactor IPv6 ORPort function logic

Return early when there is no suitable IPv6 ORPort.
Show the address and port on error, using a convenience function.

Code simplification and refactoring.

Cleanup after 32588.

router: Stop advertising incorrect auto IPv6 ORPorts

When IPv6 ORPorts are set to "auto", tor relays and bridges would
advertise an incorrect port in their descriptor.

This may be a low-severity memory safety issue, because the published
port number may be derived from uninitialised or out-of-bounds memory
reads.

Fixes bug 32588; bugfix on 0.2.3.9-alpha.

practracker: Accept larger dirvote file

Due to improved function comments.

relay: Set some output arguments in stubs

Fixes part of bug 33674; not in any released version of tor.

Merge branch 'maint-0.4.3'

Merge remote-tracking branch 'tor-github/pr/1824/head'

Merge remote-tracking branch 'tor-github/pr/1823/head' into maint-0.4.3

Merge branch 'maint-0.4.3'

relay/dirauth: Set some output arguments in stubs

And document how some functions set their output arguments.

Fixes bug 33674; bugfix on 0.4.3.1-alpha.

Set *have_low_ports_out from stub port_parse_ports_relay().

Previously we just ignored this option, which would leave it unset,
and cause an assertion failure later on when running with the User
option.

Fixes bug 33668; bugfix on 0.4.3.1-alpha.

Merge branch 'ticket33678_043' into ticket33678_master

Don't take the practracker disable changes from 0.4.3.

practracker: Disable practracker in git hooks

Disable our coding standards best practices tracker in our git hooks.

0.4.3 branches only.

Closes ticket 33678.

Merge branch 'maint-0.4.3'

dirauth: Remove a duplicate macro definition

Obviously correct changes to already-reviewed code.