lib-fts: Don't crash if fts_icu_*() are used after fts_icu_deinit()

lib-index: Avoid writing tail offset update to transaction log if it's not necessary.
This should avoid extra writes that do nothing but update the tail offset.
mdbox's map requires this behavior, so disable it for it. For others it
might cause the next sync to read more data unnecessarily, but it should be
worth the extra cost of write most of the times.

doveadm fetch: Added "refcount" field.

cassandra: Error handling cleanup.
Probably doesn't fix any actual bugs.

cassandra: Minor code cleanup to make sure we don't try to access freed transaction memory.

cassandra: Don't access freed memory when doing an assert-check.

doveadm fs delete -R: More fixes to handling errors without crashing.

lmtp: Fixed assert-crash on anvil lookup failures.
If anvil_client_query() fails, it immediately calls the callback and returns
NULL. So we need to increase anvil_queries even before calling
anvil_client_query()

doveadm fs delete -R: Fixed hang when some of the deletions failed.

cassandra: If log_level=debug/trace, log each query and its result.

cassandra: Added extra asserts and sanity checks to make sure query errors are caught.

cassandra: Don't crash on failed queries.

dict-sql: Don't try to optimize finding a matching map by using the previous match.
In some setups multiple maps can match and it's important that the matching
is done in the same order always, otherwise the results could become
somewhat random.

man: doveadm-pw.1: Fix typo.

man: doveadm-pw.1: Added a note about quoting password hashes.

fts: lib-fts search arg expansion wasn't done for non-TEXT/BODY searches when fts_enforced=no

fts: Added "doveadm fts expand" to show lib-fts search query expanded.

lib-storage: Added mail_search_args_to_cmdline()
This isn't entirely correct yet, but it's only going to be used for
debugging for now.

fts: Added "doveadm fts lookup" command.
This is mainly useful for debugging lib-fts. It doesn't perform any of the
lib-fts tokenization / filtering so you can do raw lookups.

lib-fts: fts_tokenizer_reset() didn't reset the entire state.

indexer: Improved handling multiple indexing requests to the same mailbox.
If a request arrives for a mailbox that we were already indexes, the
previous code simply sent the indexing request to the existing worker
process. This could have caused a lot of requests to be buffered to the same
mailbox if the indexing took a long time, which could have taken a while to
process even though they weren't really doing anything indexing work.

The new code instead just keeps track in memory that when the earlier
indexing is finished, it's done again once to finish any pending changes.

indexer: Try to always flush as many requests from queue as possible.
I don't think this changes the behavior much. It's mainly a code cleanup.

indexer: Use array.h API instead of writing our own.
The space savings aren't worth the extra complexity.

fts: If mail indexing fails, don't log a "BUG: Unknown internal error"

fts: If reading mail fails, log an error.

lib-lda: Fixed crash in mail_deliver_get_log_var_expand_table().
Discovered by clang static analyzer. This caused crashes with older versions
of Pigeonhole.

lib-fts: Added "english-possessive" filter.

lib-fts: Use UTF8_IS_START_SEQ()

lib: Added UTF8_IS_START_SEQ() helper macro

lib-fts: fts-filter API changed to have a non-pointer vfuncs variable.
The main benefit being that the fts-filter implementations can save a few
lines of code.

lib-fts: fts-lowercase can now use the default destroy function.

lib-fts: Implemented default create/destory functions for fts-filters.

lib-fts: Moved IS_APOSTROPHE() to fts-common.h

example-config: Updated deliver_log_format comments

lib-lda: Added %{to_envelope} to deliver_log_format

lib-fts: Compiling fix without libicu

lib-fts: Avoid compiler warning when building without libicu

imapc: If auth fails due to connection failure, don't treat it the same as failed login.

lib-fts: Fixed compiling without libicu

lib-fts: Added more supported languages.
This includes now all the ones that are currently supported by both snowball
and exttextcat.

lib-fts: Optimized truncation of partial trailing UTF-8 characters in tokenizers.

lib-fts: Use fts_icu_lcase() for fts-filter-lowercase if possible.

lib-fts: Added string_t *token to struct fts_filter
This makes the work a bit easier for simple filters that don't need any
state but want to use a string_t.

lib-fts: Added fts_icu_lcase()

lib-fts: Do all the ICU cleanup in fts_icu_deinit().

lib-fts: Include fts-filter.h in fts-filter-private.h
This avoids having to always include them both.

lib-fts: Renamed variable.
This is a lowercase-filter, not a normalizer.

lib-fts: Removed unnecessary array.h include from fts-filter.h

lib-fts: Fixed tr29 tokenizer to delete last character correctly when it's preceded by non-ASCII

pop3-login: Added support for Zimbra proxy's XOIP command.

fts: If we fail while indexing headers, return failure immediately.

fts: Error logging fix.
1) We were logging the error after it was already freed from data stack.
2) We were logging uninitialized error string when fts indexing was the one
that failed.

fts: Fixed memory leak at deinit when using multiple fts plugins at the same time using libfts.

lib-fts: Correct comment in filter internal API.

lib-fts: Call libicu u_cleanup.

various: use new uni_utf8_get_char*() interface
No need for additional uni_utf8_char_bytes() calls if you can parse and
know the size with one call.

Signed-off-by: Phil Carmody <phil@dovecot.fi>

lib-fts: Use new uni_utf8_get_char*() interface

lib: Improved unit test for uni_utf8_get_char() return value check.

lib: API change - have uni_utf8_get_char*() return _char_bytes
Often the two functions are called in close proximity (both ways round). As
_get_char*() calls _char_bytes() early on the success path, we may as well
return that value to the caller for immediate use.

