[Minor] Ensure that Redis inactive connections are well maintained

[Fix] Try to use on_connect/on_disconnect callbacks to handle internal Redis failures

[Minor] Free the proper pointer

Merge pull request #3901 from citrin/rspamc

[Fix] buffer overflow in rspamc counters

[Test] Add the test for the previous fix

[Fix] Fix parsing of the from_hostname when it is an IP address

[Fix] Another try to fix rspamd_text passing in the selectors

Suggested by: @citrin

[Minor] Free the allocated and not modified key

Found by: coverity scan

[Minor] Explain unobvious code

[Minor] Fix use after free

Reported by: @citrin

[Fix] Fix passing of rspamd_text in selectors pipelines

[Minor] Fix format string...

Merge pull request #3902 from citrin/warnings

[Minor] Fix compiler warnings in src/libutil/util.c

[Minor] Fix some more issues

Found by: coverity scan

[Minor] Qualify C++ enum

[Minor] Fix some more issues

Found by: coverity scan

[Minor] Fix more error paths in Lua

Found by: coverity scan

[Minor] Improve readability of recently changed rspamc code

[Minor] Fix compiler warnings in src/libutil/util.c

src/libutil/util.c:1362:7: warning: unused variable 'len' [-Wunused-variable]
        gint len = 0;
             ^
src/libutil/util.c:1363:8: warning: unused variable 'pass' [-Wunused-variable]
        gchar pass[BUFSIZ];
              ^
2 warnings generated.

[Fix] buffer overflow in rspamc counters

If request to /counters returns no symbols then max_len would have
a negative value:

Results for command: counters (0.003 seconds)
=================================================================
==22096==ERROR: AddressSanitizer: negative-size-param: (size=-2147483604)
    #0 0x33ff13 in __asan_memset (/usr/bin/rspamc+0x33ff13)
    #1 0x383432 in rspamc_counters_output /usr/src/debug/rspamd/src/client/rspamc.c:1064:2
    #2 0x388c49 in rspamc_client_cb /usr/src/debug/rspamd/src/client/rspamc.c:1600:6
    ...

[Minor] Remove bogus condition

[Minor] Fix macro name

Reported by: @citrin

[Fix] Fix rubbish QP sequences decoding

[Test] Add tests for broken qp decoding

Merge pull request #3899 from citrin/cmake-cleanup

[Minor] Cleanup unused cmake checks

[Minor] Cleanup unused cmake checks

[Minor] Oops, fix emplacement

[Minor] Fix some leaks on error paths

Found by: coverity scan

[Fix] Fix hash table lookup

Found by: coverity scan

[Minor] Some minor cleanups

Found by: coverity scan

[Minor] Fix some more issues

Found by: coverity scan

[Minor] Fix more issues in libmime

Found by: coverity scan

[Minor] Plug some leaks on error paths

Found by: coverity scan

[Minor] Use the proper variable in debug

Found by: coverity scan

[Minor] Do not read extra byte of garbadge

Found by: coverity scan

[Minor] Fix some double frees

Found by: coverity scan

[Minor] Remove dead loop

[Minor] Fix some dead ends

Found by: coverity scan

[Minor] Fix error path varargs handling

Found by: coverity

[Feature] Antivirus: Allow to set fake eicar patterns for testing AV engines

[Minor] Escape \uFFFD in the sources

Merge pull request #3894 from citrin/bitcoin-lua-opt

[Minor]  Skip bitcoin address check for very long words

[Minor]  Skip bitcoin address check for very long words

Exclude very long words (which can be extracted e. g. from some text
attachments) from bitcoin address check to avoid excessive resource
usage.

[Minor] One more polishing thing

[Minor] Fix unregistration of the context and improve logging

[Minor] Fix list splice

Merge pull request #3892 from fatalbanana/friendlier_tests

[Test] Avoid importing user configuration

[Test] Avoid importing user configuration

[Minor] Add attribute noreturn to start_hs_helper function

Suggested by: @citrin

