factor out PostContext

mx_mbox_open: drop path param

add mx_path_resolve()

mx_mbox_sync

mutt_mailbox_check: factor out Context

ci_send_message: factor Context out of dependents

ci_send_message: factor out Context

ci_send_message: factor Context out of mutt_make_string()

mutt_change_flag: factor out Context

mutt_save_message: factor out Context

pipe_message: factor out Context

pipe_msg: add Mailbox param

mutt_label_message: factor out Context

mutt_link_threads: factor out Context

crypt_extract_keys_from_messages: factor out Context

mutt_check_traditional_pgp: factor out Context

bounce_message: factor out Context

mutt_ev_message: factor out Context

refactor edit_or_view_one_message

add EmailList for tagged Emails

Map SASL loglevels to neomutt loglevels

fixed: sort_browser attribute modified after sorting

sort_browser has modified after calling browser_sort,
which means the first time sort is called it uses the old
sort method instead of the new one.

Fixes #1551

split up 'update' and 'resort' notifications

Fixes #1546

fix: crash with short attachment name

tags: fix multiple tags set on multiple emails

When we set multiple tags on multiple emails at once, only the first
email get all tags.

This is because we use strsep that will modify the initial buffer that
contains all tags.

This change fixes it by duplicating the buffer of tags before using
strsep on it.

pager: fix missing Mailbox reference

Allow the creation of a new mbox when saving to a local file

Fixes #1537

notmuch: implement read-only vfolder-from-query

Implemented a read-only variant of vfolder-from-query for users who want
to have an immutable window. This ensures no accidental tag changes
occur.

+install instructions in README.md

context: move message_is_tagged,message_is_visible

mutt_make_string: separate Context from Mailbox

doxy: fix headers

notmuch: remove Context in nm_read_entire_thread()

Removed the coupling between `nm_read_entire_thread()` and `struct
Context` by replacing it with `struct Mailbox`.

`Context` had two uses: `Mailbox` reference source and `ctx_update()`.

The introduction of mailbox notification code allows mailbox events to
trigger `ctx_update()`, which removes the hard dependency on `Context`.

Passing a `Mailbox` reference removes the remaining soft dependency on
`Context`.