The callers which call _char_bytes() first are simply rejecting the truncated
case quickly - all other invalid cases still call both functions, and all
other valid cases (which should be the fast path) likewise call both.

Signed-off-by: Phil Carmody <phil@dovecot.fi>

fts-solr: laxer check of uni_utf8_get_char_n() return value
If uni_utf8_get_char*() were changed to return the number of bytes in the
character on success, then all we care about is it being > 0 (i.e. not
error, not truncated).

Signed-off-by: Phil Carmody <phil@dovecot.fi>

lib: test-unichar - test invalid utf8 encodings
Chop trailing characters off valid encodings, and watch them fail.
(There's no need to do this on most of the test characters, as they're
truncated to the same byte sequence - only do 1 in 64.)

Signed-off-by: Phil Carmody <phil@dovecot.fi>

lib: test-unichar - streamline the unichars test
It's doing 2 kinds of tests, split them into separate test cases.

And the first part has started to get expensive, so just make sure
all code paths are tested by skipping most values. Only 3 from each
set of 64 (lowest 6 bits) are tested.

Signed-off-by: Phil Carmody <phil@dovecot.fi>

lib-fts: Added unit testing for unicode apostrophe handling.

lib-fts: tokenizers - Fixed removal of trailing character in truncated tokens.
If the token is truncated, we don't want to remove the trailing character
since it's not actually there.

Also we don't want to remove trailing apostrophes from a truncated word,
because they're not actually at the end of the (untruncated) token there.
This doesn't make a big difference, but it's slightly more correct.

lib-fts: Optimize tokenizers - Rewrite of apostrophe handling.
We parse through the UTF8 characters only once now, not twice.

lib-fts: tr29 tokenizer - rename variable in preparation for the next patch

lib-fts: tokenizers - don't include removed apostrophes as part of the token size

lib-fts: simple tokenizer minor cleanup - removed unnecessary token length > 0 check
fts_tokenizer_generic_simple_current_token() will check it in any case.

lib-fts: tr29 tokenizer cleanup - Avoid unnecessary goto.

lib-fts: simple tokenizer optimization - don't check unicode word breaks for ASCII chars.

lib-fts: simple tokenizer cleanup - make prev_letter updating more explicit.
It was now hidden inside one of the functions, which didn't make the
prev_letter very consistent when a word break was found. It didn't actually
matter what the prev_letter was at that point, but now the behavior is more
consistent.

lib-fts: simple tokenizer cleanup - removed unnecessary variables

lib-fts: tr29 cleanup - consistently call valid chars "token" and "non-token" chars.
Instead of word/token/text.

lib-fts: tr29 cleanup - Avoid i++ in the for loop to avoid extra calculations

lib-fts: tr29 cleanup - token can never be empty by the time it's being returned.

lib-fts: Optimization for tr29 - we don't need to track last_size explicitly

lib-fts: Correct internal helper function for tr29.

lib-fts: Fix unit tests for TR29 full stop change.

lib-fts: Change TR29 tokenizer to break at full stop (and others).
Diverge from the TR29 rules and always break at MidNumLet letters.
This fixes tokenizing first.last@domain.tld email addresses.

auth: Added %{passdb:field} and %{userdb:field} variables
The field expands to either the passdb or userdb extra field.

You can also use %{passdb:field:defaultvalue} where if field doesn't exist,
it's expanded to defaultvalue. Note that an empty value means that the field
still exists and it's not expanded to defaultvalue.

auth: Added unit test for auth %variable expansion.

auth: Make sure %{mech} and %{session} is escaped in %var expansion.
%{mech} is already very trusted and %{session} should be only from trusted
sources as well, so this doesn't fix any actual security holes. They are
also unlikely to have ever even been used in anything that requires
escaping.

auth: Moved var_expand() related code to its own file.

cassandra: Added log_level parameter to connect_string.
Available values are critical, error, warn (default), info, debug and trace.

lib-fs: Updated fs_write_stream_*() comments related to ostream corking and nfinish.

lib-fs: fs_write_stream_abort() now ignores missing o_stream_nfinish() call

fts: If precaching fails, stop precaching the rest of the mails.
If there are a lot of mails to be precached, this could mean that the
precaching is attempted for a long time and every one of them fails the same
way.

dovecot.m4: If run-test.sh fails, print the failing command to stderr.
Currently otherwise the way Dovecot tests are run it's not clearly visible.

dovecot.m4: If NOVALGRIND environment is set, don't run tests via valgrind in "make check"

lib-fs: fs_write_stream() now automatically corks the ostream and uncorks at _finish().

fts: If last-uid lookup fails, return "Internal error" instead of "BUG" to client.

lib: Avoid race conditions in mkdir*() if directory is being deleted at the same time.
Mainly this allows the call to return failure silently without logging
unnecessary errors.

log: read() errors weren't logged.

systemd service: Fixed typos in the comment section.
The settings for the file descriptor limit is LimitNOFILE.
Removed quotes around the value infinity. Otherwise systemd will
fail to parse that resource value.

systemd: Small improvements to the unit configuration file:

   * Added Documentation URIs
   * Added ExecReload, to reload the config instead of restarting
   * Added a comment for modifying/extending the unit file.

doveadm user: Don't change privileges when looking up users.
We don't need to and it might fail.

lib-storage: If no namespaces were defined, the autocreated namespace settings were a bit wrong.

lib-storage: If session_id isn't given, generate a new one.
This is useful for tracking logs written by services that aren't directly
related to any specific user session.

lib-index: Unit test fix.

lib: file_lock_try_update() comment updated.
It should possibly be removed entirely from the API.