Merge pull request #3891 from fatalbanana/doxydown

[Minor] Update embedded doxydown

[Minor] Update embedded doxydown

 - To fix nested lists

Merge pull request #3890 from fatalbanana/docs

[Minor] Updates for API documentation

[Minor] Updates for API documentation

 - Put nested lists on the correct level
 - Document more symbol type options
 - Try be clearer about dependencies

[Minor] One more try to fix the release order

[Minor] Do not unref what's not been refed

[Minor] Another try to fight with hiredis...

[Minor] Add one more check for use-after-free prevention

[Minor] Allow to capture upstream object

[Minor] Fix deinit order

[Minor] Fix redis termination case not being caught by fuzzy redis

[Minor] Fix C++ logging

[Minor] Fix list splice

[Minor] Redis_pool: Slightly improve style

[Rework] Redis_pool: fix issues found

[Rework] Further rework of the redis pool

[Rework] Start rewriting of the redis pool logic

[Minor] Try to deal with broken CD headers

[Minor] Remove unused cfg fields

[Minor] Store obfuscated subjects in Redis

Issue: #3886

Merge pull request #3889 from arkamar/dep-info

Reflect recent doctest version bump in dependency info table

[Minor] Add content type to MIME_BAD_ATTACHMET symbol

Reflect recent doctest version bump in dependency info table

This just bumps the version number of currently bundled doctest library
in dependency info table.

[Minor] Update bundled doctest to the latest version

[Fix] Fix some complicated case with the closing tags parsing

[Minor] Cleanup metrics output

Merge pull request #3883 from pdf/fix_openmetrics

[Fix] Namespace and add metadata for OpenMetrics, fix interleaving

[Fix] Namespace and add metadata for OpenMetrics, fix interleaving

[Minor] Allow to get invisible content in `rspamadm mime ex`

[Minor] Allow to get invisible content from html

[Rules] Improve zero font rule

[Rework] Save invisible content to a separate buffer

Merge pull request #3880 from fatalbanana/lua_milterheaders

Milter_headers: fix x-spamd-bar for plain lua (#3879)

[Minor] Another PUC-Rio Lua compatibility fix

 - Reported by @mebourne

Merge pull request #3875 from arkamar/system-doctest

Add SYSTEM_DOCTEST cmake option

[Minor] Dmarc_report: Fix template

Issue: #3877

[Minor] Another fix of the remaining octets logic

[Minor] Fix remaining octets logic

[Fix] Fix the case when l tag is too small

[Minor] Fix misused variable

[Fix] Preserve SPF top record in the mempool variable

[Minor] Fix stupid memory leak

[Fix] Plug memory leak in http settings reload

Reverts: d9180d3ac0fac6d0aa12cd33117cb7efc731e2a2

[Minor] Explicitly disallow to set settings on top of other settings

Add SYSTEM_DOCTEST cmake option

This gives packagers option to use system version of doctest rather than
bundled one. It is disabled by default. Additionally, there is no need
to link rspamd-test-cxx with doctest as it is is single header library.

Main reason for this change is that currently bundled version of
doctest-2.4.5 can't compile with glibc-2.34 [1]. However, this issue was
already fixed in upstream doctest-2.4.6 [2] in commit [3]. With this,
packagers can just bump doctest to fixed version.

[1] https://github.com/onqtam/doctest/issues/473
[2] https://github.com/onqtam/doctest/blob/master/CHANGELOG.md#246-2021-03-22
[3] https://github.com/onqtam/doctest/commit/099d5414e97244ec44cf46b14cd176b3a3dc52e3

Merge pull request #3874 from fatalbanana/rspamd_stats_rescore

[Minor] rspamd_stats: support rescoring symbols

[Minor] rspamd_stats: support rescoring symbols

[Minor] Fix number of arguments to invoke a lua function

[Minor] Plug dtor memory leak (not important)

[Rework] Rework learn and add classify condition

[Minor] Add function to get a lua_reference from a string returning a function