m_spanningtree Update comments around collision handling

m_spanningtree Let the NICK handler change the nick of the incoming user on collision

Don't change it in DoCollision()

m_spanningtree Update nick overrule on UID code to make sense on master

m_ssl_openssl Read the DH params file using the BIO API on all OSes

Merge insp20

Release v2.0.17

Update example configuration files, fix typos and case

Mention m_regex_stdlib in the list of regex providers after m_filter

m_spanningtree Always treat the remote client as the winner on nick collision if the nick is used by a (local) client that isn't fully registered

Make sure the DN strings obtained from the SSL mods are always valid

Update aliases/anope.conf.example

Update Windows build scripts for the recent build sys changes

m_spanningtree Remove the MAP ISUPPORT token

No other servers send this token and sending it has little to no benefit

m_hideoper: Hide hidden opers from /stats P. #874

Say "SSL certificate fingerprint" instead of "SSL fingerprint" everywhere

Access local user list via new UserManager::GetLocalUsers() and make local_users private

Move and rename typedef LocalUserList to UserManager::LocalList

Erase local users from UserManager::local_list in QuitUser()

core_stats Deduplicate stats L and l code

Move typedef OperIndex to ServerConfig::OperIndex

Move typedef ClassVector to ServerConfig::ClassVector

core_stats Simplify uptime stats code, don't output years because it is ambiguous

Expand ISupportManager documentation

Minor ISupportManager changes

- Make GetLines() a const method
- Rename Lines to cachedlines
- Get rid of a variable in Build()

Move ISupportManager class declaration to isupportmanager.h from inspircd.h

m_rmode Call Channel::GetUsers() instead of directly accessing Channel::userlist

Rename UserChanList to User::ChanList, remove UCListIter

Remove typedef UserMembIter, use Channel::MemberMap::iterator instead

Remove the now unused UserMembList typedef too

Remove typedef UserMembCIter, use Channel::MemberMap::const_iterator instead

Rename UserMembList to Channel::MemberMap, switch all code to use it

Change return type of Channel::GetUsers() to reference from pointer as it is never NULL

core_hostname_lookup Change source of log messages to MODNAME

core_dns Change source of log messages to MODNAME, remove "Resolver: " from all messages

Move the TimerMap typedef into the private part of TimerManager

Remove current time parameter of the Timer constructor

Move the definition of Timer::Timer() into the source file from the header

Use initialization list

core_whowas Rename and move WhoWasGroup to WhoWas::Entry

core_whowas Return a WhoWas::Manager::Stats struct from GetStats() instead of a string

core_whowas Split database logic into a WhoWas::Manager class

core_whowas Clean up GetStats()

core_whowas Rename misleading variables and typedefs

core_whowas Switch from map to a hash map and from irc::string to std::string

core_whowas Change the FIFO to be an intrusive list

core_whowas Store the nickname in WhoWas::Nick

core_whowas Store time added in WhoWas::Nick

core_whowas Delete elements of the per nick deque in the destructor of WhoWas::Nick

core_whowas Create class WhoWas::Nick, store pointers to those in the map

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