m_ssl_gnutls Refcount GnuTLS objects, free them when they are no longer in use instead of at /rehash ssl time

m_ssl_openssl Fix debug message

m_ssl_openssl Reset data_to_write for new sessions

m_ssl_openssl Free the ssl_cert object as soon as the session is closed instead of waiting for the next VerifyCertificate() or new connection

m_ssl_openssl Remove bogus errno assignment from CloseSession()

Merge pull request #935 from SaberUK/insp20+fix-crlf

Add a workaround for compiling modules with Windows line endings.

m_ssl_openssl Return an error from the IOHook read and write functions if the handshake returns 0

The meaning of a 0 return value quoted from the manual:
The TLS/SSL handshake was not successful but was shut down controlled and by the specifications of the TLS/SSL protocol.

m_ssl_openssl Add user-friendly config options for setting a few OpenSSL context options

Add a workaround for compiling modules with Windows line endings.

m_nationalchars Ask m_spanningtree to rebuild its hashmaps when the national_case_insensitive_map changes

Issue #923

m_spanningtree Rebuild serverlist and sidlist when receiving a specific Request

Issue #923

m_nationalchars Rebuild the hash map in m_watch when the national_case_insensitive_map changes

Issue #923

m_nationalchars Rebuild core hashmaps when the national_case_insensitive_map changes

Issue #923

Merge pull request #934 from SaberUK/insp20+fix-various-issues

Fix various issues detected by Coverity.

Kill some logically dead code detected by Coverity.

- m_watch: wl has already been confirmed to not be NULL on L161.

Replace dodgy use of const char* and memcpy with std::string.

This was backported from commit e01df63 in master.

Merge pull request #933 from SaberUK/insp20+fix-llvm34

Fix various warnings when building with LLVM 3.5.

Avoid calling methods on NULL pointers wherever possible.

The trick we use to allow this is undefined behaviour and is not
liked by LLVM. We should stop using it but it has the potential to
break to many things for a minor release.

Fix various warnings when building with LLVM 3.5.

- warning: absolute value function 'abs' given an argument of type 'long' but has parameter of type 'int' which may cause truncation of value [-Wabsolute-value]
- warning: 'this' pointer cannot be null in well-defined C++ code; pointer may be assumed to always convert to true [-Wundefined-bool-conversion]

m_ssl_openssl Allow configuring raw OpenSSL context options

m_ssl_openssl Disable session caching and session tickets

m_ssl_openssl Enable single (EC)DH use and disable SSL v2

Options enabled:
- SSL_OP_NO_SSLv2
- SSL_OP_SINGLE_DH_USE
- SSL_OP_SINGLE_ECDH_USE (if it exists)

Partial backport of #856 by @jvehent

m_ssl_openssl Clear the error queue before every SSL_* call

Merge pull request #931 from SaberUK/insp20+fix-non-gnu-stl

Fix hash_map.h on non-GNU C++ standard library implementations.

Merge pull request #930 from rburchell/revertfix

Revert "Fix an off-by-one on registration timeout handling."

Fix hash_map.h on non-GNU C++ standard library implementations.

Revert "Fix an off-by-one on registration timeout handling."

This seemingly breaks (occasionally timing users out a second early), possibly
due to rounding issues on time (User::age and Time() both return int, but a
whole second is comprised of many ms).

The root cause needs investigation, but we cannot leave a potentially breaking
bug in the tree.

This reverts commit 4414d644a163f3906f90b35186e07ce0383161f4.

Merge pull request #929 from rburchell/insp20

Fix an off-by-one on registration timeout handling.

Fix an off-by-one on registration timeout handling.

The use of > instead of >= here added a second onto the configured timeout.

We won't be breaking any existing configuration, because users.h explicitly
increases the limit if it's 0.

m_ssl_gnutls Re-set DH params when the gnutls_certificate_credentials_t struct is reallocated

Reject parameters that begin with a colon in Channel::SetDefaultModes()

Don't try to set a mode in Channel::SetDefaultModes() if it needs a parameter but does not have one

Thanks to @docwhat for helping to track this down, fixes issue #924

Remove IPv6 address compaction.

This code is incorrect and can produce addresses with multiple double
colons. No other IRCds do this and most systems today will give us the
compact version anyway.

Issue #914

Do not use the result of the dns cache when the query type of the result is different from the type of the query. #66

m_sqlite3 Fix cleanup of unsuccessful database connections

Merge pull request #910 from SaberUK/insp20+even-more-build-system-fixes

Fix unit-cc not loading configure cache when building files.

Fix unit-cc not loading configure cache when building files.

Merge pull request #900 from SaberUK/insp20+build-system-fixes

[2.0] Fix various problems with the build system.

Add a work around for trampling over PKG_CONFIG_PATH.

The extend_pkg_path method will be removed in 2.2 as it
causes problems with non-standard pkg-config paths. This
commit adds a workaround until that happens.

Fix pkg-config not being used for OpenSSL on non-FreeBSD systems.

Merge pull request #899 from SaberUK/insp20+fix-listmode-limits

[2.0] Fix listmodes when the config does not specify a wildcard size entry.

Fix listmodes when the config does not specify a wildcard size entry.

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

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

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

