The settings were already read at this point, so this didn't actually do
anything. Fixing the code to actually set it would break existing tests,
so just remove it.

If the mail store commit failed, the mail was not guaranteed to be saved.
We don't want to lose mails, even if e.g. a forward had just succeeded.

In that case, the header name and middle are empty, causing:

Panic: file edit-mail.c: line 820 (edit_mail_headers_parse): assertion failed: (body_offset > 0)

Fixed by skipping the bad header for parsing altogether.

This is needed when the mailbox is opened with EXAMINE and is a source
for the operation. imap_sieve_mailbox_run_copy_source() expects the
source mailbox's context to exist.

This avoids adding another Message-id if one is already present but
of bad quality.

Fixes compatibility with monolith.

.git can also be a file, pointing to an external location
for the actual git repository. This usually happens when
code is a subrepository.

It should be PACKAGE_NAME but that breaks too much now.

Is now intended to the used for user deinitialization.
This fixes mail_user_session_finished event not being sent
with managesieve.

To prevent a maybe-uninitialized warning the autodetecting now sets the
storage_path parameter in any case.

These were originally written while the plugin was in draft. These can
be now removed as the RFC has been finalized and the commands were
dropped.

lib-sieve: sieve-storage, sieve-file-storage - Drop deprecated handling of 'sieve_dir' configuration

It only causes confusing output.

The dump-capability just wants to get the capabilities. Any problems with
settings will be handled later on.

Also change the callers to not bother storing it separately. Instead, they
can just use mail_user_r->service_user.