Tickets :: Comment on [#8577] vacation rule : maildrop/mailbot locks

* Your Email Address
* Spam protection	Enter the letters below: .__ .__.. ..__ . . [__)\| \|\|_/ [ __\|__\| [__)\|__\|\| \[_./\| \|
Comment	> 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 \""; > > > > > >
Attachment
Watch this ticket