hs-v2: Turn logs into protocol warning

All of those can be triggered remotely so change them to protocol warnings.

Fixes #32706

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

copy 0.4.1.8 and 0.4.2.6 entries to master ChangeLog, ReleaseNotes.

Merge branch 'bug32673_035'

Merge branch 'bug33093_logging_035'

Merge branch 'maint-0.4.2'

Use "-s ours" to avoid version bump.

Bump version to 0.4.2.6

Merge branch 'maint-0.4.1' into maint-0.4.2

"Ours" merge to avoid version bump.

bump to 0.4.1.8

Merge branch 'maint-0.4.2'

Merge branch 'maint-0.4.1' into maint-0.4.2

Merge branch 'maint-0.4.0' into maint-0.4.1

Merge branch 'maint-0.3.5' into maint-0.4.0

Merge remote-tracking branch 'tor-github/pr/1634' into maint-0.4.1

Merge remote-tracking branch 'tor-github/pr/1614' into maint-0.3.5

buf_read_from_tls: Return ERROR_MISC, not WANTWRITE, on BUG().

Fixes bug 32673; bugfix on 0.3.0.4-alpha.  We introduced these
checks in ee5471f9aab5526 to help diagnose 21369, but we used "-1"
when "TOR_TLS_ERROR_MISC" would have been correct.  Found by opara.

I don't think that this is actually getting triggered in the wild,
but if it were, it could cause nasty behavior: spurious
WANTREAD/WANTWRITE returns have a way of turning into CPU-eating
busy-loops.

Change BUG() messages in buf_flush_to_tls() to IF_BUG_ONCE()

We introduced these BUG() checks in b0ddaac07428a06 to prevent a
recurrence of bug 23690.  But there's a report of the BUG() message
getting triggered and filling up the disk.  Let's change it to
IF_BUG_ONCE().

Fixes bug 33093; bugfix on 0.3.2.2-alpha.

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

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

Travis: Stop allowing stem test failures

Stop allowing failures on the Travis CI stem tests job. It looks like
all the stem hangs we were seeing are now fixed, but let's make sure we
see them if they happen again.

Closes ticket 33075.

Merge branch 'tor-github/pr/1683'

changes file for ticket 33039

tests for ticket 33039

fix leak in GETCONF

Fix a memory leak introduced by refactoring of control reply
formatting code.  Fixes bug 33039; bugfix on 0.4.3.1-alpha.

Git hooks: remove support for 0.2.9 directory layout.

We no longer will be pushing or committing things in branches that
use src/common or src/or.

Resolves ticket 32957.

update doc/HACKING/ReleasingTor.md

Bump to 0.4.3.1-alpha-dev

Merge branch 'tor-github/pr/1677'

Lower log level of standard error messages from PT's.

This patch lowers the log level of error messages from PT processes from
warning to debug.

See: https://bugs.torproject.org/33005

pick a release date

Add some changelog edits from seborn

draft a changelog blurb for 0.4.3.1-alpha

Add changes file

Run doxygen makefile target on Travis

dir_connection_t: Explain dirconn_direct better

Direct connections can use a DirPort or ORPort.
Indirect connections must use a multi-hop Tor circuit.

Comment-only changes.

Bump to 0.4.3.1-alpha

changelog: small edits

Run format_changelog.py

Initial incorporation of 0.4.3.x changelogs.

Remove changes files that already appeared in 0.4.2.5

Merge branch 'tor-github/pr/1675'

Merge branch 'tor-github/pr/1668'

doc: Minor man page fixes

Fix a client port list that's missing HTTPTunnelPort.

Wrap a long line.

doc: Improve port flags in man page

Explain that:
* most flags default to off
* flags that default to on are specified

Document PreferIPv6Automap in its "No" form, because it's on by
default.

Cleanup after 32637.

config: Make clients tell dual-stack exits they prefer IPv6

Make Tor clients tell dual-stack exits that they prefer IPv6
connections.

This change is equivalent to setting the PreferIPv6 flag on
SOCKSPorts (and most other listener ports). Tor Browser has
been setting this flag for some time, and we want to remove a
client distinguisher at exits.

Also update the man page, putting all the IP flags in their
non-default forms.

Closes ticket 32637.

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

test/nodelist: Remove ROUTERSTATUS_FORMAT_NO_CONSENSUS_METHOD

When we removed obsolete consensus methods in 32695, we also
removed ROUTERSTATUS_FORMAT_NO_CONSENSUS_METHOD, so we need to
remove it from the routerstatus format tests.

Part of 20218.

Merge branch 'ticket20218_rebased_squashed' into ticket20218_merged

* ticket 32695 removed networkstatus_consensus_has_ipv6(),
  keep that change in master.
* ticket 20218 modifies the function name and comment for
  routerstatus_has_visibly_changed(), keep that change
  in ticket20218_rebased_squashed.

Rename test to match new identifier.

split a wide line

Rename routerstatus_has_changed to routerstatus_has_visibly_changed

This is an automated commit, generated by this command:

./scripts/maint/rename_c_identifier.py \
        routerstatus_has_changed routerstatus_has_visibly_changed

It was generated with --no-verify, since it introduces a wide line.
I'll fix it in a subsequent commit.

Update routerstatus_has_changed tests.

Now they also check whether output of routerstatus_format_entry()
has changed.

Add unit test for routerstatus_has_changed()

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

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

add_c_file: Fix "control line not preceded by a blank line"

