Re^4: Create a ipset for blocking networks based on internet sources

Thanks jwkrahn, I really appreciate the corrections! I have included them in the script and read the section on error variables in perlvar.

Also, my first version used Moose (influenced by the book "Modern Perl"). After looking at the script, and comparing it with the bash scripts, it took more time and resources. I guess that Moose is for larger and more complex projects, so I have rewritten in what I assume is "procedural programming". It has been trivial to have Moose out, transforming the package into a subroutine.

One thing is puzzeling me is the regex. In a file in this format:

Start   End     Netblock        Attacks Name    Country email
116.45.99.0     116.45.99.255   24      1799                    
46.21.150.0     46.21.150.255   24      1708                    
121.243.146.0   121.243.146.255 24      1446
[download]

Applying this regex:

qr/^(\d{1,3}\.\d{1,3}\.\d{1,3}\.\d{1,3})/m

Gives me:

116.45.99.0
46.21.150.0
121.243.146.0
[download]

And this regex:

qr/^((\d{1,3}\.){3}\d{1,3})/m

Gimes me:

116.45.99.0
99.
46.21.150.0
150.
121.243.146.0
146.
[download]

I would expect the same result.

Comment on Re^4: Create a ipset for blocking networks based on internet sources Select or Download Code

Replies are listed 'Best First'.
Re^5: Create a ipset for blocking networks based on internet sources by jwkrahn (Abbot) on Apr 25, 2012 at 19:10 UTC
`qr/^((\d{1,3}\.){3}\d{1,3})/m` [download] The problem there is that you have two sets of capturing parentheses. You can change the inner capturing parentheses to non-capturing parentheses: `qr/^((?:\d{1,3}\.){3}\d{1,3})/m` [download] And you could change the foreach loop: `foreach ( $response->content =~ /$regex/g ) { @sys = (qw(ipset add), "temp_$set_name", $_); system(@sys) == 0 or die "Unable to add $_ to temp_$set_na +me because: $?"; }` [download] To a while loop that only accesses $1: `while ( $response->content =~ /$regex/g ) { @sys = (qw(ipset add), "temp_$set_name", $1); system(@sys) == 0 or die "Unable to add $_ to temp_$set_na +me because: $?"; }` [download]	[reply] [d/l] [select]
Re^6: Create a ipset for blocking networks based on internet sources by mimosinnet (Beadle) on Apr 27, 2012 at 17:23 UTC
Thanks for the tip on the use of the extended pattern (?: ) to not make backreferences! It has been also very enlightening to see the relationship between the foreach loop and the $_ variable, and what would be the equivalent of the while loop and the $1 bakcreference. Your second suggestion has made me explore more how the while loop works, realizing that while ( $response->content =~ /$regex/g ) gets the content each time that the loop is called. I have changed it by: `my $resp = $response->content; while ( $resp =~ /$regex/g ) { @sys = (qw(ipset add), "temp_$set_name", $1); }` [download] Your comments have been very useful! Although all these issues are on the documentation, it is sometimes difficult at the beginning to find the right section at the right time. It has been great to apply it to a specific example. Very greatful! Cheers!	[reply] [d/l]