cyrusimap / cyrus-imapd

Cyrus IMAP is an email, contacts and calendar server
http://cyrusimap.org
Other
544 stars 149 forks source link

sieve uses mailbox name in internal format in autoreply #1356

Open brong opened 13 years ago

brong commented 13 years ago

From: Deniss Gaplevsky Bugzilla-Id: 3452 Version: 2.4.12 Owner: Bron Gondwana

brong commented 13 years ago

From: Deniss Gaplevsky

Sieve uses internal mailbox name (no dots, no domain part for defaultdomain) when generating auto reply for "reject" rule: "Final-Recipient: rfc822; frx^123"

Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit

Your message was automatically rejected by Sieve, a mail filtering language.

The following reason was given: аЏ аБаОаЛб�б�аЕ аНаЕ аНаАаМаЕб�аЕаН аПаОаЛб�б�аАб�б� аОб� а�аАб� б�аОаОаБб�аЕаНаИб�

--26365/someserver Content-Type: message/disposition-notification

Reporting-UA: someserver; Cyrus v2.3/CMU Sieve 2.3 Final-Recipient: rfc822; frx^123 Original-Message-ID: <1303242035.4dade5334139e@mail.inbox.lv> Disposition: automatic-action/MDN-sent-automatically; deleted

brong commented 13 years ago

From: Jeroen van Meeuwen (Kolab Systems)

This bug is being closed as part of a cleanup exercise.

This bug was associated with milestone 2.3-next, but only security fixes and serious bug-fixes are eligible for inclusion in 2.3.

Should the problem need to be resolved nonetheless, please reopen this bug and set the version to the appropriate 2.4 or 2.5-next version in which you can verify the problem still exists.

brong commented 13 years ago

From: Deniss Gaplevsky

same problem with 2.4.12:

  1. mailbox in internal format - '^' instead of '.'
  2. no defaultdomain appended
brong commented 12 years ago

Attachment-Id: 1450 From: Bron Gondwana Type: text/plain File: patch_sieve.txt

patch to convert username to external format

brong commented 12 years ago

From: Bron Gondwana

Well, since nobody is going to test it otherwise ;)

In 2.4

brong commented 12 years ago

From: Jeroen van Meeuwen (Kolab Systems)

To be included in release 2.4.14 - adjusting milestone.

brong commented 12 years ago

From: Jeroen van Meeuwen (Kolab Systems)

Closing ticket

brong commented 12 years ago

From: Deniss Gaplevsky

Looks like patch brings regression:

Mar 21 00:13:12 server lmtp[9457]: sieve runtime error for jolanta.aknc id <1332281591.4f6900f7b3add@server>: Keep: Mailbox does not exist

to put message to mailbox sieve requires cyrus sd->username in path ?

brong commented 12 years ago

From: Greg Banks

We have Bug 3664 open for regression introduced by this commit.

brong commented 12 years ago

From: Bron Gondwana

I've reverted this change. Will look into a better fix later, but broken is much more serious than minor display issues...

brong commented 12 years ago

From: Jeroen van Meeuwen (Kolab Systems)

This one now belongs on the 2.4-next heap.

brong commented 12 years ago

From: Jeroen van Meeuwen (Kolab Systems)

Sorry, it's supposed to be milestone 2.4.15

brong commented 12 years ago

From: Jeroen van Meeuwen (Kolab Systems)

Needs to be post-poned to a later 2.4 release