v0.4: Multi-IP NAT1To1 support #20

New issue

Closed

opened 2026-05-10 13:18:45 -04:00 by zgaetano · 0 comments

zgaetano commented

2026-05-10 13:18:45 -04:00

Owner

Implemented in commits 57542a3 / b045b26 / 841335d / 6ec0328.\n\nChanges:\n- core/webrtc/Config: added NAT1To1IPs []string alongside the existing PublicIP field\n- core/webrtc/BuildICEConfig: prefers NAT1To1IPs when non-empty; falls back to []string{PublicIP} for backward compat (legacy configs with only PublicIP continue to work unchanged)\n- app/webrtc/subsystem.New: builds the merged list — PublicIP first, then any additional entries from NAT1To1IPs, deduplicating — and assigns it to coreCfg.NAT1To1IPs; removes the old single-IP workaround\n- 4 new tests in core/webrtc/ice_test.go covering the multi-IP path, the PublicIP fallback, both-set, and neither-set cases

Implemented in commits 57542a3 / b045b26 / 841335d / 6ec0328.\n\n**Changes:**\n- `core/webrtc/Config`: added `NAT1To1IPs []string` alongside the existing `PublicIP` field\n- `core/webrtc/BuildICEConfig`: prefers `NAT1To1IPs` when non-empty; falls back to `[]string{PublicIP}` for backward compat (legacy configs with only `PublicIP` continue to work unchanged)\n- `app/webrtc/subsystem.New`: builds the merged list — `PublicIP` first, then any additional entries from `NAT1To1IPs`, deduplicating — and assigns it to `coreCfg.NAT1To1IPs`; removes the old single-IP workaround\n- 4 new tests in `core/webrtc/ice_test.go` covering the multi-IP path, the PublicIP fallback, both-set, and neither-set cases