You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
Commit 5443f69
Browse filesBrowse the repository at this point in the historyBrowse files
Rename IP access-control API to symmetric allow/deny vocabulary
TASK-029 collapsed the v1 ban_ip/unban_ip/allow_ip/disallow_ip quartet
to a deny-only block_ip/unblock_ip pair, leaving the allowances set and
the REJECT branch of the policy callback reachable only internally. The
result: default_policy(REJECT) was public and documented but unusable
(nothing could populate the allow list, so REJECT rejected every
connection), and block_ip read as blocklist-only while the config could
flip the system into allowlist mode.
This restores a symmetric, consistently-named surface and makes the
allow-list mode actually work:
block_ip -> deny_ip
unblock_ip -> remove_denied_ip
(new) -> allow_ip
(new) -> remove_allowed_ip
ban_system(bool)-> ip_access_control(bool)
default_policy(ACCEPT) makes the deny list the exception list; REJECT
makes the allow list the exception list. An allow entry overrides a
matching deny entry (allow wins). Internal sets renamed bans->deny_list,
allowances->allow_list; accept_ctx reasons "banned"->"denied",
"not-allowed"->"not-on-allow-list".
Restores the allow-list integ coverage TASK-029 deleted (now via the
public API), updates README/RELEASE_NOTES/examples and the CI
enforcement scripts, and renames the ban_system.cpp /
hooks_accept_decision_banned.cpp / minimal_ip_ban.cpp /
banned_ip_log.cpp files to match. Also clears the pre-existing
check-doxygen warnings (define the @security alias; fix stale @ref
targets) so make check is green. Design rationale in
specs/proposals/ip-access-control-naming.md.
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01NpysYDDJac63yz2mZKKiDf
0 commit comments