Add intrusive_list_tail container that maintains a pointer to the last element

Make the name of intrusive list class configurable using a define

Split intrusive list implementation into a separate file

core_whowas Don't display used bytes in /STATS z

Calculating this is tricky and the calculation was not realistic

m_spanningtree Extract logic that finds the source user for an incoming command into a function

m_spanningtree Fix fake direction detection for commands

m_spanningtree Add optional expected nick TS parameter to SVSNICK

Drop SVSNICK if the expected nick TS is present and it does not match the user's nick timestamp

m_spanningtree Replace manual string building of outgoing commands with CmdBuilder where possible

User::ChangeNick() cannot fail if the new nick is the uuid, document this and rely on it

Get rid of irc::strings (assign()) in User::ChangeNick()

Deduplicate code in User::ChangeNick()

Remove now needless User::ForceNickChange()

Change call sites to call ChangeNick()

Move calling the OnUserPreNick() hook and the restrictbannedusers check from core to cmd_nick (core_user)

core_user Deduplicate code that calls the OnUserRegister hook

core_user Inherit CommandNick from SplitCommand, only handle it for local users

Set SSL options to sane default and enfore server cipher preferences

Original PR #856

m_spanningtree Drop support for changing user modes with FMODE

This is useless with uuids

m_spanningtree Translate user mode changes done via FMODE by 1202-protocol servers to MODE

m_spanningtree Accept legacy JOINs from 1202-protocol servers

2.0 accepted JOIN and some pseudoservers might use it

Merge pull request #878 from SaberUK/insp20+fix-minor-issues

Fix various small problems with configure.

Use TimingSafeCompare() to compare passwords and password hashes (non-hmac only)

Issue #882

Add InspIRCd::TimingSafeCompare() function that compares strings in a timing-safe way

m_spanningtree Send the cert fingerprint message to opers only after successful auth

Check fingerprint before checking password (server linking)

Checking the password before the fingerprint means that even without the
correct cert it's possible to brute force the password or leak
information about it. Checking the fingerprint means attackers must
forge the cert before they can learn any information about the password.

Add man pages.

Add systemd service file.

Fix warning from gnutls-certtool about --bits being deprecated.

This patch has made me extremely grateful for git-blame as the
GnuTLS developers apparently do not document when deprecation
warnings are introduced.

Ignore safe compiler warnings in a more reliable way.

Fix various small problems with configure.

- Fix temporary files not being deleted after use.
- Fix the option for disabling rpath.
- Fix the help message:
  * Fix indentation of some help options.
  * Fix default paths not containing 'run'.
  * Remove documentation for --disable-ipv6 (exists but does nothing).
  * Remove documentation for --library-dir (does not exist).
- Fix some minor errors in messages.

Merge pull request #879 from SaberUK/insp20+fix-denychans

Fix copy/paste error in m_denychans.

Add parameter to InspIRCd::TimeString for UTC time formats.

Missing doc added by @attilamolnar

Add formatting to InspIRCd::TimeString; switch all code to use it.

m_httpd also now uses the correct timestamp format.

Windows-specific fixes by @attilamolnar, original PR #849

Fix copy/paste error in m_denychans.

Inherit ThreadQueueData from Mutex to avoid duplicating code

Remove empty Thread destructor

threadengine_win32 Don't ask for the thread id of newly created threads pointlessly

Move and rename class ThreadData to ThreadEngine::ThreadState

Move and rename ThreadData::FreeThread() to ThreadEngine::Stop() and document what it does

Make Thread::state private

Change allocation of ThreadData to be physically part of the object containing it

Remove empty ThreadEngine constructors and destructors

Remove the unused ThreadEngine::GetName() method from both implementations

Change allocation of InspIRCd::Users to be physically part of the object containing it using fakederef

Change allocation of InspIRCd::SNO to be physically part of the object containing it using fakederef

Remove now needless dynref_init_complete

ModuleManager is constructed when ServerInstance is non-NULL, test that instead

Change allocation of InspIRCd::Modules to be physically part of the object containing it using fakederef

