m_kicknorejoin Store and compare uuids instead pointers

Fixes the off chance scenario where we disallow a join because a previously kicked user has quit and the User who is trying to join happens to be allocated at the exact same memory location

m_kicknorejoin Limit time to 30m by default

In the current implementation we only expire entries when someone joins, without a limit it was possible to make us practically never remove entries and consume (a tiny amount of) memory for each entry until the mode was removed/parameter was changed

The default limit of 30m is chosen to not surprise people when they upgrade. If you need to prevent rejoins for more than a minute then you should set a (timed)ban instead

Config option is available to change the limit (2.0 only)

m_kicknorejoin Minor improvements

- Ignore remote users
- Remove expired items in one pass

Close listening sockets regardless of the return value of shutdown()

Do not enable SO_LINGER on our sockets

Using this option allowed close() to block for up to a second

Thanks to Shamsdeen and Rix for their assistance that made this fix possible

Fixes issue #445 reported by @shaggie76
Fixes issue #494 reported by @Rixcho

Use std::string internally in UIDGenerator, move UUID_LENGTH into the class as a constant

Simplify UID generation logic

This loop is not required because we already set current_uid[pos] to 'A' before recursing if current_uid[pos] is 9

Extract UID/SID generation logic into a new class: UIDGenerator

Remove const char* versions of InspIRCd::FindNick()/FindNickOnly()/FindUUID()/FindChan()

m_spanningtree Remove redundant first parameter of PING/PONG

m_spanningtree Ignore hopcount in incoming SERVER command, always send 0 at burst

m_spanningtree Netburst: If a topic was ever set, send it even if it's empty

A new but empty topic should override a non-empty old topic

Merge pull request #476 from SaberUK/master+allowcoreunload

Add <security:allowcoreunload>.

m_spanningtree Add channel timestamp to FTOPIC

m_spanningtree Add channel timestamp to channel METADATA

Add <security:allowcoreunload>.

When enabled (the default) this setting prevents the unloading of
core modules such as cmd_privmsg.

m_spanningtree Introduce IJOIN and RESYNC

When a local user joins an existing channel, instead of an FJOIN, send an IJOIN with the channel name being the first parameter.
If the joining user received prefix modes, append the channel TS and the prefix mode letters as the second and third parameters.

When receiving an IJOIN, first check if the target channel exists. If it does not exist, ignore the join (that is, do not create the channel) and send a RESYNC back to the source.

If the channel does exist then join the user, and in case any prefix modes were sent (found in the 3rd parameter), compare the TS of the channel to the TS in the IJOIN (2nd parameter).
If the timestamps match, set the modes on the user, otherwise ignore the modes.

Outgoing IJOINs to 1202 protocol servers are converted to FJOINs, but the channel mode parameter is left empty ("+").

m_spanningtree Bump protocol version and remove unused CAPAB CAPABILITIES entries

Channel::JoinUser() and Channel::ForceChan() changes

Convert static Channel::ForceChan() to non-static Channel::ForceJoin() that joins a user to a channel, no permission checks

The (static) Channel::JoinUser() now has a LocalUser parameter, and no longer have TS and bursting parameters. If the channel doesn't exist, it is created using current time as TS

OnUserPreJoin is a local-only hook, change User* parameter to LocalUser*

Remove the deprecated invite API

m_spanningtree Remove unneeded #includes

m_spanningtree FJOIN handler: Refactor, update doc

Split the core function into several smaller functions
Accept userlist entries having no comma at all

m_spanningtree Netburst: Refactor SendChannelModes()

Rename it to SyncChannel() and change it to take a Channel* parameter, move iteration into DoBurst()

m_spanningtree Change both prefix and first parameter of SQUIT to be a SID

m_spanningtree Remove encapsulation on UserCount/OperCount

m_spanningtree Introduce new function to send channel messages

Use it from the protocol interface and PRIVMSG/NOTICE handlers
Unite OnUserNotice and OnUserMessage code into LocalMessage()

m_spanningtree Change TreeServerList to be a set, get rid of a function dedicated to adding entries to it

m_spanningtree Utils: Move code that creates a full line from its components to a new function

m_spanningtree FTOPIC handler: Keep only one topic if the two topic timestamps match but the topics differ

m_spanningtree FTOPIC handler: Return CMD_FAILURE/CMD_INVALID as appropiate

Return CMD_FAILURE to prevent propagation when the topic wasn't updated; return CMD_INVALID when the topicts is invalid

m_spanningtree atoi() to ConvToInt() conversion, add const where possible

Remove two redundant functions from Utils

m_spanningtree RSQUIT handler: Don't reimplement sending messages to remote users

Use ModuleSpanningTree::RemoteMessage instead

m_spanningtree RCONNECT handler: Fix error reporting to remote users

Describe behavior in comments

m_spanningtree Netburst: Rework SendFJoins()

Old code sent empty lines on burst after FJOINs, new version fixes that

m_spanningtree Netburst: Remove expiration check from SendXLines()

XLineManager does the checking

m_spanningtree UID handler: Log and drop the link when a server introduces a user with an unknown user mode

