[Rtir] Modify RTIR_FindIP.pm script to filter IPs
Ruslan Zakirov
ruz at bestpractical.com
Mon Jun 25 16:01:27 EDT 2012
On Wed, Jun 20, 2012 at 1:55 AM, David Moreau Simard <dmsimard at iweb.com> wrote:
> etc/RT_SiteConfig.pm:
> Set(@RTIR_IPv4_range_preference, qw (
> 192.168.0.0/16
> 127.0.0.0/24
> )
> );
Feature is good to have, but to make it into RTIR core it should be
IPv6 aware. I don't see
much need in additional module, but it's ok. Code needs cleanup. It's
complex already and you
change makes it even more nested. Why don't you put the check into
AddIP method? I think
it belongs there.
--
Best regards, Ruslan.
More information about the Rtir
mailing list