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

[Question] Limit WebRTC port range? #101

Open
robertgroh opened this issue Apr 22, 2020 · 8 comments
Open

[Question] Limit WebRTC port range? #101

robertgroh opened this issue Apr 22, 2020 · 8 comments
Labels
issued This means the ticket is already created on internal issue tracker question Further information is requested

Comments

@robertgroh
Copy link

robertgroh commented Apr 22, 2020

Is it possible to limit the port ranges used for WebRTC connections?
I see, that for UDP connection ports above 60000 and for TCP port in the 54000ish range.

Is there a known default range for UDP or TCP?
Also, can I configure a limited port range?

@karasusan karasusan added the question Further information is requested label Apr 24, 2020
@karasusan
Copy link
Collaborator

I guess the limitation of the port range should be controlled by the firewall rule.

example : GCP
https://cloud.google.com/vpc/docs/firewalls

@robertgroh
Copy link
Author

Hi @karasusan, thank you for your reply.

I guess I can limit the ports, which will be functioning (and returned as SERVER CANDIDATES from the STUN service) via blocks by firewall rules, but I want to know the port ranges, which will be tried in the first place.

Maybe this question is better located at https://github.com/Unity-Technologies/UnityRenderStreaming, what do you think?

@karasusan
Copy link
Collaborator

The same question is posted on Unity forum.
https://forum.unity.com/threads/unity-webrtc-restrict-datachannel-ports.1025188/

@juju812
Copy link
Contributor

juju812 commented Jun 30, 2021

juju812@3ef0c0e

Hi, @karasusan , here's one snippet implemented the port range feature.

But currently the port range is hard coded in Context class (which is enough for my temporary use), the port range configuration need to be exposed.

@karasusan
Copy link
Collaborator

@juju812
I am checking it. Thanks.

@juju812
Copy link
Contributor

juju812 commented Jul 2, 2021

Hi @karasusan

Do you have any schedule to integrate the port range feature? If not, maybe I can follow current configuration tamplate and complete my code. Then I'll create a PR after implement finished.

@karasusan
Copy link
Collaborator

@juju812
Thanks. I am sorry but the schedule of the feature will be a long time later.

@kannan-xiao4
Copy link
Contributor

memo: WRS-238

@kannan-xiao4 kannan-xiao4 added the issued This means the ticket is already created on internal issue tracker label Feb 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
issued This means the ticket is already created on internal issue tracker question Further information is requested
Projects
None yet
Development

No branches or pull requests

4 participants