Am 13.02.2012 14:41, schrieb Paul J Stevens:
> On 02/13/2012 02:05 PM, Reindl Harald wrote:
>> Am 13.02.2012 13:46, schrieb Paul J Stevens:
>>> On 02/13/2012 01:30 PM, Reindl Harald wrote:
>>>> Incorrect datetime value: 'NOW()'
>>>
>>> Fixed in GIT now.
>>
>> partly confirmed
>>
>> but this did it not make it into dbmail3 :-(
>> https://github.com/pjstevns/dbmail/commit/f887d2e01c5a288fcd2f6ecb1639b6fb0dba2031
> 
> fixed now.
> 
>> did you make a look if there are other places where function
>> calls maybe quoted by the sql-wrapper independent of the
>> reply-feature?
> 
> I only checked for CURRENT_TIMESTAMP/NOW(), since the auto_reply
> function was the last change in that respect.
> 
> Cursory inspection shows all other places are handled as direct string
> insertion, rather than parameters on prepared statements.

confirmed for both cases (physical accout and alias)
so we have only the problem with ".cssclass" while forwarding remaining
and a small log error below, if weg get this cleaned dbmail3 is perfect
from my point of view

i know i am pedantic but one small bug in this context :-)

if a user has no reply defined there is still the error message
"error fetching reply body" in maillog, below mysql-query-log
for each received message......

Feb 13 14:54:23 rh dbmail/lmtpd[29891]: [0x208d280] Error:[message] 
execute_auto_ran(+2385): error fetching reply body

mysql> SELECT reply_body FROM dbmail_auto_replies WHERE user_idnr = 4 AND NOW() 
BETWEEN start_date AND stop_date;
Empty set (0.00 sec)

mysql> SELECT * FROM
dbmail_auto_replies;+-----------+---------------------+---------------------+-------------------------------------------------------------------------------------+
| user_idnr | start_date          | stop_date           | reply_body
                           |
+-----------+---------------------+---------------------+-------------------------------------------------------------------------------------+
|         8 | 2010-11-05 00:00:00 | 2010-11-06 23:59:00 | Test
                           |
|         9 | 2010-11-05 00:00:00 | 2010-11-06 23:59:00 | Test
                           |
|        14 | 2010-01-01 00:00:00 | 2020-12-31 23:59:00 | Auto-Reply Test dbmail
Ab Version 2.2.14 sollten Absender/Empfänger wieder passen  |
+-----------+---------------------+---------------------+-------------------------------------------------------------------------------------+
3 rows in set (0.00 sec)
__________________________________________________________________________________

                   59 Prepare   SELECT reply_body FROM dbmail_auto_replies 
WHERE user_idnr = ? AND NOW() BETWEEN start_date AND
stop_date
                   59 Execute   SELECT reply_body FROM dbmail_auto_replies 
WHERE user_idnr = 4 AND NOW() BETWEEN start_date AND
stop_date
                   59 Close stmt
                   59 Query     START TRANSACTION
                   59 Query     DELETE FROM dbmail_messages WHERE message_idnr 
= 8504
                    5 Query     insert into SystemEvents (Message, Facility, 
FromHost, Priority, DeviceReportedTime, ReceivedAt,
InfoUnitID, SysLogTag) values (' [0x208d280] Error:[message] 
execute_auto_ran(+2385): error fetching reply body',
2, 'rh', 3, '20120213145423', '20120213145423', 1, 'dbmail/lmtpd[29891]:')
                   59 Query     COMMIT



Attachment: signature.asc
Description: OpenPGP digital signature

_______________________________________________
DBmail mailing list
DBmail@dbmail.org
http://mailman.fastxs.nl/cgi-bin/mailman/listinfo/dbmail

Reply via email to