banIP: inbound mark set

[filippocarletti]

Hi @dibdot

reviewing banIP 1.5.x I noticed that inbound connections are marked 1 and 2 for input and forward, here:

packages/net/banip/files/banip-functions.sh

Line 725 in 884876c

printf "%s\n" "add rule inet banIP wan-input meta mark set 1"

packages/net/banip/files/banip-functions.sh

Line 733 in 884876c

printf "%s\n" "add rule inet banIP wan-forward meta mark set 2"

Is that needed?

[dibdot]

Does it male any harm?
The markers make it easy to distinguish between wan-input and wan-forward in the log.

[brada4]

You can unify them in prerouting if separate counters are not needed / visualised.