Fix dirauth and relay module include.am add_c_file.py
"control line not preceded by a blank line" errors.

Also remove a duplicate ADD_C_FILE: SOURCES in the relay module.

Obviously correct fixes to already-reviewed code.

add_c_file: Replace asserts with exceptions

Closes 32962.

add_c_file: Improve script documentation

Part of 32962.

add_c_file: Improve tor source directory checks

Check that the script isn't in a tor build directory, by looking
for a src/include.am file.

Part of 32962.

Fix a small lint-changes warning.

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

Initialize publish/subscribe code when running as an NT service.

Fixes bug 32778; bugfix on 0.4.1.1-alpha.

Revert "Use tor_api.h entry points in ntmain.c."

This reverts commit dca2629480f143ed8deb71b73b15c7c5ff890571.

Revert "Restore feature where nt-services detect non-"run_tor" modes."

This reverts commit 5c240db0bf7751d74ba438a1ca4ef0d051a53df7.

Merge branch 'ticket32806'

Remove some dead checks

The only code that could set these options to be negative was in the
unit tests.

fix a stray asterisk in a comment

Merge branch 'ticket32487_squashed_and_merged'

Merge branch 'ticket32695_squashed'

Remove functions that checked for pre-ipv6 consensus.

We no longer need or need to test:
  * node_awaiting_ipv6()
  * networkstatus_consensus_has_ipv6().

Remove routerstatus_format_entry() consensus_method argument as unused

Remove support for now-obsolete consensus methods before 28.

Closes ticket 32695.

expose routerstatus_has_changed to unit tests.

Refactor routerstatus_has_changed function

- Check all fields that might change in a routerstatus
- Document the refactoring

Signed-off-by: Victor Nepveu <victor.nepveu@imt-atlantique.net>

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

Merge branch 'ticket32487_squashed' into ticket32487_squashed_and_merged

Resolved conflicts in src/core/include.am

Document why dircache is not included in --list-modules

Add have_module_dircache().

Turn several functions from stubs into macros

This may help the compiler eliminate deadcode.

Rename dirclient_modes.h identifiers to start with dirclient_

This is an automated commit, generated by this command:

./scripts/maint/rename_c_identifier.py \
        directory_must_use_begindir dirclient_must_use_begindir \
        directory_fetches_from_authorities dirclient_fetches_from_authorities \
        directory_fetches_dir_info_early dirclient_fetches_dir_info_early \
        directory_fetches_dir_info_later dirclient_fetches_dir_info_later \
        directory_too_idle_to_fetch_descriptors dirclient_too_idle_to_fetch_descriptors

Move directory_must_use_begindir() to dirclient_modes.[ch]

Move dirclient-related functions out of dirserv, and reenable them

I had incorrectly identified these functions as dircache-only, when
in fact they apply to everyone who acts a directory client.

Add a changes file for disabling the dircache code.

Disable feature/dircache files when dircache module is disabled.

To make Tor still work, we define a minimal dircache_stub.c file
that defines the entry points to the module that can actually be
seen by the compiler when we're building with dircache and relay
disabled.

Use dir_conn_clear_spool() in connection.c.

This is cleaner than iterating over the spool.

Move dirserv_get_routerdescs() to control_getinfo.c

This function had some XXX comments indicating (correctly) that it
was not actually used by the dirserver code, and that only the
controller still used it.

Move dir_split_resource_into_spoolable() to dircache module.

Only directory caches actually need to spool things.

Define a "dircache" module.

For now, this module is enabled whenever the relay module is
enabled, and disabled whenever the relay module is disabled.  Though
they are logically separate, the use cases for running one without
the other are rare enough that we don't really want to support
compiling them independently.

Merge branch 'maint-0.4.2'

Merge branch 'maint-0.4.1' into maint-0.4.2

Merge branch 'maint-0.4.0' into maint-0.4.1

Merge branch 'maint-0.3.5' into maint-0.4.0

Merge remote-tracking branch 'tor-github/pr/1513' into maint-0.3.5

Don't escape the bridge distribution value.

We already check if there are invalid values in
check_bridge_distribution_setting() and reject the value if that is the
case. We can therefore only have strings of [A-Z] | [a-z] | [0-9] | '-'
| '_' here which is according to the directory specification.

See: https://bugs.torproject.org/32753

add_c_file: Fix "control line not preceded by a blank line"

Fix dirauth and relay module include.am add_c_file.py
"control line not preceded by a blank line" errors.

Also remove a duplicate ADD_C_FILE: SOURCES in the relay module.

Obviously correct fixes to already-reviewed code.

add_c_file: Rename variables based on the type of path

* fname for generic file paths
* tor_fname for paths relative to the top-level tor directory
* src_fname for paths relative to tor's src directory

With prefixes as required to disambiguate different paths of the same
type.

Part of 32962.

add_c_file: Simplify some usage of srcdir_file()

Part of 32962.

add_c_file: Document the paths used by each part of the script

Most paths are relative to the top-level tor directory, but the
paths in the C and H files are relative to tor's src directory.

Part of 32962.

add_c_file: Improve path handling and canonicalisation

* distinguish between paths relative to the top-level tor directory,
  and paths relative to tor's src directory
* canonicalise paths before using them
* check that the script is run from the top-level tor directory
* check that the file is being created in tor's src directory

Part of 32962.

add_c_file: tolerate ./ in filenames.

add_c_file: handle adding to the end of a list correctly.

Fixes bug 32962.