[PR #433] [MERGED] Nft refactor2 #441

New issue

Closed

opened 2026-03-04 01:36:53 +03:00 by kerem · 0 comments

kerem commented

2026-03-04 01:36:53 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/spr-networks/super/pull/433
Author: @lts-rad
Created: 7/24/2025
Status: ✅ Merged
Merged: 7/29/2025
Merged by: @lts-rad

Base: dev ← Head: nft_refactor2

📝 Commits (10+)

3bc104b [api] NFT refactor with latest google/nftables
85cf631 [api] Add test files
6b98b04 [api] Finish nft call refactor
4cdb00e [api] Fix verdict maps with concatenated keys, add tests
b593649 [api] Fix KeyEnd argument, tests
0d74fef [api] Continued fixes for NFT refactor
0c903bd [api] Fix port range keys
cec0dae [api] Fix supernets set access
3913510 [api] Fix dns_access tests and add dns_access for dockerif
f702984 [api] Fix dns_access tests and add dns_access for dockerif

📊 Changes

22 files changed (+7487 additions, -537 deletions)

View changed files

➕ api/Dockerfile.test (+58 -0)
📝 api/code/api.go (+134 -40)
📝 api/code/darwin.go (+346 -0)
📝 api/code/dhcp.go (+2 -0)
📝 api/code/firewall.go (+473 -473)
➕ api/code/firewall_test.go (+2277 -0)
📝 api/code/go.mod (+5 -0)
📝 api/code/go.sum (+14 -0)
📝 api/code/interfaces.go (+41 -19)
📝 api/code/linux.go (+78 -0)
➕ api/code/nftables.go (+1899 -0)
➕ api/code/nftables_test.go (+1066 -0)
📝 api/code/plugins.go (+50 -0)
📝 api/code/traffic.go (+2 -5)
➕ api/test/alerts.json (+197 -0)
➕ api/test/api.json (+57 -0)
➕ api/test/devices.json (+20 -0)
➕ api/test/dhcp.json (+1 -0)
➕ api/test/firewall.json (+40 -0)
➕ api/test/groups.json (+3 -0)

...and 2 more files

📄 Description

No description provided

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/spr-networks/super/pull/433 **Author:** [@lts-rad](https://github.com/lts-rad) **Created:** 7/24/2025 **Status:** ✅ Merged **Merged:** 7/29/2025 **Merged by:** [@lts-rad](https://github.com/lts-rad) **Base:** `dev` ← **Head:** `nft_refactor2` --- ### 📝 Commits (10+) - [`3bc104b`](https://github.com/spr-networks/super/commit/3bc104b9a56d6d3672d89f5a89bf804a2e658ed7) [api] NFT refactor with latest google/nftables - [`85cf631`](https://github.com/spr-networks/super/commit/85cf631beb741f443a2ed88d0413ef9c426fec7d) [api] Add test files - [`6b98b04`](https://github.com/spr-networks/super/commit/6b98b04bc4084e7cbe2e3bda5951f0cff7fbcb37) [api] Finish nft call refactor - [`4cdb00e`](https://github.com/spr-networks/super/commit/4cdb00e69d9c8a772c4469da009b4879898631e8) [api] Fix verdict maps with concatenated keys, add tests - [`b593649`](https://github.com/spr-networks/super/commit/b5936490ad93a02f59a7390844408ad45b974d6a) [api] Fix KeyEnd argument, tests - [`0d74fef`](https://github.com/spr-networks/super/commit/0d74fef019c523aeef44a5e2d7ab8525210b2c31) [api] Continued fixes for NFT refactor - [`0c903bd`](https://github.com/spr-networks/super/commit/0c903bd6376abc8c7cc98615ca35b50a136767c1) [api] Fix port range keys - [`cec0dae`](https://github.com/spr-networks/super/commit/cec0dae400e3d42bc3bf1706ec9445bc1279d620) [api] Fix supernets set access - [`3913510`](https://github.com/spr-networks/super/commit/3913510c0d59745ab29dd7f11d06f41889888797) [api] Fix dns_access tests and add dns_access for dockerif - [`f702984`](https://github.com/spr-networks/super/commit/f7029846afcc59b77d8d4ef0b463575b1997b98b) [api] Fix dns_access tests and add dns_access for dockerif ### 📊 Changes **22 files changed** (+7487 additions, -537 deletions) <details> <summary>View changed files</summary> ➕ `api/Dockerfile.test` (+58 -0) 📝 `api/code/api.go` (+134 -40) 📝 `api/code/darwin.go` (+346 -0) 📝 `api/code/dhcp.go` (+2 -0) 📝 `api/code/firewall.go` (+473 -473) ➕ `api/code/firewall_test.go` (+2277 -0) 📝 `api/code/go.mod` (+5 -0) 📝 `api/code/go.sum` (+14 -0) 📝 `api/code/interfaces.go` (+41 -19) 📝 `api/code/linux.go` (+78 -0) ➕ `api/code/nftables.go` (+1899 -0) ➕ `api/code/nftables_test.go` (+1066 -0) 📝 `api/code/plugins.go` (+50 -0) 📝 `api/code/traffic.go` (+2 -5) ➕ `api/test/alerts.json` (+197 -0) ➕ `api/test/api.json` (+57 -0) ➕ `api/test/devices.json` (+20 -0) ➕ `api/test/dhcp.json` (+1 -0) ➕ `api/test/firewall.json` (+40 -0) ➕ `api/test/groups.json` (+3 -0) _...and 2 more files_ </details> ### 📄 Description _No description provided_ --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>