Allow specifying mod set multiple times#89
Allow specifying mod set multiple times#89markkrj wants to merge 1 commit intoMaxKellermann:masterfrom
Conversation
`ipset` matching module doesn't allow specifying multiple `--match-set` for a single `-m set`, so, we'll keep every `mod set` specified in the config.
|
I'm testing a script that runs before ferm and output rules that are later included by ferm, and after I opened this PR, I noticed that more modules (in this case, So if user input: it would correctly render: instead of current behavior: Currently, I patched ferm like this PR (but now included the comment module in the regex), but if there was no downside, I'd prefer removing it altogether... |
|
I think this might need a more evolved patch, as I just tested this: and ferm outputs: which iptables complains: |
Fixes #84
ipsetmatching module doesn't allow specifying multiple--match-setfor a single-m set, so, we'll keep everymod setspecified in the config.@MaxKellermann feel free to edit this PR directly or close it if you have a better idea, as I'm not a programmer.