m_spanningtree UID handler: Remove dead code

m_spanningtree FMODE handler: Rework

Check for error conditions before doing any work
Get rid of hardcoded maxmodes and a loop that copies a container

m_spanningtree IDLE handler: Rewrite

Merge pull request #461 from SaberUK/master+header-cleanup

Tidy up source files.

Tidy up source files:
- Use #pragma once instead of include guards.
- Move header files in src/modules to include/modules.
- Fixed various spacing issues.

Merge pull request #488 from SaberUK/master+loglevel-rename

Add LOG_ prefix to the log level enum values.

Add LOG_ prefix to the log level enum values.

Merge pull request #487 from SaberUK/master+better-isupport-api

Convert ISUPPORT to use a map instead of a string.

Convert ISUPPORT to use a map instead of a string.

Fix clang warning about IsOper()

Merge pull request #484 from SaberUK/insp20+fix-modulemanager

Fix ModuleManager error caused by a lack of arguments.

Fix ModuleManager error caused by a lack of arguments.

Run OnUnloadModule hook before starting to unregister services provided by the module

BanCache: Move expiration code into a function, call it from RemoveEntries()

BanCache: Remove BanCacheManager::RehashCache()

The function was never called

BanCache: Simplify BanCacheManager::RemoveEntries()

BanCache: Remove BanCacheHit::IP field, and BanCacheManager::RemoveHit()

The IP field was only used in RemoveHit(), RemoveHit() was only called from GetHit()

BanCache: Do one hash lookup in BanCacheManager::AddHit()

BanCache: Don't repeat ourselves, one BanCacheManager::AddHit() and one BanCacheHit constructor is enough

Replace IS_AWAY() and IS_OPER() macros with User::IsAway() and User::IsOper()

m_spanningtree Create new TreeServers for incoming connections only when they've accepted our credentials, not when they send SERVER

Update Window's .gitignore

Fix ListModeBase on Windows

Remove some uline checks that ran after an IS_LOCAL() check

Move most whois related code from the core into cmd_whois

XLine: Provide a default implementation for DisplayExpiry()

ListModeBase: Cache max items per channel

ListModeBase: Minor changes to original u_listmode code

- Add constructors to ListItem, ListLimit
- Rename fields in ListItem
- Store time as time_t instead of string
- Store limits in a vector instead of a list
- Don't deallocate the list when it becomes empty

Migrate u_listmode.h into the core, change +b to use it

Add builtin modes using AddService()

Unite include/modes/*.h into include/builtinmodes.h

Allow dynamic references to reference ServiceProviders, introduce ModeReference

Mode handlers can be referenced using mode/<modename>

Rework dynamic_reference management, introduce dynamic_reference_nocheck

New dynamic references are now resolved at construction and all of them are re-resolved when a relevant service is added or removed; resolution is no longer done in operator->

dynamic_reference_nocheck is a variant of dynamic_reference that does not check for value being null in operator-> / operator*
dynamic_reference still throws an exception when used in this case

Both kinds of dynamic references support .check(): an exception is thrown if this is called when value is null

Remove OnAddBan and OnDelBan hooks

Add support for generating dhparams with GnuTLS to genssl.

Merge pull request #475 from SaberUK/master+default-to-xml-config

Default to the XML config format.

Default to the XML config format.

Merge pull request #474 from SaberUK/master+better-config-deprecation

Improve the config deprecation system.

Add the ability to deprecate config values as well as keys.

Purge the 1.1 config deprecations.

Merge pull request #472 from SaberUK/master+merge-headers

Merge inspircd_{config,version}.h into a single header file.

Don't attempt to unload or reload modules that are waiting to be unloaded

Fix LUSERS not working in a PURE_STATIC build

Merge inspircd_{config,version}.h into a single header file.

Merge pull request #465 from Shawn-Smith/master+ServicesOperGrammar

[2.2] Fixed grammar issue

Fixed grammar in whois output for +k'd services.

Stop recreating hashmaps every hour, move garbage collection code related to local users to UserManager

Line ending conversion

Move whowas settings into cmd_whowas from ConfigReader

Make cmd_whowas act like a module, remove special handling

Convert InspIRCd::Duration() to be static

Merge pull request #460 from SaberUK/master+genssl-bugfix

Fix a small typo in genssl.

Fix a small typo in genssl.

Merge pull request #459 from ChrisTX/insp20+fixgeoip

m_geoip: Fix compilation

m_geoip: Fix compilation

Merge pull request #456 from SaberUK/master+genssl

Extract SSL generation to a tool which can be shipped by distros.

Extract SSL generation to a tool which can be shipped by distros.

Whitespace and empty destructor removal, minor coding style changes

Ignore empty sid in ServerConfig::Fill()

Merge pull request #452 from SaberUK/master+nuke-hashmap

Purge the deprecated hash_map from existance.

Move member variables from User to LocalUser

- idle_lastmsg
- dns_done
- quitting_sendq
- exempt
- lastping

cmd_stats List opers without iterating the whole userlist

cmd_stats List opers without iterating the whole userlist