[GH-ISSUE #507] Modoboa 1.1.0, postfix autoreply not working #488

New issue

Closed

opened 2026-02-27 11:12:01 +03:00 by kerem · 1 comment

kerem commented

2026-02-27 11:12:01 +03:00

Owner

Originally created by @virtuaris on GitHub (Jan 6, 2014).
Original GitHub issue: https://github.com/modoboa/modoboa/issues/507

hi tonioo,

after a fresh install of modoboa 1.1.0 i noticed that the autoreply feature is not working; everything is configured as it should be;
executing ''manage.py autoreply'' as postfix autoreply transport user (vmail) returns no error but no mail is sent;
so i checked the extension command file (modoboa/extensions/postfix_autoreply/management/commands/autoreply.py) and was puzzled by the following lines (15,):
if armessage.fromdate < timezone.now(): return
literally this says to me "if the valid-from timestamp of the autoreply message is in the past, do nothing";
imho this should be
if armessage.fromdate > timezone.now(): return
(at least this works for me)
or do i missunderstand your intentions?

bests

harald

Originally created by @virtuaris on GitHub (Jan 6, 2014). Original GitHub issue: https://github.com/modoboa/modoboa/issues/507 hi tonioo, after a fresh install of modoboa 1.1.0 i noticed that the autoreply feature is not working; everything is configured as it should be; executing ''manage.py autoreply'' as postfix autoreply transport user (vmail) returns no error but no mail is sent; so i checked the extension command file (modoboa/extensions/postfix_autoreply/management/commands/autoreply.py) and was puzzled by the following lines (15,): `if armessage.fromdate < timezone.now(): return` literally this says to me "if the valid-from timestamp of the autoreply message is in the past, do nothing"; imho this should be `if armessage.fromdate > timezone.now(): return` (at least this works for me) or do i missunderstand your intentions? bests harald

kerem

2026-02-27 11:12:01 +03:00

closed this issue
added the
bug
label

kerem commented

2026-02-27 11:12:02 +03:00

Author

Owner

@tonioo commented on GitHub (Jan 6, 2014):

Hi Harald,

it seems you're right, the condition is wrong. This part is pretty tricky to unit test, I need to find a way.

@tonioo commented on GitHub (Jan 6, 2014): Hi Harald, it seems you're right, the condition is wrong. This part is pretty tricky to unit test, I need to find a way.

kerem referenced this issue

2026-02-27 11:20:44 +03:00

[PR #930] [MERGED] Upgrade mode. #2063