c3surf: feedback from the fli4l-Team


1.
- dein OPT ist falsch gepackt - man kann es ohne umkopieren nicht direkt in die fli4l-Struktur entpacken.
+ merke ich mir und mache es richtig


2.
- dadurch das du Quasi die "ganze Dokumentation" mit in die config gepackt hast, ist die config total unübersichtlich.
+ 26.05.2008: config Datei wurde abgespeckt und um "read the documentation" ergänzt. Noch im text Format.


3.
- im rc-script benutzt du die Funktion "add_to_crontab" jedoch definierst du im ext-check _keine_ Abhängigkeit zum Paket easycron.
+ 26.05.2008: in Version 2.0.1 erledigt


4.
- Zur besseren Uebersicht wuerde ich befuerworten, wenn die
die Chains ueber eine extra Indirektion einbindest, also quasi:

Input
  prot:tcp -> c3surf_block_ports

c3surf_block_ports
  prot:tcp port c3surf_block

Wenn es nur einen Port gibt, kann man die extra indirektion ja
weglasen.

+ Es wurden zwei chains eingefügt, damit erfolgt die Port-Sperre jetzt immer über indirektion.


5.
- Die Kommentare machen das Einbinden der Chains
  unuebersichtlich. Lieber einen Kommentar am Anfang des Scripts, das
  quasi erklaert, warum man sich an den Anfang der Chain setzt und
  nicht mittenrein und dann nicht den Code mit Kommentaren
  "Verschmutzen"

+ Kommentare bei den Chains entfernt


6.
- state:NEW,ESTABLISHED,RELATED,INVALID - gibts noch mehr states?
+ glaube nicht, ich werde es weglassen. Erledigt.

7.
- while schleifen vielleicht durch seq ersetzen, wie in den anderen
  scripten
+ releaseplanung: bei nächstem major change überprüfe ich wo das while Konstrukt sinnvoller durch seq ersetzt werden sollte. Will ja nicht gemein sein, aber dann solltet Ihr Euch Euere Templates einemal ansehen :-). Ist zudem eine Programmierstil-Frage, habe es daher zurückgestellt.


8.
- aufeinanderfolgende echos vielleicht durch 
cat <<EOF
EOF
+ jup, das ist Anfängerpech, ich dachte immer ich brauche unbedingt das echo für das Ersetzten der "$Variablen" durch ihren Inhalt. Ist ein Merkposten für Überarbeitungen.

