[IPCop-devel] Re: [IPCop-cvs] ipcop/config/cfgroot header.pl,1.34.2.54,1.34.2.55

[Franck B. <fr...@fr...>]

Le Samedi 30 Avril 2005 08:32, vous avez =E9crit :
> on 30/4/05 1:25 am, Franck Bourdonnec at fra...@us...
>
> wrote:
> > Update of /cvsroot/ipcop/ipcop/config/cfgroot
> > In directory
> > sc8-pr-cvs1.sourceforge.net:/tmp/cvs-serv13651/config/cfgroot
> >
> > Modified Files:
> > Tag: IPCOP_v1_4_0
> > header.pl
> > Log Message:
> > Add a new cgi page under firewall tab, designed to receive future reque=
st
> > about specificcaly firewall options.
> > Example: disable ping.
> > This is only the GUI part, nothing is done yet.
> > (need to create /var/ipcop/optionsfw/settings file to work)
> >
> >
> > Index: header.pl
>
> Hold on Franck, is this too big a change, just before v1.4.6 is released?
>
> We also have to consider how we _patch_ header.pl, as we were going to st=
op
> replacing it completely, which might break users addons.
>
> Can I suggest you hold it until v1.4.7, to give us some time?
>
> Regards.
>
> Eric
No problem,

If it's too early I remove all or we can just remove the entry menu=20
(headder.lp) and let the file 'optionsfw' and classic stuff for 'settings'=
=20
ready for next release. It won't hurt (absent from .iso).
I dont think a special page especially from 'firewalling' is bad isn'it ?

The next step in using variable is module  rc.firewall:
=2Deval readhash /var/ipcop/optionsfw/settings   (nothing new here)
=2Dif 'SKIP-PING eq YES'
		iptables -A INPUT icpmt-type-8    (original line not touched)
=2Dfi

Not really a big issue, we know ipcop is ok whitout ping enabled , see numb=
er=20
of message speaking of this.

Tell me if I remove entry in header.pl for 1.4.6,=20

I want to continue to update rc.firewall with my addition commented, to that
everybody can vote on accept or not (or see a bug) befoer enabling.



=46ranck