[Bps-public-commit] RT-BugTracker-Public branch, master, updated. 3f60f13d86329e1e1fc406aea8c4662b6a564955

[Kevin Falcone]

The branch, master has been updated
       via  3f60f13d86329e1e1fc406aea8c4662b6a564955 (commit)
      from  e9359b7167c293e5ecf0cba20914de10aa35003b (commit)

Summary of changes:
 .../BugTracker-Public/Ticket/Elements/ShowMessageHeaders/Default      | 4 ++++
 1 file changed, 4 insertions(+)

- Log -----------------------------------------------------------------
commit 3f60f13d86329e1e1fc406aea8c4662b6a564955
Author: Thomas Sibley <tsibley at cpan.org>
Date:   Thu Sep 12 21:42:39 2013 -0700

    Censor email addresses in mbox From_ headers too, which sometimes sneak in
    
    For example, see:
    
        https://rt.cpan.org/Ticket/Display.html?ShowHeaders=1;id=88631#txn-1262014
    
    (about half way down)

diff --git a/html/Callbacks/BugTracker-Public/Ticket/Elements/ShowMessageHeaders/Default b/html/Callbacks/BugTracker-Public/Ticket/Elements/ShowMessageHeaders/Default
index ce01940..33d10da 100644
--- a/html/Callbacks/BugTracker-Public/Ticket/Elements/ShowMessageHeaders/Default
+++ b/html/Callbacks/BugTracker-Public/Ticket/Elements/ShowMessageHeaders/Default
@@ -8,5 +8,9 @@ return unless RT::BugTracker::Public->IsPublicUser;
 for my $h (@$headers) {
     next if $h->{'Tag'} =~ /^(?:Subject)$/i;
     $h->{'Value'} =~ s/@/ [...] /g;
+
+    # Sometimes mbox From_ lines make it into RT headers
+    $h->{'Tag'} =~ s/@/ [...] /g
+        if $h->{'Tag'} =~ /^From\s+/;
 }
 </%init>

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