libathemecore/function.c: sendemail_urlencode: cast argument to isalnum(3)

This may avoid diagnostics on systems with signed chars. Not actually a
problem in practice because we do not consider such characters to be
valid in a nickname, which is required for them to be valid account
names.

cf. https://github.com/atheme/atheme/commit/fcc4e19a573001263245#r53314237

libathemecore/function.c: sendemail(): allow for urlencoding account

If one wishes to replace their register email template to prefix the account
name and registration verification token with a URL (to implement a webserver
to verify registrations, so people don't have to copy and paste commands to
execute on IRC), we should allow for the possibility that people will have
non-alphanumeric characters in their account name, and provide a URL-encoded
version of it.

This will ensure that such links are always clickable in various MUAs.

Merge pull request #793 from ProgVal/patch-1

INSTALL: Remove unnessarily agressive remark

Merge pull request #794 from ProgVal/patch-2

Explain configuration outside the "Compiling" section

Merge pull request #803 from ValwareIRC/master

gib de topic reject reason more verbose explanation

Merge pull request #807 from spb/hs-take-single

Allow HS TAKE without params if there's only one HS OFFER (#2)

Merge pull request #790 from edk0/confirm-by-emailchg

Make VERIFY EMAILCHG confirm the account if necessary

strncpy->mowgli_strlcpy, fix buffer size

Allow services-generated passwords length to be dynamically configured

Closes #800

Make a local copy before modifying the host string, as we might be using the copy from the offer db instead of a command parameter

Allow HS TAKE without params if there's only one HS OFFER

Update modules/chanserv/topic.c

Please change color to colour to match the rest of the language as it uses British English.

Co-authored-by: Data <redacted>

update a bit more ♥♥

Update function.c

Give correct information when declining a topic because of control chars.

Explain configuration outside the "Compiling" section

It didn't really make sense as configuration isn't part of the compilation.

Additionally, not everyone installs Atheme by compiling it themselves,
or they may not have the file at hand while reading this document
(eg. reading the documentation before actually doing it)

INSTALL: Remove unnessarily agressive remark

People who set up their first IRC network will read this documentation,
it seems unnecessary to shame them for... not knowing how to setup
and IRC network.

Merge pull request #792 from mquin/patch-1

Remove reference to SET FOUNDER in help for DROP

Remove reference to SET FOUNDER in help for DROP

.gitattributes: remove files that no longer exist

email/default/setpass: don't encourage contact for unsolicited receipt

Nothing can be done to the account without the token in this e-mail,
and any competent malicious actor who does have unauthorised access to
a user's e-mail inbox is also just going to delete the e-mail after
compromising the user's account anyway.

Therefore, it doesn't make sense to encourage the user to reach out to
the network administration.

Also clarify that password reset tokens are now automatically
invalidated by a successful login.

libathemecore/services.c: myuser_login(): invalidate password change tokens

libathemecore/ptasks.c: handle_kill(): don't use slog() if we got killed

If a client on our server is killed, we cannot reliably use slog(),
because the client may be OperServ, and slog() mesages are sent from
OperServ.

It turns out that some IRCds care about receiving messages from UIDs
it doesn't know about (because it just killed them), and will send
KILLs for them over and over again.

Reported-By: @A-UNDERSCORE-D

modules/protocol/unreal4: correct module name

Log correct target name for verifies

VERIFY EMAILCHG confirms account too

m4/atheme-libtest-*.m4: correct copy/paste mistake on temporary variable name

Merge pull request #786 from atheme/fix/module_unload

Module unloading use-after-free fixes

modules/operserv/modmanager: account for an entire list of non-reloadables

When recursing a given module's reverse dependencies, it is
possible that we run into a permanent module that is depended
upon by a semi-permanent (reload-only) module. This will result
in the reload of that module being skipped (as there is a
permanent module in its reverse dependency chain), but the
mod_recurse_revdeps() function will still set the r_dep pointer
to the semi-permanent one.

We then test if r_dep is set to perform a database save (in case
reloading of the reload-only module fails). However, if our
entire list of modules to reload has recursive non-reloadability
status, we will be saving the database for nothing, as we then
go on to ignore every module in the list.

