Sun, 05 Apr 2026 20:44:13 +0000 https://bugs.horde.org/ticket/8577 vacation rule : maildrop/mailbot locks mailbot uses a lock before opening sender's email database. the database file is defined with arg '-d '. In file ingo/lib/Script/maildrop.php:523, database filename is defined as 'vacation'. Therefore mailbot uses file 'vacation.lock' as the file lock , which is the same file lock defined for maildrop (line 516). so, if a user set start and end vacation dates, maildrop and mailbot will try to lock the same file which will never work. here is a patch: - $this->_action[] = " cc \"| mailbot -D " . $params['action-value']['days'] . " -c '" . NLS::getCharset() . "' -t \$HOME/vacation.msg -d \$HOME/vacation -A 'From: $from' -s '" . MIME::encode($params['action-value']['subject'], NLS::getCharset()) . "' /usr/sbin/sendmail -t \""; + $this->_action[] = " cc \"| mailbot -D " . $params['action-value']['days'] . " -c '" . NLS::getCharset() . "' -t \$HOME/vacation.msg -d \$HOME/vacationbot -A 'From: $from' -s '" . MIME::encode($params['action-value']['subject'], NLS::getCharset()) . "' /usr/sbin/sendmail -t \""; Tue, 15 Sep 2009 11:36:03 +0000 https://bugs.horde.org/ticket/8577#t55746 To not destroy the seen-database if Ingo is being updated while auto-responses are active, wouldn't it make more sense to rename the vacation lock file instead? I don't follow the maildrop code completely, but isn't the vacation.lock created by flock only when processing the current message? That wouldn't break anything if upgrading Ingo on an active system. Tue, 15 Sep 2009 15:11:04 +0000 https://bugs.horde.org/ticket/8577#t55748 > To not destroy the seen-database if Ingo is being updated while > auto-responses are active, wouldn't it make more sense to rename the > vacation lock file instead? I don't follow the maildrop code > completely, but isn't the vacation.lock created by flock only when > processing the current message? That wouldn't break anything if > upgrading Ingo on an active system. yes, changing maildrop's file lock will prevent live/production system from loosing user's db history. So I guess the path would be (or some other file name!) : - $this->_action[] = ' flock "vacation.lock" {'; + $this->_action[] = ' flock "vacationdrop.lock" {'; Tue, 15 Sep 2009 15:26:27 +0000 https://bugs.horde.org/ticket/8577#t55750 Changes have been made in Git for this ticket: Use a lock name that's different from mailbot's seen database name (Bug #8577). http://git.horde.org/diff.php/ingo/lib/Script/Maildrop.php?rt=horde-hatchery&r1=9b22223f642b6ab65a71694ff34f709f06dfce9d&r2=5a978e4ae04ac8ffa7b04046f91416d7aa347d01 Tue, 15 Sep 2009 21:00:39 +0000 https://bugs.horde.org/ticket/8577#t55753 Thanks for the confirmation. Tue, 15 Sep 2009 21:01:12 +0000 https://bugs.horde.org/ticket/8577#t55754 Changes have been made in CVS for this ticket: MFH: Use a lock name that's different from mailbot's seen database name (Bug #8577). http://cvs.horde.org/diff.php/ingo/docs/CHANGES?rt=horde&r1=1.55.2.126&r2=1.55.2.127&ty=u http://cvs.horde.org/diff.php/ingo/lib/Script/maildrop.php?rt=horde&r1=1.7.2.14&r2=1.7.2.15&ty=u Tue, 15 Sep 2009 21:09:28 +0000 https://bugs.horde.org/ticket/8577#t55755 Changes have been made in Git for this ticket: Use a lock name that's different from mailbot's seen database name (Bug #8577). http://git.horde.org/diff.php/ingo/lib/Script/Maildrop.php?rt=horde-git&r1=9b22223f642b6ab65a71694ff34f709f06dfce9d&r2=5a978e4ae04ac8ffa7b04046f91416d7aa347d01 Wed, 13 Jan 2010 00:11:21 +0000 https://bugs.horde.org/ticket/8577#t57612