docs: fix iptables logging references after simplification#1292
Conversation
- Correct LOG rule attribution from setup-iptables.sh to src/host-iptables.ts (FW_BLOCKED_UDP, FW_BLOCKED_OTHER are host-level DOCKER-USER chain rules) - Remove FW_DNS_QUERY section from logging quickref (prefix no longer exists) - Remove incorrect docker exec awf-agent dmesg commands (host kernel logs are not visible from inside the container) - Remove incorrect --log-uid reference (flag not used in host-iptables.ts) - Remove stale UID=0 field from iptables log example - Remove stale line number reference (src/squid-config.ts:40) Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Pull request was closed
|
Smoke Test Results — run 23208107287 ✅ GitHub MCP — Last 2 merged PRs:
✅ Playwright — github.com title contains "GitHub" Overall: PASS
|
Smoke Test Results ✅ PASS
|
|
🔮 The oracle records this smoke reading.
Warning
|
🏗️ Build Test Suite Results
Overall: 8/8 ecosystems passed — ✅ PASS
|
Documentation Sync - March 7–13, 2026
This PR synchronizes documentation with the code changes from commit
5d2ef18("fix(docker): simplify to localhost+Squid-only iptables (#1270)").Changes Made
AGENTS.md: Corrected the### iptables Loggingsectioncontainers/agent/setup-iptables.shtosrc/host-iptables.ts(whereFW_BLOCKED_UDPandFW_BLOCKED_OTHERrules actually live, in the DOCKER-USER chain)--log-uidflag reference (this flag is not used)src/squid-config.ts:40line number referencedocs/logging_quickref.md: Removed outdated DNS query logging section and fixed container dmesg note[FW_DNS_QUERY]log prefix no longer exists anywhere in the codebasedocker exec awf-agent dmesg | grep FW_BLOCKEDcommand (host DOCKER-USER chain logs are only visible on the host)UID=0from the iptables log example (not logged since--log-uidis not used)docs/troubleshooting.md: Removed incorrect container dmesg commanddocker exec awf-agent dmesg | grep FW_BLOCKED(same reason as above)Code Changes Referenced
5d2ef18: "fix(docker): simplify to localhost+Squid-only iptables (fix(docker): simplify to localhost+Squid-only iptables #1270)" — this PR simplified the container-level iptables insetup-iptables.sh. The LOG rules (FW_BLOCKED_UDP,FW_BLOCKED_OTHER) were never insetup-iptables.sh; they exist insrc/host-iptables.ts. TheFW_DNS_QUERYlog prefix was removed entirely.Verification
FW_BLOCKED_UDP/FW_BLOCKED_OTHERexist only insrc/host-iptables.ts(lines 400, 412)FW_DNS_QUERYdoes not exist anywhere in the codebase--log-uidflag is not used insrc/host-iptables.tsorcontainers/agent/setup-iptables.shdocker exec awf-agent dmesgcannot show host DOCKER-USER chain kernel logs