Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

v0.22.0 breaks webrtcICEUDPMuxAddress #1680

Closed
2 of 13 tasks
mrlt8 opened this issue Apr 11, 2023 · 3 comments · Fixed by #1699
Closed
2 of 13 tasks

v0.22.0 breaks webrtcICEUDPMuxAddress #1680

mrlt8 opened this issue Apr 11, 2023 · 3 comments · Fixed by #1699
Labels
bug Something isn't working webrtc

Comments

@mrlt8
Copy link
Contributor

mrlt8 commented Apr 11, 2023

Which version are you using?

v0.22.0

Which operating system are you using?

  • Linux amd64 standard
  • Linux amd64 Docker
  • Linux arm64 standard
  • Linux arm64 Docker
  • Linux arm7 standard
  • Linux arm7 Docker
  • Linux arm6 standard
  • Linux arm6 Docker
  • Windows amd64 standard
  • Windows amd64 Docker (WSL backend)
  • macOS amd64 standard
  • macOS amd64 Docker
  • Other (please describe)

Describe the issue

webrtcICEUDPMuxAddress was working previously, but seems to be broken after upgrading to v0.22.0.
Switching to webrtcICETCPPMuxAddress with -p 8189:8189 still works as expected.

Describe how to replicate the issue

docker run -p 8889:8889 -p 8189:8189/udp \
-e MTX_PATHS_TEST_SOURCE=rtsp://test-server:8554/stream \
-e MTX_WEBRTCICEHOSTNAT1TO1IPS=10.0.0.111 \
-e MTX_WEBRTCICEUDPMUXADDRESS=:8189 \
aler9/rtsp-simple-server

Did you attach the server logs?

 DEB [WebRTC] [conn 172.17.0.1:44760] peer connection state: closed
 INF [WebRTC] [conn 172.17.0.1:44760] closed (deadline exceeded)
 DEB [WebRTC] [conn 172.17.0.1] [s->c] HTTP/1.1 404 Not Found
Access-Control-Allow-Credentials: true
Access-Control-Allow-Origin: *
Server: mediamtx

Did you attach a network dump?

no

EDIT:

Works as expected when rolling back down to v0.21.6:

docker run -p 8889:8889 -p 8189:8189/udp \
-e RTSP_PATHS_TEST_SOURCE=rtsp://test-server:8554/stream \
-e RTSP_WEBRTCICEHOSTNAT1TO1IPS=10.0.0.111 \
-e RTSP_WEBRTCICEUDPMUXADDRESS=:8189 \
aler9/rtsp-simple-server:v0.21.6
@aler9 aler9 added bug Something isn't working webrtc labels Apr 12, 2023
@aler9
Copy link
Member

aler9 commented Apr 12, 2023

Hello, i've investigated the issue and it turned out that it's a regression in pion/ice.
I've sent a PR with a fix here: pion/ice#550

@aler9
Copy link
Member

aler9 commented Apr 13, 2023

added in MediaMTX v0.22.1

@github-actions
Copy link
Contributor

This issue is being locked automatically because it has been closed for more than 6 months.
Please open a new issue in case you encounter a similar problem.

@github-actions github-actions bot locked and limited conversation to collaborators Oct 17, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working webrtc
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants