Feedback-ID: i3ad947a1:Fastmail
User-Agent: Cyrus-JMAP/3.7.0-alpha0-720-gbf5afa95ff-fm-20220708.001-gbf5afa95
Mime-Version: 1.0
Message-Id: <6010ddbb-acfb-4806-b7a3-b04f4c633024@beta.fastmail.com>
In-Reply-To: <CLBLJEF0Y8M8.37UIZII0LE4TG@moth.falsifian.org>
References: <20220705060618.68685-1-falsifian@falsifian.org>
 <CLAGWLDXIV6M.2KH3WJUCNXDRM@TimBook-Arch>
 <CLBLJEF0Y8M8.37UIZII0LE4TG@moth.falsifian.org>
Date: Sun, 10 Jul 2022 00:25:24 -0500
From: "Tim Culverhouse" <tim@timculverhouse.com>
To: "James Cook" <falsifian@falsifian.org>, ~rjarry/aerc-devel@lists.sr.ht
Subject: Re: [PATCH aerc] Allow not marking viewed messages as seen.
Content-Type: text/plain;charset=utf-8
Content-Transfer-Encoding: quoted-printable

> Looking at that IMAP RFC link, I think the right thing to do is to fet=
ch
> BODY.PEEK instead of BODY. Maybe aerc could always fetch BODY.PEEK
> instead of BOD, so setting \Seen is always deliberate. I've never
> written code that speaks IMAP so take this all with a grain of salt. (I
> see "BODY.PEEK" appears in isync's source [0] though I haven't looked
> carefully.)

I think all that=E2=80=99s needed would be to set {section}.Peek =3D tru=
e in worker/imap/fetch.go [0].

You could add a field to the FetchBodyPart and FetchFullMessage messages=
 which tells the worker what the users config value is for peek.

Good find!

https://git.sr.ht/~rjarry/aerc/tree/master/item/worker/imap/fetch.go