Bug 901 - Razor reporting doesn't use razor_timeout setting
Summary: Razor reporting doesn't use razor_timeout setting
Status: RESOLVED FIXED
Alias: None
Product: Spamassassin
Classification: Unclassified
Component: Libraries (show other bugs)
Version: SVN Trunk (Latest Devel Version)
Hardware: Other other
: P2 normal
Target Milestone: ---
Assignee: SpamAssassin Developer Mailing List
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2002-09-16 08:42 UTC by Theo Van Dinter
Modified: 2002-09-20 13:30 UTC (History)
0 users


Attachment Type Modified Status Actions Submitter/CLA Status
look at the razor_timeout setting, not hardcoded. patch None Theo Van Dinter [HasCLA]
Patch to solve several problems with Razor, DCC, and Pyzor patch None Theo Van Dinter [HasCLA]
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Theo Van Dinter 2002-09-16 08:42:00 UTC 
I was just noticing that there's a hard coded 10 second alarm when reporting to
Razor, but lookups use a razor_timeout setting (which defaults to 10 seconds).
Comment 1 Theo Van Dinter 2002-09-16 08:51:14 UTC 
Created attachment 336 [details]
look at the razor_timeout setting, not hardcoded.
Comment 2 Theo Van Dinter 2002-09-17 08:03:05 UTC 
I'm finding a number of changes for Razor, DCC, and Pyzor, both checking and
reporting (mostly reporting).

So I'm going to include them all here, please don't apply the first patch yet.
Comment 3 Theo Van Dinter 2002-09-17 08:31:42 UTC 
Created attachment 339 [details]
Patch to solve several problems with Razor, DCC, and Pyzor
Comment 4 Theo Van Dinter 2002-09-17 08:38:05 UTC 
Ok, this second patch fixes a bunch of stuff:
 - razor_report now uses razor_timeout
 - Reporter.pm now uses the is_{dcc,pyzor}_available from Dns.pm, ported. 
Solves the same "{dccproc,pyzor}: command not found" bug when people use
"spamassassin -r".
 - {dcc,pyzor}_report now users {dcc,pyzor}_timeout and the configured
{dcc,pyzor}_path instead of `{dccproc,pyzor}`
 - Minor annoying bits in Dns.pm where is_{dcc,pyzor}_available had crufty bits
not displaying anything
 - Minor indentation issue in Reporter.pm

This should be applied with the patch for bug 900 (fixes Razor2 reporting
response value). :)
Comment 5 Duncan Findlay 2002-09-20 21:30:41 UTC 
committed