udp-mux: RemoveConnByUfrag removes existing connection when having 1:1 NAT mapping

[m1k1o]

udp-mux: RemoveConnByUfrag removes existing connection when having 1:1 NAT mapping

[r1a2y3]

Hi, I haven't try to reproduce your issue. In my opinion, you try to use 2 local address but only one external IP And you want to use udp mux?? Such as local ip (192.168.10.10 / 192.168.10.20, multiplex udp port 3000, they have the same external ip. [128.29.20.30]). Maybe the extIPMapper should specify the relation 192.168.10.10->128.29.20.30, rather than set all ipv4 external ip to 128.29.20.30

---

By the way, I looked through issues because I found the multiudpmux can't works when the remote use multiudpmux too... Such as two sfu cascaded, but they all use one single udp port.

[m1k1o]

Its just about how many local interfaces I have. When inside docker container, those IPs will be 127.0.0.1 and 172.18.0.10 (dynamic, cannot really specify relation) but my real external IP is 192.168.1.10.

This issue is actually generally about wrong handling any UDP duplicate candidates, if the number of them is an even number. This happens to be easily reproducible with NAT 1:1, but its not limited to only this usecase.

Should be as following:

• 1 canidate: OK
• 2 duplicate candidates: First added, Second overwrites first one but immediately closes and removes it, so we are left with no canidates.
• 3 duplicate candidates: After adding and removing one, thrid one is left intact, so OK:

[r1a2y3]

Thanks @m1k1o . Got it! Maybe a temporary workaround is to limit your candidates to certain specified range that you want to avoid it. I use the below code to specify candidates now.

        // se
        se := webrtc.SettingEngine{}

        // handle candidates
        nw := vnet.NewNet(nil)
        if len(c.WebRTC.Candidates) > 0 {
                candidates := c.WebRTC.Candidates
                if ifs, err := nw.Interfaces(); err == nil {
                        for _, ifc := range ifs {
                                if addrs, err := ifc.Addrs(); err == nil {
                                        tmpIfc := vnet.NewInterface((net.Interface)(ifc.InterfaceBase))
                                        for _, c := range candidates {
                                                for _, addr := range addrs {
                                                        if strings.HasPrefix(addr.String(), c+"/") {
                                                                tmpIfc.AddAddr(addr)
                                                        }
                                                }
                                        }
                                        *ifc = *tmpIfc
                                }
                        }
                }
        }
        se.SetVNet(nw)

[cnderrauber]

@m1k1o You can use MultiUDPMux (NewMultiUDPMuxFromPort) to listen on multiple interfaces for udp mux, then each muxed connection is managed by corresponding UDPMux, will not be closed incorrectly

[m1k1o]

@cnderrauber yes, that actually fixes my issue. But this makes the change backwards incompatible, although it should still work the same way but only with deprecation warning.