Apply suggestions from code review

Co-authored-by: Vsevolod Stakhov <vsevolod@rspamd.com>

feel silly about this one, forgot the username param.

else if was meant to be elseif

changed errors to warnings.
nullptr on contructor was never meant to be there

Added support for Redis 6 ACL (username/password)

[Minor] Another leak

[Minor] Fix leak

[Minor] Simplify

Merge pull request #4557 from twesterhever/temp-align-dnsbl-scores

[Minor] Slightly adjust DNSBL scores

[Minor] Simplify transform

[Minor] Allow rspamd_lua_require_function to work without function

Merge pull request #4573 from rspamd/cfg_utils_rework

[Project] Rewrite configuration utils in C++

[Minor] Fix some docs

[Minor] Fix some more issues

[Minor] Set top section explicitly

[Minor] Allow workers to have any attributes (needed by fuzzy)

[Minor] Restore mempool_strdup bahaviour when arg is NULL

[Minor] Do not treat hyperscan load failure as a fatal error

[Minor] Add missing include

[Rewor] Refactor some legacy stuff

[Feature] Add utility to split string like stuff for C++ code

No ranges, as they are a bit ugly to use yet

[Rework] Some final fixes

[Rework] Final efforts to make it compileable

Merge pull request #4577 from moisseev/webui

[WebUI] Tweak bootstrap colors for accessibility

[Rework] More steps to do refactoring

[WebUI] Tweak bootstrap colors for accessibility

Resolves: #4566

Merge pull request #4572 from lingfish/json-raw

[Feature] Add new exporter to push JSON to a raw TCP socket

[Rework] Further refactoring

[Rework] Further conversion routines

Rename exported to be more explicit

[Rework] More rework in cfg_rcl

So many places need to be rewritten

[Rework] More abstractions to hide C++ internals

[Rework] Move rcl logic to C++

[Rework] Finish utils rework

[Rework] Use more c++ in cfg_utils

[Fix] Do not accept invalid ucl object types

Issue: #4571

Add new exporter to push JSON to a raw TCP socket; for example to Graylog.

[Fix] Fix output of non-RSA DKIM keys

Issue: #4570

[Fix] Do not set output type if list application failed

[Minor] Add some extra debug

[Minor] Treat last dot specially

[Fix] Fix some corner cases of single-host urls parsing

[Test] Add one more test for urls parsing

[Fix] Fix `url:set_redirected` method

Merge pull request #4569 from moisseev/webui

[WebUI] Restore contrasting foreground color

[WebUI] Restore contrasting foreground color

for badges

Merge pull request #4567 from moisseev/webui

[Minor] Update JS libraries

[Minor] Update JS libraries

[Minor] Fix usage

[Fix] Fix lua schema enrichment logic for Redis params

[Fix] Fix merge table utility

[Feature] Check for plugin configuration errors on `configtest`

[Minor] Update function docs

[Minor] Allow to call function from stack in rspamd_lua_universal_pcall

[Minor] Move config stuff out

[Minor] Remove unused utility, as it has been broken for ages

[Minor] Allow to log configuration errors from plugins

[Minor] Reformat all Lua code, no functional changes

[Minor] Distinguish failures from unknown errors

[CritFix] Fix leak in `gzip` function

Issue: #4564

Merge pull request #4562 from helsinki-systems/fix/redis_schema_parsing

[Fix] Redis schema parsing

fix: redis schema parsing

Merge pull request #4560 from a16bitsysop/tests

[Fix] CMakeLists.txt remove whitespace added by linter as it makes te…

[Fix] CMakeLists.txt remove whitespace added by linter as it makes tests fail

[Minor] Update version to 3.7

Release 3.6

