<div dir="ltr">Thanks for your reply.  The only thing I added was the 'next; # Do not add IPv4 ranges to tickets' line.  <div><br></div><div><div>--- /opt/rt4/local/plugins/RT-IR/lib/RT/Action/RTIR_FindIP.pm.ORIG<span class="" style="white-space:pre">     </span>2013-05-24 15:31:09.356845837 -0400</div>

<div>+++ /opt/rt4/local/plugins/RT-IR/lib/RT/Action/RTIR_FindIP.pm<span class="" style="white-space:pre">       </span>2013-05-28 17:00:25.764303577 -0400</div><div>@@ -87,6 +87,7 @@</div><div>             );</div><div>         }</div>

<div>         elsif ( $4 && defined $5 ) { # IPv4/mask</div><div>+            next; # Do not add IPv4 ranges to tickets</div><div>             my $cidr = join( '.', map $_||0, (split /\./, $4)[0..3] ) ."/$5";</div>

<div>             my $range = (Net::CIDR::cidr2range( $cidr ))[0] or next;</div><div>             $spots_left -= $self->AddIP(</div></div><div><br></div><div style>In this case where would this modified RTIR_FindIP.pm go and properly supersede the original?  /opt/rt4/local/lib or /opt/rt4/local/plugins/RT-IR/local/lib?</div>

</div><div class="gmail_extra"><br><br><div class="gmail_quote">On 28 May 2013 13:49, Kevin Falcone <span dir="ltr"><<a href="mailto:falcone@bestpractical.com" target="_blank">falcone@bestpractical.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On Fri, May 24, 2013 at 02:11:58PM -0700, Landon wrote:<br>
>    I'm wondering if this is the best way to stop RTIR from detecting CIDR ranges. I've been<br>
>    looking into /opt/rt4/local/plugins/RT-IR/lib/RT/Action/RTIR_FindIP.pm and modified the<br>
>    following section:<br>
>    elsif ( $4 && defined $5 ) { # IPv4/mask<br>
>    next; # Do not add IPv4 ranges to tickets<br>
>    my $cidr = join( '.', map $_||0, (split /\./, $4)[0..3] ) ."/$5";<br>
>    my $range = (Net::CIDR::cidr2range( $cidr ))[0] or next;<br>
>    $spots_left -= $self->AddIP(<br>
>    IP => $range, CustomField => $cf, Skip => \%existing<br>
>    );<br>
>    }<br>
>    Is this the best way? Or is there something else I should have done instead?<br>
<br>
</div></div>It's almost impossible for me to parse your changes since the<br>
whitespace has been completely lost.<br>
<br>
Can you instead post a unified diff (diff -u) of your change?<br>
<br>
Also, if you're going to change RT/extension code, you should be using<br>
local/lib instead of hacking in place.  Otherwise upgrading will be...<br>
fun.<br>
<span class="HOEnZb"><font color="#888888"><br>
-kevin<br>
</font></span><br>_______________________________________________<br>
Rtir mailing list<br>
<a href="mailto:Rtir@lists.bestpractical.com">Rtir@lists.bestpractical.com</a><br>
<a href="http://lists.bestpractical.com/cgi-bin/mailman/listinfo/rtir" target="_blank">http://lists.bestpractical.com/cgi-bin/mailman/listinfo/rtir</a><br>
<br></blockquote></div><br><br clear="all"><div><br></div>-- <br><div dir="ltr"><div><br></div><span style="font-family:arial;font-size:small">Landon Stewart :: <a href="mailto:lstewart@iweb.com" target="_blank">lstewart@iweb.com</a></span><br style="font-family:arial;font-size:small">

<span style="font-family:arial;font-size:small">Lead Specialist, Abuse and Security Management</span><br style="font-family:arial;font-size:small"><span style="font-family:arial;font-size:small">Spécialiste principal, gestion des abus et sécurité</span><br style="font-family:arial;font-size:small">

<span style="font-family:arial;font-size:small"><a href="http://iweb.com" target="_blank">http://iweb.com</a> :: +1 (888) 909-4932</span><br></div>
</div>