Merge pull request #308 from edk0/with-asan

Add --with-asan to build with asan

Link asan in a clang-friendly way

Add --with-asan to build with asan

Merge pull request #304 from edk0/reject-expired

reject: Don't reject for expired K-lines

reject: reorder to avoid leaks

reject: Don't reject for expired K-lines

Merge pull request #302 from edk0/sasl-usercloak

Import extensions/sasl_usercloak from ircd-seven

Merge pull request #297 from bmwiedemann/date

Set EXTERNAL_BUILD_TIMESTAMP from SOURCE_DATE_EPOCH

Merge pull request #303 from edk0/modreload-uaf

modules: fix use-after-free when reloading

Merge pull request #301 from edk0/reject-free-fix-fix

Remove from the list of propagated bans on expiry

modules: fix use-after-free when reloading

sasl_usercloak: fix typo in format string

thanks @ProgVal

sasl_usercloak: make the magic string more specific

Require '/account' at the end of the spoof, rather than
'account' anywhere.

sasl_usercloak: modernize

Remove duplicated notify_banned_client

sasl_usercloak: check K-lines after host change

Revert "recheck users after applying SASL account cloaks"

This reverts commit 4d401d3c60019cf96b07a012106cab9678b7a79d.

recheck users after applying SASL account cloaks

(so K-lines on them will take effect properly)

Make sasl_usercloak.so update the original host, as well as the visible host if it hasn't already changed. Allows for the sasl spoof to be used to override services ones in some circumstances.

Be sure to zero out host buffer before using it

Add sasl_usercloak module, to allow injecting SASL account name into a user's host on connection

Remove from the list of propagated bans on expiry

Fixes a use-after-free introduced in 548e31d3b, which fixed a related
use-after-free introduced in a9536f755.

Merge pull request #300 from edk0/reject-free-fix

Fix a use-after-free introduced in #298

Fix a place aconfs could be freed while referenced

(not a bug until a9536f755 since bans were not referenced before)

Merge pull request #299 from edk0/tkline-reason

Add config option to hide durations of temporary K/D-lines

Merge pull request #298 from edk0/rejectcache

Remember and send reasons for rejectcache rejections

Add hide_tkdline_duration to documentation .confs

Add general::hide_tkdline_duration

reject: Remember and send reasons for rejections

rejectcache entries can now use either a K-line aconf or a static
string as a reason. This will be sent in a 465 numeric before the usual
ERROR. In the case of K-lines, it resembles the 465 you would have been
sent without being rejected:

; nc -s 127.6.6.6 127.0.0.1 5000
:staberinde.local 465 * :You are banned from this server- Temporary
    K-line 4320 min. - abc123 (2019/12/31 01.07)
ERROR :Closing Link: (*** Banned (cache))
; nc -s 127.128.0.0 127.0.0.1 5000
:staberinde.local 465 * :You are not authorised to use this server.
ERROR :Closing Link: (*** Banned (cache))

Set EXTERNAL_BUILD_TIMESTAMP from SOURCE_DATE_EPOCH

to make the package build reproducible by default without
everyone having to discover the custom variable.

See https://reproducible-builds.org/ for why this is good
and https://reproducible-builds.org/specs/source-date-epoch/
for the definition of this variable.

This code assigns the plain integer to keep the code simple.
Otherwise we would have to deal with differences between GNU date
and BSD date or include extra build deps like perl or python.

Merge pull request #296 from edk0/modreload

m_modules: make modreload work like restart

m_modules: make modreload work like restart

/modrestart used to be implemented as a normal command and could crash
when used remotely because it would reload m_encap, which was on the
call stack at the time. This was fixed in 41390bfe5f. However,
/modreload has exactly the same problem, so I'm giving it the
same treatment.

Incidentally: This bug was first discovered in ircd-seven, where the
`/mod*` commands themselves live in the core, so m_encap was the only way
the crash could happen (and it didn't most of the time, because m_encap
would only be moved if you got unlucky). But `/mod*` are in modules in
charybdis, so /modrestart would have unloaded the code it was in the
middle of executing. With that in mind, I'm not sure how it ever
appeared to work.