* [Conf] Add `one_shot` to some specific multimap rules
* [Conf] Add language detection configuration
* [Conf] Add missing attributes for the language detection configuration
* [Conf] Remove outdated composite rules
* [Feature] Add `sentinel_password` option
* [Feature] Add ability to deny specific fuzzy flags by default
* [Feature] Add controller endpoint to get fuzzy hashes from messages
* [Feature] Add extra symbol when URL redirector reaches nested limit
* [Feature] Add function to transliterate utf8 to ascii with some normalisation
* [Feature] Add html parsing limit
* [Feature] Add order to urls structure
* [Feature] Add some missing functions to `lua_rsa` library
* [Feature] Allow fuzzy workers to exchange blocked information
* [Feature] Allow to have weak flags in fuzzy storage
* [Feature] Allow to read options from maps in the multimap plugin
* [Feature] Allow to use other methods when fasttext detection is enabled
* [Feature] Count stats per key per flag
* [Feature] Finish all features of dkim_keygen in Lua
* [Feature] Khash: Allow static initialisation
* [Feature] Maps: Add on_load support
* [Feature] Preliminary implementation of dynamic composites
* [Feature] Process HTML parts before text ones
* [Feature] Reorganise struct rspamd_url to be 64 bytes size
* [Feature] Save fuzzy ratelimit buckets
* [Feature] Use in-place deflation for strings
* [Feature] external_relay: add ip_map strategy
* [Fix] Avoid race between config new/free by using a counter
* [Fix] Do not use `rspamadm.dkim_keygen`
* [Fix] Feed fasttext language model with the pre-tokenized words
* [Fix] Fix `rspamd_has_only_html_part`
* [Fix] Fix an old issue with order of destruction race between redis pool and lua
* [Fix] Fix format string usage
* [Fix] Fix parsing due to old bug revealed
* [Fix] Fix parsing of the mask values that are invalid
* [Fix] Ignore non-unique stop words
* [Fix] Include the last character when parsing the last header with no value
* [Fix] More fixes to fuzzystat
* [Fix] Set proper counter
* [Fix] Try harder to clean pending bucket
* [Fix] Try harder to remove bad hyperscan files
* [Fix] Update stats before encryption...
* [Fix] dmarc gramar - allow spaces before ";"
* [Fix] rbl: fix dependency registration when symbols_prefixes is used
* [Fix] remove obsolete rspamd-redirector files
* [Fix] test external_relay: count should always be the last rule, as it have no matching condition
* [Project] Allow to register multimap symbols dynamically
* [Project] Implement fasttext language detection
* [Rework] Default max shots must not influence options
* [Rework] Write dkim keygen tool in lua
* [Rules] Add thread hijacking composite rule

Merge pull request #4558 from twesterhever/temp-detect-google-amp-urls

[Minor] Tweak HAS_GOOGLE_REDIR to detect Google AMP URLs as well

Merge pull request #4559 from fatalbanana/html_only

[Minor] Test & fix has_only_html_part regression

[Minor] Test & fix has_only_html_part regression

[Fix] Fix `rspamd_has_only_html_part`

[Minor] Tweak HAS_GOOGLE_REDIR to detect Google AMP URLs as well

Rationale: https://cofense.com/blog/google-amp-the-newest-of-evasive-phishing-tactic/

[Minor] Align scores of Spamhaus DBL, SURBL, URIBL DNSBL symbols

Given that they have about the same false positive rate, it makes sense
to treat them equal in terms of scoring:
- Particular threats (phishing, malware) are scored a bit higher than
  mere spam domain listings
- "Abused legitimate" listings are scored lower for some DNSBLs already,
  this has now been aligned.
- For SURBL, cracked and abused sites are treated with the same score.

[Minor] Increase score of URIBL_XBL

This aids with detecting FQDNs hosted on hacked machines, such as used
in Fast Flux-style botnet spam.

[Minor] Reduce score of URIBL_SBL_CSS

Given that CSS is an automated component of SBL, this should not receive
the same scoring as manually conducted SBL listings. Particularly for
shared hosting environments, CSS hits on IP addresses derived from FQDNs
sometimes were found to be scored a bit too high.

[Minor] Allow symbols insertion for skipped tasks

Merge pull request #4556 from twesterhever/temp-improve-freemail-aff

[Minor] Improve catch rates of FREEMAIL_AFF

[Minor] Improve catch rates of FREEMAIL_AFF

[Feature] Add `sentinel_password` option

[Minor] Link with libatomic where possible

Issue: #4552

[Dep] Update fmtlib to 10.0.0

[Minor] Add fmt::format specialisation

[Minor] Add missing include

[Minor] Remove SunPro support as it is not really supported

[Minor] Remove unused var

[Minor] Improve log message

[Fix] Avoid race between config new/free by using a counter

[Minor] Fix pointers operations

[Minor] Remove no longer used files

[Minor] Add .overcommit.yml file so I won't loose my checks

[Minor] Add Lua methods to get urls order

[Rework] Use clang-format to unify formatting in all sources

No meaningful changes.

[Feature] Add order to urls structure

[Minor] Oops: I have written too long in C++

[Feature] Process HTML parts before text ones

[Minor] Slightly change the text processing logic

Merge pull request #4549 from fatalbanana/ratelimit

[Minor] Minor fixes for previous ratelimit changes

[Minor] Minor fixes for previous ratelimit changes

[Minor] Propagate port from url parser

[Minor] Restore old port behaviour

[Feature] Reorganise struct rspamd_url to be 64 bytes size