Change allocation of InspIRCd::Logs to be physically part of the object containing it using fakederef

Change allocation of InspIRCd::Modes to be physically part of the object containing it using fakederef

Add the fakederef class

Change all occurrences of plain sort() to std::sort()

core_userhost Show real host if the target is the same as the user doing the /USERHOST

core_userhost Append data to the output in a saner way

core_userhost Do the HasPrivPermission() check only once, not once per nick

core_ison Use iterators

core_ison Extract duplicated code into a function

Change append(" ") to push_back(' ')

core_ison Truncate string after sending it on overflow instead of recreating it

core_ison Don't deduplicate nicks

Deduplication makes us do pointless processing for every ISON and offers
no benefit at all to proper clients that send a nick only once

core_ison Remove check that is always true

core_list Improve readability by assigning the Channel being inspected to a variable

core_list Check whether the chan name/topic has to be Match()ed once, not once per chan

core_list Do the HasPrivPermission() check only once, not once per chan

Check Q-Lines on nick change in core_xline

Reject nickname case changes too when enforcing <security:restrictbannedusers>

Change the type of the user parameter in the OnUserPreNick() hook from User to LocalUser

No remote users were passed to this hook before.

Remove needless IS_LOCAL() checks.

Increment serverstats::Collisions when a collision is handled, not when a module denies a nick change

m_chghost Allow ulined servers to change the host of unregistered users (for SASL)

m_spanningtree Share server description updates via SINFO desc

Update the description field in the Server object representing the local server on rehash

m_spanningtree Remove server-to-server VERSION handler

m_spanningtree Translate SINFO version to VERSION and vice versa for 1202 protocol servers

m_spanningtree Exchange public server version strings via SINFO instead of VERSION

m_spanningtree Show the full version string of servers to opers

Issue #637, suggested by @CuleX

m_spanningtree Send and receive full version strings via SINFO

m_spanningtree Add a field to TreeServer for storing full version strings

m_spanningtree Add server-to-server SINFO command handler and builder

Don't send SINFO to 1202 protocol servers

m_spanningtree Add TreeSocket::SendServerInfo() that sends all additional data about a server

Merge pull request #872 from theckman/remove_whitespace_from_docs

Remove whitespace from docs

remove unneeded whitespace from the docs/examples

Kill needless #includes in source files

Remove unused #define ERROR -1

m_spanningtree Remove declaration for nonexistent function: OnLine()

m_spanningtree Use vector back() where it makes sense

m_spanningtree Initialize boolean member variables in the constructor of the module

m_spanningtree Initialize TreeServer::UserCount for TreeRoot in its constructor

Make the default ProtocolInterface instance part of class InspIRCd

The protocol module no longer has to delete it on load and construct it on
unload, only assign a new value to ServerInstance->PI

m_spanningtree Change allocation of the specialized ProtocolInterface to be physically part of ModuleSpanningTree

Change allocation of InspIRCd::Parser to be physically part of the object containing it

Remove global Commandtable typedef

Make CommandParser::cmdlist private

Add CommandParser::GetCommands() and typedef CommandMap and use it instead of directly accessing cmdlist

Use CommandParser::GetHandler() in ServerConfig::ApplyDisabledCommands()

Change allocation of InspIRCd::BanCache to be physically part of the object containing it

Pull in bancache.h from inspircd.h

Move the definition of the BanCacheHit constructor into the source file from the header

Move BanCacheHash typedef into the private part of BanCacheManager

Change allocation of BanCacheManager::BanHash to be physically part of the object containing it

Change allocation of InspIRCd::Threads to be physically part of the object containing it

Change allocation of InspIRCd::stats to be physically part of the object containing it

While at it, remove "stats" from all field names

m_spanningtree Explicitly specify the routing for RESYNC as the one inherited from ServerCommand doesn't suit it

Command::GetRouting() defaulted to local only when RESYNC was added, but
ServerCommand overrides it.

m_spanningtree Strip membership id from KICKs sent to 1202 protocol servers