Relax non-empty requirement on compose_format, pager_format, and status_format (#1531)

Fixes #1530

Revert "Don't override ndata with temporary context"

This reverts commit c128c131185fb076476991f477565ba115352faa.

Don't override ndata with temporary context

postponed/sendlib code can reopen an already opened mailbox.

This overrides m->ndata with a temporary context but never revert it back to
the previous one.

At first glance, when postponed/sendlib code open the mailbox they don't care
to have their context updated with the notification system. It should be
the Context used for the UI that should be updated instead.

So until we don't open a mailbox twice anymore, just set ndata when it
looks accurate to workaround the issue.

fix: check for mailbox for folder-hook

Fixes #1527

fix: saving email to fcc

Sort by UID so sequence ranges in UID STORE make sense

This fixes the use case where a bunch of messages are tagged, removed,
then the mailbox is sync'd in a thread view. In this case, the sequence
number given by email->index is not monotonic, which results in weird
sequence sets being passed to UID STORE.

fix crash in ci_send_message()

Fixes #1525

fix crash in mutt_parse_unmailboxes()

Fixes #1523

close command prompt when deleting the colon

Fixes #1505

merge: upstream changes

 * Factor out rfc2047_decode_envelope()
 * Parse and store mime headers in the BODY
 * Add mx operation save_to_header_cache
 * Add protected header received email support
 * Add rfc2047_encode_envelope helper
 * Refactor save_fcc() function
 * Move fcc after send
 * Prompt when mutt_write_fcc() fails
 * Refactor postpone_message()
 * When postponing, encode descriptions before encrypting
 * More postpone cleanup
 * Fix flags if mutt_get_postponed() has no messages
 * Display mail sent message after Fcc finishes
 * Handle improperly encoded pgp/mime octetstream part
 * Add config and data structure for protected header write support
 * Add a new mode for mutt_write_rfc822_header()
 * Finish protected header write support
 * Read in protected headers when resuming a postponed message
 * Free pgpkeylist if the send fails
 * Rfc2047 encode and fold X-Label when writing updates
 * Restore the xlabel_changed reset
 * Make a copy of x_label before encoding it for updates
 * spelling consistency
 * Updated French translation
 * Added support for the "light" color prefix (in addition to "bright")
 * Light colors: do not do the + 8 on negative values, such as COLOR_DEFAULT
 * Create envelope->changed to mark all field changes
 * Change x-label editing flag to be in the envelope
 * Add $crypt_protected_headers_save
 * Color protected subject as a header in the pager
 * Respect ignore for the protected subject display
 * fix config order
 * Remove unused MUTT_FORMAT_MAKEPRINT flag
 * Sleep $sleep_time after displaying "mail sent" message
 * Update copyright for the next stable release

Update copyright for the next stable release

Co-authored-by: Richard Russon <rich@flatcap.org>

Sleep $sleep_time after displaying "mail sent" message

In case returning the index displays some kind of error right away, to
give time to see the success message.

Co-authored-by: Richard Russon <rich@flatcap.org>

Remove unused MUTT_FORMAT_MAKEPRINT flag

The code in mutt_FormatString() that was using it was commented out 19
years ago.

Co-authored-by: Richard Russon <rich@flatcap.org>

fix config order

Co-authored-by: Richard Russon <rich@flatcap.org>

Respect ignore for the protected subject display

Co-authored-by: Richard Russon <rich@flatcap.org>

Color protected subject as a header in the pager

Co-authored-by: Richard Russon <rich@flatcap.org>

Add $crypt_protected_headers_save

Setting this option will save the protected header back into the
clear-text message headers.  This improves
usability (searching/limiting/replying) when reopening a mailbox
without header cache.  However, it is a security trade-off, so
defaults off and strongly warns about what it is doing in the documentation.

Co-authored-by: Richard Russon <rich@flatcap.org>

Change x-label editing flag to be in the envelope

Handle like In-Reply-To and References editing.  Change the flag
setting to be inside mutt_copy_header().

The resetting of the changed flag occurs explicitly in the imap and
maildir/mh sync_mailbox code.  It occurs for mbox in the
mx_update_tables() call in mx_sync_mailbox().

Co-authored-by: Richard Russon <rich@flatcap.org>

Create envelope->changed to mark all field changes

In subsequent commits, we're going to add the x-label and subject
headers changed flags into the envelope.  To avoid the list of checks
exploding everywhere, just use a single field to check and reset those
values.

Several places in the code are checking for a null header->env.  I
wasn't aware this was possible, so I've added todo notes to track down
when this occurs.

Co-authored-by: Richard Russon <rich@flatcap.org>

Light colors: do not do the + 8 on negative values, such as COLOR_DEFAULT

Co-authored-by: Richard Russon <rich@flatcap.org>

Added support for the "light" color prefix (in addition to "bright")

At the same time, restrict the advance by 8 to colors in the range 0-7
and no longer use the blink attribute to emulate light background colors
in some terminals (e.g. linux console and rxvt), as this is really blink
in other terminals (e.g. xterm); light background colors can still be
obtained by choosing a proper $TERM value (tested with linux console,
using TERM=linux-16color, and rxvt).

Co-authored-by: Richard Russon <rich@flatcap.org>

Updated French translation

Co-authored-by: Richard Russon <rich@flatcap.org>

spelling consistency

Co-authored-by: Richard Russon <rich@flatcap.org>

Make a copy of x_label before encoding it for updates

This isn't actually a bug.  Context->label_hash strdups the keys, so
we are safe from dangling references.  However, the subj_hash uses
direct references, so to keep things consistent and safe, make a copy
and encode that.

Co-authored-by: Richard Russon <rich@flatcap.org>

Restore the xlabel_changed reset

The cherry pick accidentally obliterated the reset.  That flag is now
located in hdr->env->changed in the branch and is reset properly
elsewhere.  Keep it in place in the stable branch.

Co-authored-by: Richard Russon <rich@flatcap.org>

Rfc2047 encode and fold X-Label when writing updates

Also, add a missing "skip" in mutt_copy_hdr() for the non-simple case.

These fixes were backported from the kevin/mutt_protected_header branch.

Co-authored-by: Richard Russon <rich@flatcap.org>

Free pgpkeylist if the send fails

Co-authored-by: Richard Russon <rich@flatcap.org>

Read in protected headers when resuming a postponed message

Co-authored-by: Richard Russon <rich@flatcap.org>

Finish protected header write support

Write out the protected headers when writing the mime header part.

Hide protected subjects with $crypt_protected_headers_subject, for
outgoing, postponed, and fcc'ed messages.

Don't hide in postponed and fcc'ed if $crypt_protected_headers_read
isn't set.

Add a few missing cases where mime_headers needed to be cleaned up on
error.

Remove the protected headers for $fcc_clear.

Co-authored-by: Richard Russon <rich@flatcap.org>

Add a new mode for mutt_write_rfc822_header()

Convert the mode parameter to an enum, to make the code a bit more readable.

Co-authored-by: Richard Russon <rich@flatcap.org>

Add config and data structure for protected header write support

Add config vars $crypt_protected_headers_write (unset by default),
and $crypt_protected_headers_subject.

Store the protected headers during mime_protect().

Co-authored-by: Richard Russon <rich@flatcap.org>

Handle improperly encoded pgp/mime octetstream part

Some clients (or even mail servers) improperly encode the octetstream
part.

Thanks to Riccardo Schirone for the original merge request patch.
This commit also handles the attachment menu, and makes the decoding
conditional so it's not done if it isn't necessary.

Co-authored-by: Richard Russon <rich@flatcap.org>

Display mail sent message after Fcc finishes

For the case of IMAP, the uploading message would otherwise stay on
the screen.  In any case, it's best to show that message as the last
thing before returning to the index.

Thanks to chdiza for reporting the issue!

Co-authored-by: Richard Russon <rich@flatcap.org>

Fix flags if mutt_get_postponed() has no messages

Flags needs to be reset so memory cleanup is done at the end.

Co-authored-by: Richard Russon <rich@flatcap.org>

More postpone cleanup

Provide an error message if $postponed is not set.

Make sure the clear content is freed for encrypted messages.

If the write_fcc() fails, make sure to restore the clear content.

Co-authored-by: Richard Russon <rich@flatcap.org>

When postponing, encode descriptions before encrypting

This matches the behavior of the normal send process.  I don't think
there is actually an issue here since nothing is sent, but it makes
sense to be consistent.

Co-authored-by: Richard Russon <rich@flatcap.org>

Refactor postpone_message()

Co-authored-by: Richard Russon <rich@flatcap.org>

Prompt when mutt_write_fcc() fails

Allow the user to retry, specify an alternate mailbox, or skip.

Co-authored-by: Richard Russon <rich@flatcap.org>

Move fcc after send

Co-authored-by: Richard Russon <rich@flatcap.org>

Refactor save_fcc() function

Co-authored-by: Richard Russon <rich@flatcap.org>

Add rfc2047_encode_envelope helper

Co-authored-by: Richard Russon <rich@flatcap.org>

Add protected header received email support

Add $crypt_protected_headers_read config variable to enable reading
and updating the index/header cache.

Print protected Subject header in the pager as parts are rendered.

Once opened, update the index, subject hash, and header cache.

Co-authored-by: Richard Russon <rich@flatcap.org>

Add mx operation save_to_header_cache

This will be used when reading protected headers, to store the
encrypted subject in the header cache so it can be searched with.

Co-authored-by: Richard Russon <rich@flatcap.org>

Parse and store mime headers in the BODY

Don't store the field in the header cache though.

Co-authored-by: Richard Russon <rich@flatcap.org>

Factor out rfc2047_decode_envelope()

Co-authored-by: Richard Russon <rich@flatcap.org>

clang-format

acutest: sync to upstream

merge: factor Context out of mailbox functions

 * imap: drop Context from imap_copy_messages()
 * tidy mbox_close
 * mailbox: add notify callback
 * mx_fastclose_mailbox: use mailbox notifications
 * mx_fastclose_mailbox: factor out Context
 * imap_expunge_mailbox: use mailbox notifications
 * imap: drop mdata->ctx
 * mbox_open: factor out Context
 * mbox_check: use mailbox notifications
 * mbox_check: factor out Context
 * mbox_sync: use mailbox notifications
 * mbox_sync: factor out Context
 * split out context functions
 * mx_mbox_close drop index_hint
 * imap tidy

imap tidy

mx_mbox_close drop index_hint

split out context functions

mbox_sync: factor out Context

mbox_sync: use mailbox notifications

mbox_check: factor out Context

mbox_check: use mailbox notifications

mbox_open: factor out Context

imap: drop mdata->ctx

imap_expunge_mailbox: use mailbox notifications

mx_fastclose_mailbox: factor out Context

mx_fastclose_mailbox: use mailbox notifications

mailbox: add notify callback

tidy mbox_close

imap: drop Context from imap_copy_messages()

folder-hook: try matching on description

Use the description, in addition to the path, when determining to
execute a `folder-hook`. Since `named-mailboxes` and `virtual-mailboxes`
use descriptions to hide ugliness of raw paths, the description may
differ from the path and/or contain information not included in the
path.

The inclusion of description checking makes `folder-hook`s improves its
ergonomics. `notmuch` users were particularly affect by poor
`folder-hook` ergonomics since `virtual-mailboxes` paths often had large
overlap, making it hard to craft a good regex. A workaround was to add a
unique dummy key.

merge: add summary pages

 * add ICommand for new :ex-style commands
 * Implement :set [all] commands
 * Implement :version command

Implement :version command

Implement :set [all] commands