<feed xmlns='http://www.w3.org/2005/Atom'>
<title>firewall4/tests, branch master</title>
<subtitle>OpenWrt nftables firewall</subtitle>
<id>https://git-03.infra.openwrt.org/project/firewall4/atom?h=master</id>
<link rel='self' href='https://git-03.infra.openwrt.org/project/firewall4/atom?h=master'/>
<link rel='alternate' type='text/html' href='https://git-03.infra.openwrt.org/project/firewall4/'/>
<updated>2023-11-03T13:33:55Z</updated>
<entry>
<title>ruleset: apply egress MSS fixup later to apply final MTU before wire</title>
<updated>2023-11-03T13:33:55Z</updated>
<author>
<name>Andris PE</name>
</author>
<published>2023-06-21T10:06:24Z</published>
<link rel='alternate' type='text/html' href='https://git-03.infra.openwrt.org/project/firewall4/commit/?id=698a53354fd280aae097efe08803c0c9a10c14c2'/>
<id>urn:sha1:698a53354fd280aae097efe08803c0c9a10c14c2</id>
<content type='text'>
Reduce scope of MSS fixup to TCP SYN packets only and relocate the fixing
of egress MSS to the mangle/postrouting chain in order to properly apply
final known MTU size.

Fixes: openwrt/openwrt#12112
Signed-off-by: Andris PE &lt;neandris@gmail.com&gt;
[fix S-o-b tag, fix commit author, reword commit message]
Signed-off-by: Jo-Philipp Wich &lt;jo@mein.io&gt;
</content>
</entry>
<entry>
<title>tests: adjust zone log limit testcases</title>
<updated>2023-11-03T13:14:15Z</updated>
<author>
<name>Jo-Philipp Wich</name>
</author>
<published>2023-11-03T13:14:15Z</published>
<link rel='alternate' type='text/html' href='https://git-03.infra.openwrt.org/project/firewall4/commit/?id=de3483c561a728d5234a0a3f49b5dde4527a0f3f'/>
<id>urn:sha1:de3483c561a728d5234a0a3f49b5dde4527a0f3f</id>
<content type='text'>
Fix testcase failure introduced by a previous commit.

Fixes: a5553da ("ruleset: reduce ksoftirqd load by refering to looopback by numeric id")
Signed-off-by: Jo-Philipp Wich &lt;jo@mein.io&gt;
</content>
</entry>
<entry>
<title>ruleset: reduce ksoftirqd load by refering to looopback by numeric id</title>
<updated>2023-11-03T13:11:06Z</updated>
<author>
<name>Andris PE</name>
</author>
<published>2023-09-19T15:23:59Z</published>
<link rel='alternate' type='text/html' href='https://git-03.infra.openwrt.org/project/firewall4/commit/?id=a5553dae70439c7e4fa910490fcf12a1ffff5bd2'/>
<id>urn:sha1:a5553dae70439c7e4fa910490fcf12a1ffff5bd2</id>
<content type='text'>
Reduce ksoftirq load by half using more efficient reference to loopback
which always has index equal to one.

Should help a lot with openwrt/openwrt#12914, openwrt/openwrt#12121 and
similar iperf3 cases clamping against 100% CPU usage.

Signed-off-by: Andris PE &lt;neandris@gmail.com&gt;
[fix S-o-b tag, fix commit author, rewrap commit message]
Signed-off-by: Jo-Philipp Wich &lt;jo@mein.io&gt;
</content>
</entry>
<entry>
<title>ruleset: dispatch ct states using verdict map</title>
<updated>2023-11-03T13:09:43Z</updated>
<author>
<name>Andris PE</name>
</author>
<published>2023-09-07T19:04:35Z</published>
<link rel='alternate' type='text/html' href='https://git-03.infra.openwrt.org/project/firewall4/commit/?id=19a8caf614ec338513e58535ea02c6ee52988170'/>
<id>urn:sha1:19a8caf614ec338513e58535ea02c6ee52988170</id>
<content type='text'>
In case the dropping of invalid conntrack states is enabled, using a verdict
map allows us to use only one rule instead of two, lowering the initial rule
match overhead.

Signed-off-by: Andris PE &lt;neandris@gmail.com&gt;
[whitespace cleanup, rebase, extend commit subject and message]
Signed-off-by: Jo-Philipp Wich &lt;jo@mein.io&gt;
</content>
</entry>
<entry>
<title>Revert "ruleset: dispatch ct states using verdict map"</title>
<updated>2023-11-03T13:09:16Z</updated>
<author>
<name>Jo-Philipp Wich</name>
</author>
<published>2023-11-03T13:09:12Z</published>
<link rel='alternate' type='text/html' href='https://git-03.infra.openwrt.org/project/firewall4/commit/?id=22c53921c11115e5437385719b6e73800a68cd33'/>
<id>urn:sha1:22c53921c11115e5437385719b6e73800a68cd33</id>
<content type='text'>
This reverts commit 785798c8fd72ff3c4c8940922173290bb25bc18e.

