Strip duplicates from module's options

mod_stream_mgmt: Clean up on timed out resumption

During resumption, make sure the old process and the corresponding
session entry are disposed also in the case where the call that queries
the old process times out.

Don't forget to remove mgmt_force_enqueue flag

mod_push_keepalive: Preserve timeout on resumption

Don't forget to carry over the original XEP-0198 resumption timeout
value while resuming.

Revert "Support for default values in default_room_opts"

This reverts commit 5054a9933f29d6cc8a88486c624987b1b9f05b17.
The commit is plain wrong: similar options are not required to
be in pair.

Simplify ejabberd_sup code

Fix: mod_offline:store_offline_msg/1 expects a message, not list (#2312)

Simplify result of get_room_affiliation command (#2301)

Merge pull request #2301 from tsaqova/ejabberd_commands_get_room_affiliation

add ejabberd_command to get affiliation of a user in MUC room

Increase log level for DIGEST-MD5 FQDN

SASL DIGEST-MD5 anyway deprecated anyway, so not point in
logging this. This is now logged in `debug` mode.

Merge pull request #2311 from nosnilmot/validate-listen-opts

Validate additional listen opts

Validate additional listen opts

The options "inet", "inet6" and "backlog" are valid listen options, but are
currently logged as errors (even though they do work):

2018-02-28 16:08:44.141 [error] <0.338.0>@ejabberd_listener:validate_module_option:630 unknown listen option 'backlog' for 'ejabberd_c2s' will be likely ignored, available options are: access, shaper, certfile, ciphers, dhfile, cafile, client_cafile, protocol_options, tls, tls_compression, starttls, starttls_required, tls_verify, zlib, max_fsm_queue

This adds the necessary validators so they are correctly recognized.

add ejabberd_command to get affiliation of a user in MUC room

Support for default values in default_room_opts

Use length on fields in mysql indexes

Simplify code for splitting auth string in cyrsasl

This may fix problem from issue #2296

Merge pull request #2304 from weiss/run-auth-callbacks-earlier

Run SASL result callbacks earlier

xmpp_stream_in: Run auth result callbacks earlier

Call Mod:handle_auth_success/4 and Mod:handle_auth_failure/4 before
sending the SASL response rather than afterwards.  This way, callbacks
can send a custom response and disconnect.

mod_stream_mgmt: Cope with exit during resumption

Don't crash if the old process exits while it is queried for the session
state.

Merge pull request #2299 from weiss/add-username-to-indexes

Add username to peer indexes for MAM lookups

sql/*: Add username to peer indexes

The username is available for all MAM queries in question, and adding it
to the indexes can improve the lookup performance significantly.

Merge pull request #2297 from af8a524db1/move_make_rand_string

Move make_rand_string() to 'randoms' module

Move make_rand_string() to 'randoms' module

mod_carboncopy: Apply cosmetic change

The xmpp:has_subtag/2 function returns a boolen() value, so it can be
used with the 'not' operator.

mod_carboncopy: Copy outgoing MUC PMs

Incoming MUC PMs aren't carbon-copied, as the MUC service usually forks
them.  However, don't suppress copying of outgoing PMs, where no such
forking takes place.

Move some log messages to debug level

Use nicks in disco#items or disco#info report

Add 'negotiation_timeout' to the known options list

Introduce 'negotiation_timeout'

The option can be used to specify a period (in seconds) for a stream
negotiation to complete. If the timer fires, the stream is considered
as failed and the underlying connection gets closed. This is a global
option (you cannot set it per domain) and the default is 30 seconds.

mod_admin_extra: Fix srg_get_info with '@all@'

Don't let the srg_get_info command crash if the roster group has '@all@'
or the '@online@' users as members.

Don't emit validator's warning if the module is not found

Replace ?MYLANG with connection's language wherever possible

Push blocking related IQs from bare JID

Fixes #2287

Don't inject node name inside "id" attribute

Fixes #2284

Don't ask other nodes to invalidate cache when the key is not updated

Fix 'badmatch' crash

The crash was introduced in 4b012a99d2bdd6d22f05676e9a7989409e314fca

Introduce 'access' option for mod_block_stranger

The option is supposed to be used when `allow_local_users`
and `allow_transports` are not enough. It's an ACL where `deny`
means the message will be rejected (or a CAPTCHA would be
generated for a presence), and `allow` means the sender is
whitelisted and the stanza will pass through.

The default value is `none`, which means nothing is whitelisted.

Cache 'isuser' queries to external auth program

Improve example extauth script

Handle gracefully that mnesia is already loaded when starting tests

Export helper function

Export helper functions

Export aux functions from mod_muc_room

Rename some keys and functions for clarity

Update incoming stanzas counter on invalid XML

Better solution for a previous fix

Don't forget to add invalid XML responses to sending queue

Fix external components unregistration

Introduce option 'global_routes' for ejabberd_service

The option emulates legacy behaviour which registers all routes
defined in `hosts` on a component connected. This behaviour
is considered harmful in the case when it's desired to multiplex
different components on the same port, so, to disable it,
set `global_routes` to `false`. The default value is `true`,
e.g. legacy behaviour is emulated: the only reason for this is
to maintain backward compatibility with existing deployments.

Fix indentation

Change formatting of commands markdown documentation

Add case on create_room, avoid useless call

Really run use_cache/1 and cache_nodes/1 callbacks for mod_mam

Merge pull request #2274 from Torxen/feature/fixed-return-value-mod_muc

Issue: mod_muc.erl expect wrong return value

Dialyzer checks always failed because the return value of the function 'get_subscribed_rooms' in 'mod_muc_sql' is different to the defined value in 'mod_muc'. Fix was to update the return value in 'mod_muc.erl'.

Remove 'iqdisc' option

Since we got rid of all bottle-neck processes and we have
a connection pool for every database, the option is no longer
needed and in fact is detrimental: in practice what you get
is just a bunch of overloaded processes in the IQ handlers pool
no matter how much you increase the `iqdisc` value.

Given that there are close to zero operators understanding
the meaning of the option and, hence, not using it all,
it's not simply deprecated but completely removed.

The commit also deprecates the following functions:
- gen_iq_handler:add_iq_handler/6
- gen_iq_handler:handle/5
- gen_iq_handler:iqdisc/1

Bump xmpp version

Merge branch 'master' of github.com:processone/ejabberd

Rely on use_cache/1 callback in mod_caps

mod_stream_mgmt: Abort connection on count error

If the client acknowledged more stanzas than the server sent, close the
connection with a stream error rather than hiding client bugs by silently
adjusting the server's count.

Introduce option 'validate_stream'

If set to `true`, all incoming XML packets are fully validated
against known schemas. If an error occurs, the packet will be bounced
with the corresponding error reason. The default value is `false`.
The option might be useful to protect client software from sofisticated
bugs related to XML validation as well as for client developers
who want to catch validation errors at early stage of development.

Note that the option might have slight performance impact, so use it
with care on loaded machines.

Fix type spec

Commands markdown also lack arguments when policy=user

Introduce 'rate_limit' option of mod_avatar

The option controls how many avatars a user can upload per minute.
The option takes positive integer values. The default is 10.
Note that the option only takes effect when an avatar is about
to convert to a different format, i.e. it implies that `convert`
option is configured.

Merge branch 'master' of github.com:processone/ejabberd

Export is_online/1 function

Merge pull request #2266 from mieciu/master

Fix MySQL ERROR 1170 (42000) 'server_host'

Fix MySQL error server_host used in key specification without a key length

Improve log message when module startup has failed

Another try to fix travis

Properly escape string in travis config

Print something when doing tests in travis, as not doing that get us killed

Don't run tests in verbose mode on travis

Fix compilation ordering in mix by s/-behavior/-behaviour/

Fix mod_roster start invocation in elixir tests

Fix elixir tests

Use correct default when getting list of modules from config

Remove line left from debugging

Start ejabberd_config earlier

Disable riak in travis

Run tests only on backends enabled by configure

Match all possible values in mod_privilege checks, or we get crashes

Fix exception in mod_privilege:process_presence_in

This should fix problem reported in issue #2248

Bump sqlite deps version

Fix elixir tests

Fix processing of ldap_memberattr_format_re option

This makes sure that "" value is handled as before options processing
overhaul.

This fixed ldap shared roster testcase

Don't return undefined from ejabberd_config:get_myhosts()

Use named functions instead of carrying self as argument

Restore original return value to ejabberd_config:add_option

ejabberdctl: Omit redundant erl option

The "-detached" flag implies "-noinput".

mod_caps: Only store CAPS if contact is subscribed

If a user is subscribed to a contact but not vice versa, don't store the
contact's CAPS.  This makes sure no PEP items are leaked to the contact.

mod_caps: Fix indentation

Improve logging of idle s2s connections

Find and fix typos using 'codespell'

Fix a typo in call to create_captcha()

Thanks to Paul Schaub for spotting this

Introduce option 'captcha' for mod_block_strangers

When the option is set to `true`, the module will generate
CAPTCHA challenges for incoming subscription requests. The option
also implies that option `drop` is set to `true`. Note that
the module won't generate CAPTCHA challenges for messages: they
will still be rejected if `drop` is set to `true`.

Fixes #2246

Fix more tests

Adopt tests to changes in xmpp

Switch to newer fast_xml and xmpp and update record fields in the code

Handle ignore from xmpp_socket callback start

This should help with issue #2244