Support filtering rules to move mails to folders #187

New Issue

IssueBot · 2022-09-18T16:41:02+03:00

IssueBot commented

2022-09-18 16:41:02 +03:00

Anonymous reports:

Maybe you've already considered this and decided against but I would
really find it valuable if simple filtering rules would be supported,
e.g., as done in alpine/pine to move a mail to another folder based on
the subject prefix or To/CC/From.
Otherwise one would have to use procmail or something like that but
since meli already has IMAP support, it would make more sense to have it
in one solution.

Anonymous reports: Maybe you've already considered this and decided against but I would really find it valuable if simple filtering rules would be supported, e.g., as done in alpine/pine to move a mail to another folder based on the subject prefix or To/CC/From. Otherwise one would have to use procmail or something like that but since meli already has IMAP support, it would make more sense to have it in one solution.

Manos Pitsidianakis commented

2022-09-19 15:57:55 +03:00

I guess the question would be what this filtering would look like. I wouldn't want to introduce a custom DSL just for meli 🤔. There's already code for connecting to a manage sieve connection and a REPL binary in the repository src/managesieve.rs that lets you interact with an IMAP's sieve filters but nothing fancier. I will see how difficult it would be to parse sieve since there there are many RFC extensions etc, and offer optional client-side filtering for all backends.

I guess the question would be what this filtering would look like. I wouldn't want to introduce a custom DSL just for meli 🤔. There's already code for connecting to a manage sieve connection and a REPL binary in the repository `src/managesieve.rs` that lets you interact with an IMAP's sieve filters but nothing fancier. I will see how difficult it would be to parse sieve since there there are many RFC extensions etc, and offer optional client-side filtering for all backends.

Manos Pitsidianakis added the

enhancement

label 2022-09-19 15:58:01 +03:00

Manos Pitsidianakis referenced this issue from a commit

2022-12-26 21:27:59 +02:00

melib/sieve.rs: add beginning of sieve parser

Manos Pitsidianakis referenced this issue from a commit

2022-12-27 18:29:42 +02:00

melib/sieve.rs: add beginning of sieve parser

Manos Pitsidianakis commented

2022-12-27 19:01:53 +02:00

There's been some preliminary work for a sieve parser here: 3c847ad26a/melib/src/sieve.rs

procmail syntax is too arcane and bad imho. I think sieve even though not novel has promise as established work.

I'm still thinking about how to integrate this on the client frontend...

There's been some preliminary work for a sieve parser here: https://git.meli.delivery/meli/meli/src/commit/3c847ad26afcc4a4cdcfbdbf70f35be57d0da1ab/melib/src/sieve.rs procmail syntax is too arcane and bad imho. I think sieve even though not novel has promise as established work. I'm still thinking about how to integrate this on the client frontend...

Manos Pitsidianakis commented

2022-12-27 22:20:31 +02:00

Tracking PR: https://git.meli.delivery/meli/meli/pulls/174

Manos Pitsidianakis added a new dependency 2022-12-27 22:20:41 +02:00

#209 WIP: Add Sieve parser and interpreter

Manos Pitsidianakis added the

Retired

label 2023-09-07 15:55:30 +03:00

Manos Pitsidianakis removed a dependency 2023-09-07 15:56:43 +03:00

#209 WIP: Add Sieve parser and interpreter

Manos Pitsidianakis closed this issue

2023-09-07 15:56:52 +03:00