Revert commit due to bad commit metadata.

Signed-off-by: Jo-Philipp Wich &lt;jo@mein.io&gt;
</content>
</entry>
<entry>
<title>ruleset: dispatch ct states using verdict map</title>
<updated>2023-11-03T13:04:39Z</updated>
<author>
<name>User User-User</name>
</author>
<published>2023-09-07T19:04:35Z</published>
<link rel='alternate' type='text/html' href='https://git-03.infra.openwrt.org/project/firewall4/commit/?id=785798c8fd72ff3c4c8940922173290bb25bc18e'/>
<id>urn:sha1:785798c8fd72ff3c4c8940922173290bb25bc18e</id>
<content type='text'>
In case the dropping of invalid conntrack states is enabled, using a verdict
map allows us to use only one rule instead of two, lowering the initial rule
match overhead.

Signed-off-by: Andris PE &lt;neandris@gmail.com&gt;
[whitespace cleanup, rebase, extend commit subject and message]
Signed-off-by: Jo-Philipp Wich &lt;jo@mein.io&gt;
</content>
</entry>
<entry>
<title>fw4: add log_limit to rules and redirects</title>
<updated>2023-11-03T12:37:19Z</updated>
<author>
<name>Luiz Angelo Daros de Luca</name>
</author>
<published>2023-08-01T19:51:58Z</published>
<link rel='alternate' type='text/html' href='https://git-03.infra.openwrt.org/project/firewall4/commit/?id=187405075911d408fa48e97ce343e76a2a30ef12'/>
<id>urn:sha1:187405075911d408fa48e97ce343e76a2a30ef12</id>
<content type='text'>
Just like zone log_limit, now you can specify a different log limit to a
single rule or redirect.

Signed-off-by: Luiz Angelo Daros de Luca &lt;luizluca@gmail.com&gt;
[whitespace cleanup, properly format limit expressions]
Signed-off-by: Jo-Philipp Wich &lt;jo@mein.io&gt;
</content>
</entry>
<entry>
<title>tests: fix expected test output</title>
<updated>2023-05-30T08:19:51Z</updated>
<author>
<name>Jo-Philipp Wich</name>
</author>
<published>2023-05-30T08:19:01Z</published>
<link rel='alternate' type='text/html' href='https://git-03.infra.openwrt.org/project/firewall4/commit/?id=23a434d0d15d61db61bb065c89f266a326c78a88'/>
<id>urn:sha1:23a434d0d15d61db61bb065c89f266a326c78a88</id>
<content type='text'>
A previous commit enabled flowtable counters without properly adjusting
the testcase output.

Fixes: 04a06bd ("fw4: enable flowtable counters")
Signed-off-by: Jo-Philipp Wich &lt;jo@mein.io&gt;
</content>
</entry>
<entry>
<title>tests: add testcase for automatic includes</title>
<updated>2023-02-03T11:04:58Z</updated>
<author>
<name>Jo-Philipp Wich</name>
</author>
<published>2023-02-03T11:04:58Z</published>
<link rel='alternate' type='text/html' href='https://git-03.infra.openwrt.org/project/firewall4/commit/?id=ce9a37829a765c62ce8d9da37caa9bbfb5bb58ec'/>
<id>urn:sha1:ce9a37829a765c62ce8d9da37caa9bbfb5bb58ec</id>
<content type='text'>
Signed-off-by: Jo-Philipp Wich &lt;jo@mein.io&gt;
</content>
</entry>
<entry>
<title>fw4: fix handling the ipset "comment" option</title>
<updated>2023-02-03T11:04:15Z</updated>
<author>
<name>Jo-Philipp Wich</name>
</author>
<published>2023-01-07T16:00:18Z</published>
<link rel='alternate' type='text/html' href='https://git-03.infra.openwrt.org/project/firewall4/commit/?id=39e8c70957c795bf0c12f04299170ae86c6efdf8'/>
<id>urn:sha1:39e8c70957c795bf0c12f04299170ae86c6efdf8</id>
<content type='text'>
The comment option for ipset definitions is incorrectly declared as bool
and not actually used anywhere in the nftables output rendering.

Solve this issue by changing it to the proper "string" type and expose
the user configured comment as "comment" property in the generated nftables
output.

Also add some initial test coverage for ipset declarations to better spot
such inconsistencies in the future.

Ref: https://github.com/openwrt/luci/pull/6187#issuecomment-1374506633
Reported-by: Paul Dee &lt;itsascambutmailmeanyway@gmail.com&gt;
Signed-off-by: Jo-Philipp Wich &lt;jo@mein.io&gt;
</content>
</entry>
</feed>
