<p dir="ltr">It's probably a negligible difference in overhead and so forth, but it might be nice to only load the module if FTP rules actually exist.  I, for one, never plan to support FTP in particular on my servers.  Maybe a future optimization, at least?</p>

<div class="gmail_quote">On May 19, 2014 3:52 AM, "Alexandre DERUMIER" <<a href="mailto:aderumier@odiso.com">aderumier@odiso.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
ok, I'll send a patch this afternoon<br>
----- Mail original -----<br>
<br>
De: "Dietmar Maurer" <<a href="mailto:dietmar@proxmox.com">dietmar@proxmox.com</a>><br>
À: "Alexandre DERUMIER" <<a href="mailto:aderumier@odiso.com">aderumier@odiso.com</a>><br>
Cc: "pve-devel" <<a href="mailto:pve-devel@pve.proxmox.com">pve-devel@pve.proxmox.com</a>><br>
Envoyé: Lundi 19 Mai 2014 11:15:38<br>
Objet: RE: [pve-devel] loading nf_conntrack_ftp module by default ?<br>
<br>
> maybe in Firewall.pm, sub update() (which is called in run_server) ?<br>
<br>
I just added an init() function - please use that:<br>
<br>
<a href="https://git.proxmox.com/?p=pve-firewall.git;a=commitdiff;h=8b453a09f302dd91db5c02c92da144df37503d79" target="_blank">https://git.proxmox.com/?p=pve-firewall.git;a=commitdiff;h=8b453a09f302dd91db5c02c92da144df37503d79</a><br>

_______________________________________________<br>
pve-devel mailing list<br>
<a href="mailto:pve-devel@pve.proxmox.com">pve-devel@pve.proxmox.com</a><br>
<a href="http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel" target="_blank">http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel</a><br>
</blockquote></div>