[fsck.com #3042] Re: [rt-users] comment or corresponedence size limit

[Phil Homewood]

Jesse Vincent wrote:
> Roughly 12k, There's a constant in html/Ticket/Elements/ShowTransaction,
> iirc. I'd welcome a patch to move that to the config file.

Attached, and submitted as [fsck.com #3042].
-- 
Phil Homewood, Systems Janitor, http://www.SnapGear.com
pdh at snapgear.com Ph: +61 7 3435 2810 Fx: +61 7 3891 3630
SnapGear - Custom Embedded Solutions and Security Appliances
-------------- next part --------------
Index: etc/RT_Config.pm.in
===================================================================
RCS file: /raid/tracking-cvs/rt.3.0/etc/RT_Config.pm.in,v
retrieving revision 1.5
diff -u -r1.5 RT_Config.pm.in
--- etc/RT_Config.pm.in	27 May 2003 17:22:28 -0000	1.5
+++ etc/RT_Config.pm.in	8 Jul 2003 22:49:06 -0000
@@ -310,6 +310,12 @@
 
 # Set($WebSessionClass , 'Apache::Session::File');
 
+# $MaxInlineBody is the maximum attachment size that we want to see
+# inline when viewing a transaction. 13456 is a random sane-sounding
+# default.
+
+Set($MaxInlineBody, 13456);
+
 # }}}
 
 # {{{ RT UTF-8 Settings
Index: html/Ticket/Elements/ShowTransaction
===================================================================
RCS file: /raid/tracking-cvs/rt.3.0/html/Ticket/Elements/ShowTransaction,v
retrieving revision 1.10
diff -u -r1.10 ShowTransaction
--- html/Ticket/Elements/ShowTransaction	2 Jul 2003 06:19:51 -0000	1.10
+++ html/Ticket/Elements/ShowTransaction	8 Jul 2003 22:49:06 -0000
@@ -53,7 +53,9 @@
 	   eval {$headers =~ s/^([^:]+)(?=:)/loc($1)/em; } # we eval here to catch errors when 5.6 panics
       }
      # 13456 is a random # of about the biggest size we want to see inline text
-     my $MAX_INLINE_BODY = 13456;
+     # It's here to catch anyone who hasn't updated RT_Config.pm since this
+     # constant was moved out there.
+     my $MAX_INLINE_BODY = $RT::MaxInlineBody || 13456;
      if ($message->ContentType =~ m{^(text/plain|message|text$)}i && 
 				    $message->ContentLength < $MAX_INLINE_BODY ) {
          require Text::Quoted;