Skip to content

[Core] Add fc00::/7 (ULA) to builtin private/WAN ACL lists - #3101

Open
lixuanqun wants to merge 1 commit into
signalwire:masterfrom
lixuanqun:cursor/fc00-ula-acl-ee5d
Open

[Core] Add fc00::/7 (ULA) to builtin private/WAN ACL lists#3101
lixuanqun wants to merge 1 commit into
signalwire:masterfrom
lixuanqun:cursor/fc00-ula-acl-ee5d

Conversation

@lixuanqun

Copy link
Copy Markdown

Unique Local Addresses (RFC 4193) are not globally routable and should be treated like other private ranges for ICE candidate ACL purposes.

Add fc00::/7 to rfc1918.auto (allow), wan.auto and wan_v6.auto (deny), matching how fe80::/10 link-local is already handled. Include unit tests for CIDR parsing and builtin ACL matching.

Fixes #2703

Description

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Documentation update
  • Code cleanup / refactor

Related Issues

Testing

  • Added/updated unit tests
  • Tested manually
  • Tested with live SignalWire credentials (if applicable)

Checklist

  • I have read the CONTRIBUTING guidelines
  • My code follows the project's style guidelines
  • I have added tests for my changes (if applicable)
  • I have updated documentation (if applicable)
  • All existing tests pass

Additional Notes

Unique Local Addresses (RFC 4193) are not globally routable and should
be treated like other private ranges for ICE candidate ACL purposes.

Add fc00::/7 to rfc1918.auto (allow), wan.auto and wan_v6.auto (deny),
matching how fe80::/10 link-local is already handled. Include unit tests
for CIDR parsing and builtin ACL matching.

Fixes signalwire#2703

Co-authored-by: li xuanqun <793005378@qq.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Required add fc00::/7 into predefined lists

2 participants