$isPg undefined in send-quarantine-digests.pl

[dmorton]

looks like a line got removed from send-quarantine-digests.pl in [977] that is critical:

my $isPg = $dsn =~ /^dbi:Pg/; 

[dmorton]

Addition from Marc Fournier:

needs to be

my $isPg = $dsn =~ /^dbi:Pg/i;

I believe ... I have mine set as DBI:Pg ...

but, then the next error:

wm# /var/amavis/maia/scripts/send-quarantine-digests.pl
DBD::Pg::st execute failed: ERROR:  syntax error at or near "$3" at character 109
2006-07-23 22:24:31 Maia: [send-quarantine-digests] FATAL ERROR: Couldn't execute query: ERROR:  syntax error at or near "$3" at character 109 

[rjl]

Use the get_db_type() subroutine used in many of the other Maia scripts, which queries the DBI driver name directly, rather than trying to fish it out of the DSN string:

# Determine the database type: "mysql" or "pg"
sub get_db_type($) {
   my($dbh) = @_;

   return $dbh->{Driver}->{Name};
}

[dmorton]

used get_db_type, [1089]

[dmorton]

I have a report that it should check for "Pg" not "pg", and then not sure about the second half of that error...

[dmorton]

well, duh, the original error was about the case sensitivity.

Still trying to track down the $3 error, which seems to be in the confirm token generation query.

[dmorton]

postgresql doesnt like the INTERVAL ? DAYS part... it need to be INTERVAL ? and the days added to the variable passed in... or else some other date arithmetic needs to be done.

[dmorton]

I think we got this sorted out in [1112] and [1113]. Thanks to riogeed on IRC for testing. We settled on the date arithmetic in order to keep the parameter constant.