Fix various small problems with configure.

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.

Fix copy/paste error in m_denychans.

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

Merge pull request #872 from theckman/remove_whitespace_from_docs

Remove whitespace from docs

remove unneeded whitespace from the docs/examples

Fix harmless loop off-by-ones found by Moonlightning

m_spanningtree Forward NICK messages when they cause a collision with the new nick param rewritten to the uuid

Suppressing these messages meant that we relied on an appropriate NICK to
come from the direction the SAVE was sent to; this left all servers behind
us uninformed until the target server of the SAVE reacted with a nick change
to uuid.

This was problematic because someone can legitimately change nick to the
past nick of the SAVEd user on a server that already has the SAVEd user
with the uuid nick and that nick change message can reach servers that
haven't yet seen the nick change to uuid of the SAVEd user.

m_spanningtree Fix cleanup of connections when unloading a module that provides an IOHook

Update server.cpp

Rewrote "invalid credentials" error message to remove confusion regarding "server snomask", which some users interpreted as "I must make sure the snomask on each server is identical" (which makes no sense), when really the intention is basically "look at the logs on the other server".

Merge pull request #854 from SaberUK/insp20+deprecated-plist-entry

Replace deprecated entry in launchd config with the modern equivalent.

Replace deprecated entry in launchd config with the modern equivalent.

Update year in resource file for Windows

Fix m_permchannels and m_xline_db on Windows

The call to remove() can fail if the database doesn't already exist. Just
ignore the error, which effectively makes the rename() call below work as
it does on nix

Do not export callers

Their functions are all inlined and Windows gives unresolved symbol errors

m_cgiirc Call User::InvalidateCache() after changing host and dhost in OnCheckReady()

m_spanningtree Advertise the availability of m_globops in CAPAB CAPABILITIES

m_spanningtree Set the bursting flag in the FJOIN handler based on whether the source of the FJOIN is bursting or not

Previously the flag was true if our route to the source was bursting, which resulted in m_joinflood incorrectly registering netjoins when a remote server was bursting

Fixes issue #839 reported by @joshtek0

Merge pull request #832 from SaberUK/insp20+fix-ssl-question

Backport questions from master's genssl tool.

Fix undefined symbol error on module load if compiled with Clang 3.4

Fixes issue #803 reported by @SaberUK

Backport questions from master's genssl tool.

Compress the Windows installer with zlib to reduce AV false positives

Change Windows libraries to be dynamically linked

m_ssl_openssl Avoid Applink on Windows by calling PEM_read_bio_DHparams() instead of PEM_read_DHparams()

Allow uppercase letters in IPv6 CIDR masks

m_auditorium Handle NULL return from Channel::GetUser() in OnSendWhoLine()

Found by Coverity, CID: 1135647

Ignore NULL callbacks in ModuleManager::Reload()

Found by Coverity, CID: 1201453

m_ojoin Remove pointless FindChan() call

Found by Coverity, CID: 1135649

Change FormatMessage() usage in DLLManager::RetrieveLastError() to match SocketEngine::LastError()

See 4451fd13a1accec8d7c2b1bb7fdafd6e50a2566b

Filter out newlines from error messages on Windows

m_watch Be more strict when checking whether the watch list of a user is full

After a rehash MAX_WATCH can become lower than it was before

Merge pull request #827 from Robby-/insp20-helpopconf

Some more helpop config updates, and fix a few module syntax messages.

Some more helpop config updates.

Correct the syntax messages of a few modules.

Release 2.0.16

Fix IPv6 address handling in 6aaf7047297b739377e7d509cb914f32447fb281

Fix typo in helpop-full that prevented it from loading and other minor helpop issues

Fix snotice about disabled command usage being sent to the wrong snomask

Add REG_ALL checks to treat unregistered users as nonexistent in more cases

Fix typos and update text in the configure script

Some of these were picked from #448

Major example configuration and helpop update

Big thanks to @m4z (#621) and @Robby- (#630) who contributed a lot to this

m_watch Validate targ before use, noticed while rewriting m_watch for 2.2

m_globalload Don't pass a callback to ModuleManager::Reload() if reloading m_globalload

m_httpd Fix typo noticed by @SaberUK

m_operprefix Don't set +y on the oper if hideoper is being unset due to deopering

Fixes issue #801 reported by @UselessOper

Fix NAMES list getting truncated on servers having an excessively long server name

Backport "Refactor Channel::UserList() to use std::string"

This gets rid of questionable snprintf() usage, reported by @hifi

Backported commit: 8a06d54076551387f83a29360478ee6605e241b6

cmd_names Allow opers having the channels/auspex priv to view the NAMES list of +s channels from outside

A version of the check removed by this commit is in Channel::UserList() which takes the priv into account

Discovered by @Cronus89

Improve detection for non-cidr masks to prevent unwanted matches

Fixes issue #762 reported by @neoinr

m_spanningtree Do pointer comparison before deleting one of the pointers in TreeSocket::Squit()

While the previous code worked fine in practice, it was incorrect in theory

m_spanningtree Don't send snotices to servers about remote servers splitting

The snotice is sent to opers when the SQUIT is processed by their server