Can not release email from SQL quarantine with new stable version 2.9.1

[Stef Simoens]

Hi,

I had the same problem with 2.9.0

See my post on http://lists.amavis.org/pipermail/amavis-users/2014-June/002957.html for a patch.
Works in my 2.9.0 installation.

As this has not been addressed yet, I'm holding off upgrading (and I didn't re-patch bc the updates to 2.9.0 I considered minimal for our useage).

Kr,
--
Stef Simoens
Verstuurd vanaf mijn mobiele telefoon

> Op 18-jul.-2014 om 16:57 heeft Gionatan Danti <g.danti at assyoma.it> het volgende geschreven:
> 
> Hi all,
> I just discovered a problem with the latest stable release 2.9.1 (from EPEL repository). With this version, I am unable to release an email from SQL quarantine (with previous 2.8.x is all ok).
> 
> When executing amavisd-release, I had the following output:
> [root at spambox ~]# amavisd-release 0-vFmNRKpWLO5K
> 450 4.5.0 Failure: Can't open sql obj for reading: No such file or directory at (eval 109) line 375, <GEN21> line 4.
> 
> See below for details [1].
> 
> I tracked down the reason the a change in line 20177 of /usr/sbin/amavisd. First line is the new version (2.9.1), second line is my correction:
> 
> 20177c20177
> <     $msginfo->mail_id(scalar generate_mail_id());
> ---
> >     $msginfo->mail_id($mail_id);
> 
> As you can see, in the new version the $msginfo->mail_id is loaded with a newly-generated mail_id instead of the one passed by amavisd-release. Applying the above modification, all is fine.
> 
> I am missing something?
> Regards.
> 
> [1]
> Log file (log_level=5):
> Jul 18 16:37:34 spambox amavis[3577]: policy protocol: mail_file=0-vFmNRKpWLO5K
> ...
> Jul 18 16:37:34 spambox amavis[3577]: (rel-JOJ8MmeVGMwi) preprocess_policy_query: opening in sql: JOJ8MmeVGMwi
> Jul 18 16:37:34 spambox amavis[3577]: (rel-JOJ8MmeVGMwi) preprocess_policy_query: missing partition_tag in request, fetching msgs record for mail_id=JOJ8MmeVGMwi
> Jul 18 16:37:34 spambox amavis[3577]: (rel-JOJ8MmeVGMwi) sql begin, nontransaction
> Jul 18 16:37:34 spambox amavis[3577]: (rel-JOJ8MmeVGMwi) Connecting to SQL database server
> Jul 18 16:37:34 spambox amavis[3577]: (rel-JOJ8MmeVGMwi) connect_to_sql: trying 'DBI:mysql:database=amavis;host=localhost'
> Jul 18 16:37:34 spambox amavis[3577]: (rel-JOJ8MmeVGMwi) connect_to_sql: 'DBI:mysql:database=amavis;host=localhost' succeeded
> Jul 18 16:37:34 spambox amavis[3577]: (rel-JOJ8MmeVGMwi) sql: preparing and executing (1 args): SELECT partition_tag FROM msgs WHERE mail_id=?
> Jul 18 16:37:34 spambox amavis[3577]: (rel-JOJ8MmeVGMwi) release: no records with msgs.mail_id=JOJ8MmeVGMwi in a database, trying to read from a quar. anyway
> Jul 18 16:37:34 spambox amavis[3577]: (rel-JOJ8MmeVGMwi) release: opening mail_id=JOJ8MmeVGMwi, partition_tag=
> Jul 18 16:37:34 spambox amavis[3577]: (rel-JOJ8MmeVGMwi) sql begin, nontransaction
> Jul 18 16:37:34 spambox amavis[3577]: (rel-JOJ8MmeVGMwi) sql: preparing and executing (2 args): SELECT mail_text FROM quarantine WHERE partition_tag=? AND mail_id=? ORDER BY chunk_ind
> Jul 18 16:37:34 spambox amavis[3577]: (rel-JOJ8MmeVGMwi) Amavis::IO::SQL::open r drv=mysql (SELECT mail_text FROM quarantine WHERE partition_tag=? AND mail_id=? ORDER BY chunk_ind); key=JOJ8MmeVGMwi, p_tag=, s:
> Jul 18 16:37:34 spambox amavis[3577]: (rel-JOJ8MmeVGMwi) Amavis::IO::SQL::open key=JOJ8MmeVGMwi, p_tag=: no such record
> 
> NOTE: as you can see, amavisd-releaes passes the correct mail_id (first log line), but this correct value is then overwritten by the newly-generated one (by function generate_mail_id()).
> 
> -- 
> Danti Gionatan
> Supporto Tecnico
> Assyoma S.r.l. - www.assyoma.it
> email: g.danti at assyoma.it - info at assyoma.it
> GPG public key ID: FF5F32A8