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.

Sign up for GitHub

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

Jump to bottom

WebRTC: WebRTC over TCP directly, not TURN, TCP transmission (non-TURN) #2852

Closed
Tracked by #2843
winlinvip opened this issue Jan 6, 2022 · 1 comment · Fixed by #3166
Closed
Tracked by #2843

WebRTC: WebRTC over TCP directly, not TURN, TCP transmission (non-TURN) #2852

winlinvip opened this issue Jan 6, 2022 · 1 comment · Fixed by #3166
Assignees
@winlinvip @xialixin @lipeng19811218
Labels
Enhancement Improvement or enhancement. Feature It's a new feature. Kubernetes For K8s, Prometheus, APM and Grafana. TransByAI Translated by AI/GPT. WebRTC WebRTC, RTC2RTMP or RTMP2RTC.
Milestone
5.0

Comments

@winlinvip
Copy link
Member

winlinvip commented Jan 6, 2022 •
edited
Loading

RFC

Usage

Please see doc or 中文文档.

Example

Thank you @xialixin. H5 also supports direct TCP connection. SDP Offer: offer.txt, SDP Answer: answer.txt

Here is the additional SDP for UDP: udp-answer.txt udp-offer.txt. The difference lies in the Answer.

a=candidate:0 1 udp 2130706431 192.168.3.82 8000 typ host generation 0
a=candidate:2 1 TCP 2124414975 10.0.1.1 8998 typ host tcptype passive
a=candidate:tcpcandidate 1 tcp 1076302079 39.100.155.146 40390 typ host tcptype passive

ICE screenshot:

image

Wireshark: Filter by tcp.stream eq 7 webrtc-tcp.pcapng.zip

The whole capture by ip host 39.100.155.146

TRANS_BY_GPT3
@winlinvip winlinvip mentioned this issue Jan 6, 2022
Closed
2 tasks
@winlinvip winlinvip added this to the 5.0 milestone Jan 6, 2022
@winlinvip winlinvip added Enhancement Improvement or enhancement. Feature It's a new feature. WebRTC WebRTC, RTC2RTMP or RTMP2RTC. labels Jan 6, 2022
@winlinvip winlinvip mentioned this issue Jan 6, 2022
Closed
@xianchuhan
Copy link

xianchuhan commented Jan 9, 2022 •
edited by winlinvip
Loading

This is essentially ice supporting TCP, which Janus also supports. I encountered this at my previous company, where some clients had strict restrictions on UDP in their networks and only allowed TCP by default.

TRANS_BY_GPT3

@winlinvip winlinvip added the Kubernetes For K8s, Prometheus, APM and Grafana. label Sep 2, 2022
@winlinvip winlinvip mentioned this issue Sep 2, 2022
Closed
@winlinvip winlinvip changed the title WebRTC: 支持TCP直连的方式，非TURN而是TCP直连 WebRTC: WebRTC over TCP directly, not TURN, 支持TCP直连的方式，非TURN而是TCP直连 Sep 2, 2022
@winlinvip winlinvip changed the title WebRTC: WebRTC over TCP directly, not TURN, 支持TCP直连的方式，非TURN而是TCP直连 WebRTC: WebRTC over TCP directly, not TURN, 支持TCP直连的方式(非TURN) Sep 2, 2022
@winlinvip winlinvip changed the title WebRTC: WebRTC over TCP directly, not TURN, 支持TCP直连的方式(非TURN) WebRTC: WebRTC over TCP directly, not TURN, TCP直连传输(非TURN) Sep 2, 2022
@winlinvip winlinvip changed the title WebRTC: WebRTC over TCP directly, not TURN, TCP直连传输(非TURN) WebRTC: WebRTC over TCP directly, not TURN, TCP传输(非TURN) Sep 2, 2022
@ossrs ossrs deleted a comment from xianchuhan Sep 2, 2022
@winlinvip winlinvip linked a pull request Sep 4, 2022 that will close this issue
WebRTC: Support transport over TCP directly, not TURN. #3166
Merged
5 tasks
winlinvip referenced this issue Sep 5, 2022
@winlinvip
WebRTC: Extract SrsRtcNetwork layer for UDP/TCP.
625069a
winlinvip referenced this issue Sep 5, 2022
@winlinvip
WebRTC: Support config, listener and SDP for TCP transport.
770d959
winlinvip referenced this issue Sep 5, 2022
@lipeng19811218 @winlinvip
WebRTC: Support TCP network.
07339e1
winlinvip referenced this issue Sep 5, 2022
@lipeng19811218 @winlinvip
WebRTC: Refine code and destroy session when tcp close.
efa0851
winlinvip added a commit that referenced this issue Sep 5, 2022
@winlinvip
For #2852: WebRTC: Support only one TCP connected candidate.
b0e5595
winlinvip added a commit that referenced this issue Sep 5, 2022
@winlinvip
For #2852: WebRTC: Support only one TCP connected candidate.
2144ff8
@winlinvip winlinvip changed the title WebRTC: WebRTC over TCP directly, not TURN, TCP传输(非TURN) WebRTC: WebRTC over TCP directly, not TURN, TCP transmission (non-TURN) Jul 29, 2023
@winlinvip winlinvip added the TransByAI Translated by AI/GPT. label Jul 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@winlinvip winlinvip

@xialixin xialixin

@lipeng19811218 lipeng19811218

Labels
Enhancement Improvement or enhancement. Feature It's a new feature. Kubernetes For K8s, Prometheus, APM and Grafana. TransByAI Translated by AI/GPT. WebRTC WebRTC, RTC2RTMP or RTMP2RTC.
Projects
None yet
Milestone
5.0
Development

Successfully merging a pull request may close this issue.

WebRTC: Support transport over TCP directly, not TURN. winlinvip/srs
4 participants
@winlinvip @xialixin @lipeng19811218 @xianchuhan