https://bz.apache.org/SpamAssassin/show_bug.cgi?id=8448
Bug ID: 8448
Summary: t/bayessql.t fails with Redis Bayes legacy setting
Product: Spamassassin
Version: SVN Trunk (Latest Devel Version)
Hardware: All
OS: All
Status: NEW
Severity: normal
Priority: P2
Component: Learner
Assignee: [email protected]
Reporter: [email protected]
Target Milestone: Undefined
Created attachment 6139
--> https://bz.apache.org/SpamAssassin/attachment.cgi?id=6139&action=edit
let the database in bayes_sql_dsn apply when bayes_redis_database is not set
t/bayessql.t fails 3 tests when t/config has
run_bayes_sql_tests=y
bayes_store_module=Mail::SpamAssassin::BayesStore::Redis
bayes_sql_dsn=server=127.0.0.1:6379;database=2
The Bayes data is written to Redis database 0 instead of 2, so the test finds
database 2 empty after clearing it.
Cause: Since 4.0.2 the default value 0 of bayes_redis_database overrides the
database from bayes_sql_dsn. Bug 8362 fixed the same problem for the password.
The attached patch removes the default of bayes_redis_database. When it is not
set, the database from bayes_sql_dsn is used, or 0 if none is given, as before.
--
You are receiving this mail because:
You are the assignee for the bug.
