==================================================================
  Please DO NOT REPLY to this mail or send email to the developers
  about this bug. Please follow-up to Bugzilla using this link:
    https://bugs.contribs.org/show_bug.cgi?id=9428

  Have you checked the Frequently Asked Questions (FAQ)?
    http://wiki.contribs.org/SME_Server:Documentation:FAQ

  Please also take the time to read the following useful guide:
    http://www.chiark.greenend.org.uk/~sgtatham/bugs.html
==================================================================

            Bug ID: 9428
           Summary: code improving for unknown plugins
    Classification: Contribs
           Product: SME Contribs
           Version: 9.1
          Hardware: ---
                OS: ---
            Status: CONFIRMED
          Severity: normal
          Priority: P3
         Component: smeserver-mailstats
          Assignee: [email protected]
          Reporter: [email protected]
        QA Contact: [email protected]

Currently when someone add a qpsmtpd plugin that is not handled by the
mailstats script it will return one error per line found :

Unexpected failure string in log file: require_resolvable_client
[...]
Unexpected failure string in log file: require_resolvable_client


it will also count this plugin in the qpsmtpd table
Qpsmtpd codes league table:
---------------------------------------------
Count    Percent    Reason    
---------------------------------------------
145    33.1%    check_badcountries
97    22.1%    require_resolvable_client


but as I understand it will not count it in the general table as it does not
know where to categorize it.

I would suggest to catch them with a else statement and to add them either to
non conf. either to an unknown categories.

I would also suggest  in both situation tag the unknown plugins either in the
qpsmtpd table with a * after their name, either as a string after the table :
"the following unknown plugins have been put by default in cat 'non conf.' : 
require_resolvable_client, ...."
This way we will avoid the cron error mail, and reader will still be aware that
he could report the plugin to categorize it.

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
_______________________________________________
Mail for each SME Contribs bug report
To unsubscribe, e-mail [email protected]
Searchable archive at https://lists.contribs.org/mailman/public/contribteam/

Reply via email to