[Rt-commit] rt branch, 4.2/quiet-internal-formatter-message, created. rt-4.2.10-186-gfb8e303

Jesse Vincent jesse at bestpractical.com
Sat Mar 7 16:47:58 EST 2015


The branch, 4.2/quiet-internal-formatter-message has been created
        at  fb8e3034fa03e06818c95fb7d1e800c414c4d624 (commit)

- Log -----------------------------------------------------------------
commit fb8e3034fa03e06818c95fb7d1e800c414c4d624
Author: Jesse Vincent <jesse at bestpractical.com>
Date:   Sat Mar 7 12:48:17 2015 -0800

    Drop the "internal formatter" message from info to debug.
    
    An informational message added in 40ba502 appears rather frequently
    in normal RT operation (once per commandline script in some cases)
    and isn't relevant to audit logs or directly actionable.
    
    If this is important enough to log about on every start of the
    server and every commandline script, we may want to consider making
    it a warning and recommending corrective action in the text.

diff --git a/lib/RT/Interface/Email.pm b/lib/RT/Interface/Email.pm
index 4fdae1b..b244f1d 100644
--- a/lib/RT/Interface/Email.pm
+++ b/lib/RT/Interface/Email.pm
@@ -1835,7 +1835,7 @@ sub _HTMLFormatter {
     # Always fall back to core, even if it is not listed
     for my $prog (@order) {
         if ($prog eq "core") {
-            RT->Logger->info("Using internal Perl HTML -> text conversion");
+            RT->Logger->debug("Using internal Perl HTML -> text conversion");
             require HTML::FormatText::WithLinks::AndTables;
             $formatter = \&_HTMLFormatText;
         } else {

-----------------------------------------------------------------------


More information about the rt-commit mailing list