Merge pull request #287 from edk0/filter

Add extensions/filter (port from ircd-seven)

filter: avoid a memory leak per @amdj

Merge pull request #293 from edk0/webirc

m_webirc: improve TLS handling

Merge pull request #294 from edk0/deny-webirc-auth

m_webirc: deny using webirc. as a real auth block

m_webirc: deny using webirc. as a real auth block

m_webirc: it's "TLS" to you

m_webirc: don't bail out when denying 'secure'

m_webirc: const-correctness

m_webirc: respect ircv3's `secure` option

m_webirc: enforce need_ssl

tests/Makefile.am: check-local: don't depend on extensions/*

This breaks if a file in extensions/ doesn't get built due to
conditional compilation, and the tests don't need anything
under extensions/ anyway.

Merge pull request #286 from edk0/strip-unprintable

strip_unprintable: Don't strip all bytes with the MSB set

strip_unprintable: clarify type conversion

This type conversion is more easily shown to work as intended than the
conversion introduced in 9dcb9e169.

Fix strip_unprintable on high bit bytes

Merge pull request #290 from edk0/quiet-override

override: don't spam about (un)setting the mode

Merge pull request #289 from edk0/rehash-privileged-modes

Recheck umodes for opers after rehash

override: don't spam about (un)setting the mode

We will spam about any uses of it, so this just seems like extra noise.

Recheck umodes for opers after rehash

filter: explicit type conversion

Add hyperscan to CI deps

filter: correct type for modinit

Fix build when hyperscan is missing

filter: use AV2

filter: add DROP and ABORT

filter: require an extra "+" before data

This disambiguates it from control commands

filter: add modes to disable incoming filters

filter: make the kill reason more honest

filter: send the failure response only on PRIVMSG

filter: match in two passes, before and after stripping

filter: move to extensions

filter: bump version

filter: use a static buffer instead of strdup

filter: add version number

filter: send a numeric error for ACT_DROP

filter: add #defines for nick,user,host

filter: don't give the target of PMs to filters

filter: include identified marker

filter: drop messages if we're ACT_KILLing them

filter: Filter only locally-sourced messages

(Assume the remote server filtered remote ones.)

filter: kill last or we can't send the snote

Add filter.c

Depend on hyperscan

Travis CI: Bump to Ubuntu Bionic 18.04 LTS

Also bump Linux compilers from (GCC 4.8, GCC 4.9, GCC 5, GCC 7,
GCC 8, Clang 3.9, Clang 4.0, Clang 5.0, Clang 6.0) to (GCC 7,
GCC 8, Clang 7, Clang 8). There's no need to test against every
ancient compiler under the sun. Furthermore, we no longer need
an apt sources list for Bionic.

[ci skip] (Tested already <https://travis-ci.org/charybdis-ircd/charybdis/builds/594225622>)

Merge pull request #285 from edk0/ratelimits

Remove ratelimits on /stats and /motd when they don't have a server parameter

m_motd: don't ratelimit with no server argument

m_stats: don't ratelimit with no server argument

Don't enable warnings on osx

librb: Fix nossl

librb: Fix nossl

travis: Use newer versions of Clang

Fix Clang 6 compiler warnings

travis: Remove -Wno-unused-variable

librb: Avoid unused variable warning

librb: Specify -Wno-unused-parameter

travis: Don't use -Werror on clang

travis: Notify on IRC

Fix GCC 8 compiler warnings

travis: Ignore more unused warnings

ircd: Use a larger buffer for ilog() buf2

travis: Ignore unused-parameter warnings

librb: Fix GCC 8 warning; 640 bytes should be enough

ircd: Zero out the global_client_list

Otherwise we unconditionally add "me" to it twice in some unit tests,
which results in a loop in the list.

travis: Build with GCC 7 and GCC 8

Fix compiler warning for SeesOper

travis: Compile with warnings as errors

Merge pull request #283 from edk0/grant

m_grant improvements

Add help for GRANT

m_grant: match "deoper" case-insensitively