[Minor] Slightly refactor the code

Merge pull request #3927 from fatalbanana/minor_config_stuff

Minor config stuff (#3924)

Merge pull request #3929 from fatalbanana/fix_tests_20211008

[Test] whitelist: fix tests

[Test] whitelist: fix tests

[Minor] Do not push invalid real_ip for received

[Minor] Add explicit move operators for mime_string

[Minor] Fix parsing received that start from a comment

[Minor] Require either by or from parts in received headers

[Minor] Restore compatible behaviour for incomplete received headers

[Minor] config: remove RBL defaults that re-enforce true defaults

[Minor] config: don't lie about antivirus whitelist

[Minor] Arc: Specify explicit order for the headers added

[Minor] Lua_mime: Allow to add headers in specific order

[Minor] Fix pushing empty strings

[Minor] Remove from_ip view as it is useless

[Test] Add some more received tests

[Test] Move tests from lua to C++

[Minor] Fix ownership issue

[Minor] Remove bogus forward for non-forwarding reference

[Test] Adopt received framework to allow unit testing

[Minor] Add utility function for the default size mempool

[Minor] One more fix for old libicu

[Minor] Fix rfind usage

[Minor] Fix test message as we now process received headers as expected

[Fix] Fix parsing of the unquoted attributes in HTML

[Fix] Avoid copy for received structure as it has raw C pointers

[Minor] Fix build where vector size() is not constexpr

[Minor] Add workaround for old libicu

[Project] Finish received headers rework part

[Minor] Mark operators constexpr

[Project] Start using of the new received structure

[Project] Move C++ specific declarations to C++ header

[Project] Rework received headers parsing to C++

[Minor] Add some more helpers

[Minor] Add some utility methods

[Minor] Add missing include

[Project] Allow mempool allocated mime strings

[Feature] Ratelimit: Add per bucket configurations

This change add custom per bucket configuration
- symbol: for custom symbol
- message: for custom message
- skip_soft_reject: to disable soft rejection logic for this bucket

[Minor] Allow to construct an empty string with a filter

[Project] Add trim operations

[Minor] Remove const_iterator, template filter functor

[Test] Update iterator tests

[Project] Add constant iterators

[Minor] Remove optional as function_view has nullable semantics

[Minor] Missing include

[Project] Add helper library to handle mime strings in a more safe matter

[Minor] Add C++ safe guards

[Minor] Improve learn error reporting

Merge pull request #3915 from a16bitsysop/static

[Fix] fix static building

[Minor] Finish transition to signed log id

[Minor] Add a special state to tags content parsing to avoid illegal lookahead

[Fix] Add temporary guard to prevent linked list exploitation

[Fix] fix static building

[Minor] Rework to fix issues in Lua API

Found by: coverity scan

[Minor] Fix some more issues

Found by: coverity scan

[Fix] Remove bogus G_LIKELY

[Minor] Remove unreachable code

[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.