Exit early if this is the case, avoiding a pointless wallops and
database save.

operserv/modmanager: MODRELOAD: avoid potential use-after-free

Listing the same module twice in the list of MODRELOAD parameters, or
listing a module that would already be unloaded for depending on a
module given earlier in the list, would attempt to unload a module that
had already been unloaded.

Instead of storing the module pointer when initially processing the
list, we now call module_find_published() again just before unloading
the module. (We do remember the unload capability value computed during
the initial processing to avoid having to recurse again.)

module_unload: avoid potential use-after-free

Given three modules (A, B, C), where:

B depends on A
C depends on B
C depends on A

Module A can end up with this reverse dependency graph (as represented
in the modules' required_by lists):

A -> B -> C
A -> C

Given B is listed first in required_by, it will be unloaded;
recursively, C will be unloaded. However, unloading a module also
removes it from its dependencies' required_by lists, thus removing C
from A's required_by list. While the _SAFE variant of the list iteration
macro is designed to handle the current element being removed, it is not
able to handle an arbitrary number of following elements being removed
as well.

As unloading a module will always remove it from the required_by list,
we can instead keep unloading the first element of the list until the
list is empty.

Update loginnolimit.c

Fixes #789

Update my e-mail address

modules/saslserv/main.c: rename flag for secure client connection

Hooks should be able to judge whether to allow an SASL negotiation
based on whether the client is connected to the network securely or
not; not just whether it is using TLS. Some IRCds have the concept
of "secure origins", and can treat e.g. connections from localhost
as secure against eavesdropping. Rename the flag appropriately.

Merge pull request #782 from atheme/amdj/stats-t-unpriv

libathemecore/ptasks.c: STATS:T: make nonsensitive data available

libathemecore/ptasks.c: STATS:T: make nonsensitive data available

The statistics for server, user, and chan can already be obtained via
the LUSERS command on IRC, so it does not make sense for services to
prevent accessing these by requiring services privileges.

Furthermore, the number of registered accounts, nicknames, and channels
should also not be considered sensitive data. The Atheme project does
not condone hiding such basic and non-sensitive data from network users.

Finally, put missing mynicks count in default MOTD file.

modules/chanserv/access: re-arrange empty check order

Discovered while I was about to merge #784. Didn't want to trip CI again
twice with the fix and merge, so I'll merge and fix it in one shot. The
rest of the pull request establishes the check order as permissions,
closure, and then emptiness.

Merge pull request #784 from phy1729/chanserv-ensure-not-closed

chanserv/{clear,clone}: Use STR_CHANNEL_IS_EMPTY

chanserv/{akick,clear,clone,getkey,info,invite,recover,sync,topic,unban,xop}: Sort checks

Checks go in the order access, closure, empty (if needed).

chanserv/{ban,kick,quiet,set,sync,taxonomy,unban}: Ensure channel is not closed

Merge pull request #780 from mquin/mquin-ns-help-examples

Add examples to NickServ help entries.

modules/nickserv/set_accountname: log when hook denies account name change

modules/saslserv/main: downgrade severity of no mechanism log message

It was discovered while testing some improvements to contrib/sasl_blacklist
that clients can trigger this message remotely and at will, just by not
adhering to the SASL specification.

Add examples to NickServ help entries.
For consistenty with other entries.
Adds examples to set_accountname, set_enforce, set_pubkey, and setpass.

chanserv/flags help: only mention groups if groupserv exists

The Great Migration

help/default/nickserv/drop: fix command help

Add match_masks_through_vhost option for Solanum

This reflects a similar Solanum PR: solanum-ircd/solanum#150

It might be slightly more proper to have this option apply specifically
to the solanum protocol module once we have one, but we don't, plus
there isn't currently any precedent for config options added from
protocol modules.

The option is deliberately rehashable as the Solanum option is as well,
allowing a synchronized change in configuration across a network without
requiring restarts.

Update version in SECURITY.md

libathemecore/object.c: add method for deleting object privatedata

include/atheme/hooktypes.in: add a hook for password (hash) change

This will allow modules to detect when a user's account password
(or its hash) has been changed (after the fact; use the hook added
in the previous commit if you need to access the plaintext password
for some reason).

nickserv/set_password: allow a hook to deny a password change

This prevents bypassing nickserv/pwquality, by initially registering
with a password that it does not object to, and then changing it to
a more insecure one.

libathemecore/connection.c: free vhost_addr after using it.

Caught by Clang's AddressSanitizer, eventually, after a few inexplicable
corrupt addresses were output. I'm not sure why it didn't catch this
immediately.

Fixes: 642759fc2907efe5e0ba ("libathemecore/connection.c: use
                              connection->name to store addrs & ports")

libathemecore/connection.c: fix cut/paste mistake

Fixes: 642759fc2907efe5e0ba ("libathemecore/connection.c: use
                              connection->name to store addrs & ports")

include/atheme/global.h: remove obsolete/unused global maxfd vars

modules/protocol/charybdis: check_forward(): add +oO flags to check

If the user has the *ability* to obtain channel operator status (via
services), consider that as good as the user *having* channel operator
status.

Also document why the existing code is the way it is, and allow checks
against myuser to succeed even if user is present.

Reported-by: grumble <redacted>

libathemecore/connection.c: connection_stats(): don't justify fds

This only ends up making the output look weird.

libathemecore/connection.c: connection_stats(): indicate if listening

libathemecore/connection.c: connection_stats(): indicate the uplink

libathemecore/connection.c: use connection->name to store addrs & ports

Also obtain IP addresses where the sockets are created, instead of in
connection_add(). This is a better approach than that taken by commit
1bb7e1e587306239ca87, and restores the ability to perform non-blocking
connections.

Finally, since connection_add() creates the mowgli.pollable object for
the fd, don't bother testing if the fd is valid. Add an assertion for a
valid fd being passed, because an invalid fd will only result in
nothing working anyway. Likewise assert that the connection name is
valid, and that at least one I/O handler was supplied.

include/atheme/connection.h: add an intrusive connection list node

include/atheme/connection.h: rearrange struct in member size order

include/atheme/connection.h: remove obsolete/unused struct member

libathemecore/connection.c: tidy up all functions

Apply const-correctness, move variables to where they are needed, etc.

include/atheme/connection.h: add a typedef to reduce boilerplate

libathemecore/connection.c: ask getaddrinfo(3) to fill in the port instead

libathemecore/connection.c: refactor how to ignore errno on connect(2)

libathemecore/connection.c: tidy up macro definitions

libathemecore/connection.c: move empty_handler() up and rename it

include/atheme/connection.h: reindent and tidy up CF_ macros

Also add missing CF_IS_*() macros, and use them directly.

contrib/: add script for converting pbkdf2 hashes to pbkdf2v2 hashes

modules/contrib/: transition to independent build system

Having to duplicate configure tests in all supported Atheme release and
development branches, to enable contrib modules to be built reliably on
all of our supported platforms, was quickly becoming untenable.

dist/atheme.conf.example: expand on how to use pseudoservice::access{}

dist/atheme.conf.example: upgrade URI scheme for commented helpurl

modules/operserv/{readonly,set_commitinterval}: change fault code

cf. https://github.com/atheme/atheme/pull/767#pullrequestreview-600277627

.gitattributes: update for commit 8c3926d73eb23f567a25

m4/atheme-libtest-mowgli.m4: add a usability test if using external lib

When we are not using the internal libmowgli submodule, do a link test
to see if the library is usable, just like we do with the other library
tests.

This one is fairly extensive, as this codebase makes heavy use of this
library.

Build System: Improve handling of enabling and configuring submodules

- When performing the libmowgli test, don't pollute CFLAGS / CPPFLAGS /
  LDFLAGS / LIBS variables. Set library-specific variables, just like
  the other library tests do

  This allows us to configure submodules later without having to save
  the environment variables first

- When --with-libmowgli=yes is given, require that pkg-config is
  available and that it can detect the library, erroring out otherwise

- Put the logic for handling enabling and configuring submodules into
  its own M4 file

modules/Makefile: build module subdirectories alphabetically

extra.mk.in: Tidy up and document some groups of variables

m4/atheme-libtest-ldap.m4: prefix library variables with LIB

This more closely aligns it with the other library tests.

configure.ac: move digest/RNG API tests to immediately after libraries

The library tests set the variables that these use, so it makes sense
that they appear immediately after.

configure.ac: move warning about pkg-config being absent

This belongs with where it is tested for. Also break it up a
bit to make it easier to read.

configure.ac: rename AC_CONFIG_HEADER to AC_CONFIG_HEADERS

The former was never documented anywhere and is now obsolete.

configure.ac: move AC_CONFIG_FILES to the top

This is an autoconf configuration instantiation, just like all of the
other macros around it.

configure.ac: rename autoconf/ to build-aux/

This is the name that a lot of other projects use, the name that is
recommended by the GNU autoconf documentation, and the name that is
used by the upstream buildsys project [1] (though we have diverged
from that significantly).

[1] https://fossil.nil.im/buildsys/timeline

configure: put directory macros in a header file, not in CPPFLAGS

This will be necessary for my near-future intention to change the contrib
modules repository to be self-building; i.e. to have its own configure
script and build system.

Also don't try to expand directories like MODDIR for pretty printing,
because it sometimes doesn't work depending on the directory arguments
given to ./configure. Yeah, the config output looks worse, but oh well.

m4/atheme-api-random-frontend.m4: detect more secure arc4random(3)s

It turns out that a lot more than recent OpenBSD systems have a
secure algorithm backing their arc4random(3) implementation.

Remove the check for just OpenBSD, and add checks for:

- Mac OS 10.12+
- FreeBSD 12.0+
- NetBSD 7.0+
- OpenBSD 5.4+

Created with input from rqsd from IRC, in response to commit
8ff53e4d2a843c5fdd7a.

Note that this is not a statement that Atheme will successfully
compile & run on the above platforms.

m4/atheme-libtest-mowgli.m4: warn when pkg-config is unavailable

m4/atheme-*.m4: use autoconf flow control macros exclusively

The AS_IF and AS_CASE macros enable one to write "if" and "case"
shell statements in a portable way. They still generate more or
less the same output, but code which works better on various
obscure platforms.

They also allow autoconf to examine the conditional block bodies
for macro calls, to ensure those macros are available. This will
become more necessary in future versions of autoconf.

Also use these to replace an unguarded grep(1) invocation in the
Perl library testing macro.

libathemecore/: rename openbsd random backend to arc4random

There may well be platforms in the future that have a secure algorithm
backing their arc4random(3) implementation. For the moment we continue
to support only OpenBSD, but make it easier to adjust in the future.

configure.ac: remove checks for programs that aren't used

modules/**/Makefile: remove PLUGIN_CFLAGS & PLUGIN_LDFLAGS

The buildsys.mk.in and buildsys.module.mk files already add these
variables to CFLAGS and LDFLAGS during execution of CompileModule,
CompilePlugin, and Link steps.

libathemecore/Makefile: remove LIB_CFLAGS from CFLAGS

The buildsys.mk.in file already adds this variable to CFLAGS during
execution of CompileLib steps.

configure.ac: approach noexecstack differently

This can also be passed as -Wl,-z,noexecstack which saves Clang warning
about it being unused during compilation. Since -Wl arguments are only
used by the linker, take it out of CFLAGS. This continues to work on
GCC.

configure.ac: prepend LIBPCRE_CFLAGS and LIBPCRE_LIBS

The entire codebase depends on libpcre if it is detected successfully,
so we need to build all of the .c files with the appropriate CFLAGS &
LIBS.

This was taken care of in libathemecore/Makefile, but nowhere else.
It's better to just have configure.ac take care of adding it if they
are non-empty.

modules/saslserv/ecdh-x25519-challenge: fix ifdef soup

m4/atheme-featuretest-contrib.m4: fix test for res_query(3)

Most platforms have these as enums, which are converted to int (for use
as arguments) by the compiler automatically. However, some platforms do
not have these as enums, and use macros to define their values instead.

The former platforms also have those macros to define them in terms of
the corresponding enum, so use the macros instead of the enums for
broader compatibility, defining them in terms of their enums when they
don't exist.

glibc2 and musl have enums and compatibility macros, uclibc and
uclibc-ng have enums only, and OpenBSD libc has macros only.

doc/VERSION_LETTERS: sync with reality