public inbox for development@lists.ipfire.org
 help / color / mirror / Atom feed
From: Stefan Schantl <stefan.schantl@ipfire.org>
To: development@lists.ipfire.org
Subject: Re: ipblacklist V2
Date: Sat, 05 Mar 2022 19:52:04 +0100	[thread overview]
Message-ID: <e14a83280a4f0ba003c6f4a285c4deba012c35f7.camel@ipfire.org> (raw)
In-Reply-To: <ef8ac1dcde46b22207dde653d6717a95d2a737e7.camel@ipfire.org>

[-- Attachment #1: Type: text/plain, Size: 3626 bytes --]

Hello *,

I've made some development progress, which I want to share here:

Most parts of the main backend script ("ipblacklist") from Tim and Rob
are ported into a new functions library (ipblocklist-functions.pl) and
into the main firewall script (rules.pl).

This process is almost finished and currently allows to create the
firewall rules, download the blocklists and to convert them into an
ipset compatible format.

Next step will be to import the frontend code (WUI) and adjust it to
use the backend code (functions) from the "ipblocklist-functions.pl".

At this time the blocklist feature should be in a use-able state again
and I'll go to create an automatic update script and to import all the
logging pages stuff etc.

The development progress and single commits can be found here:

https://git.ipfire.org/?p=people/stevee/ipfire-2.x.git;a=shortlog;h=refs/heads/ipblocklist

As usual please feel free to ask any questions or to share your opinion
here.

I wish you a nice day,

-Stefan
> Hello Rob, Hello Tim, Hello *,
> 
> as anounced on this list, I'm currently working on getting the
> ipblacklist feature as a core component into IPFire.
> 
> I already had a look on the code, which looks nice and very clean to
> me. As I'm currently also working on getting all ipset related set
> stuff and rule creation under one hood, this perfectly fits to this.
> 
> So my idea to put the ipblacklist feature over the line, was to split
> some parts of the ipblacklist "main script" (especially the ipset and
> iptables related stuff) into the perl-based script which is
> responsible
> for iptables rule creation.
> 
> In this case some other parts of the script (which where necessary in
> the past, because ipblacklist initial has been designed as an addon)
> also can be stipped.
> 
> Affected parts for example would be the "start", "stop", "enable" and
> "disable" code, which is not longer required and therefore safely can
> be dropped.
> 
> In the very end the main task for the script would be to download,
> update, convert and store the blacklists into an ipset compatible
> format.
> 
> Apart from this, I currently do not see any bigger changes for the
> WUI
> related stuff.
> 
> @Tim: I hope these changes are okay for you.
> 
> Getting started, I noticed, that there currently are two git
> repositories available, which contain the source for ipblacklist.
> 
> There is the origin one from Tim and a slightly modified (fixed) v3
> version from Rob. I' currently trying to determine, which one would
> be
> the best to start from - are there any deeper changes/differences
> between them?
> 
> Please feel free to ask any kind of questions or share you opinion.
> As
> usual, I'll share any progress here.
> 
> Best regards,
> 
> -Stefan 
> > Hi.
> > 
> > I have been looking at Tim FitzGeorge's code for ipblacklist v2 on 
> > https://patchwork.ipfire.org/project/ipfire/list/?series=1215 to
> > see
> > if I 
> > can help progress its incorporation into IPFire.
> > After I extracted the programs from Patchwork I have been able to
> > build them 
> > into my firewall where they are running very successfully.
> > The code on the server seems to be in good shape and apart from a
> > few
> > small 
> > patches and additions of a few missing scripts I think it could be 
> > successfully introduced into the IPFire code base.
> > I am more than happy to help in seeing this process carried out but
> > need to 
> > know if this is acceptable to yourselves.
> > 
> > Regards
> > Rob Brewer
> 
> 



  parent reply	other threads:[~2022-03-05 18:52 UTC|newest]

Thread overview: 26+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <ef8ac1dcde46b22207dde653d6717a95d2a737e7.camel@ipfire.org>
2022-03-01 13:13 ` Michael Tremer
2022-03-01 16:08 ` Rob Brewer
2022-03-05 18:52 ` Stefan Schantl [this message]
2022-03-05 21:46   ` Rob Brewer
2022-03-07 20:39     ` Michael Tremer
2022-03-07 22:54       ` Rob Brewer
2022-03-08 10:59         ` Rob Brewer
2022-03-08 15:45           ` Michael Tremer
2022-04-03  9:16           ` Stefan Schantl
2022-04-03 21:09             ` Rob Brewer
2022-02-04 18:17 Rob Brewer
2022-02-07 10:31 ` Michael Tremer
2022-02-07 12:58   ` Rob Brewer
2022-02-09 14:38     ` Adolf Belka
2022-02-09 21:43       ` Rob Brewer
2022-02-09 22:14         ` Adolf Belka
2022-02-10  9:45           ` Michael Tremer
2022-02-09 13:23   ` Rob Brewer
2022-02-09 14:29     ` Adolf Belka
2022-02-10  9:41     ` Michael Tremer
2022-02-10 15:12       ` Rob Brewer
2022-02-10 16:48         ` Michael Tremer
2022-02-12 21:29           ` Tim FitzGeorge
2022-02-13 12:44             ` Rob Brewer
2022-02-15 12:58               ` Michael Tremer
2022-02-15 12:54             ` Michael Tremer

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=e14a83280a4f0ba003c6f4a285c4deba012c35f7.camel@ipfire.org \
    --to=stefan.schantl@ipfire.org \
    --cc=development@lists.ipfire.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox