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

Allow blocking IPs from transmitting WebRTC video #571

Merged
merged 11 commits into from
Nov 17, 2023

Conversation

rafaellehmkuhl
Copy link
Member

@rafaellehmkuhl rafaellehmkuhl commented Nov 17, 2023

This patch allows the user to select which IPs should be allowed to transmit WebRTC video streams.
The selection is persistent between boots.
The only thing missing now is a popup saying to the user that he should define those (when they are undefined).

The solution was developed by @joaoantoniocardoso at #564. I kept his commits and polished the solution from there.

Fix #526

Screen.Recording.2023-11-17.at.17.13.29.mp4

Copy link
Member

@joaoantoniocardoso joaoantoniocardoso left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice, tested, and it's working great! :)

@rafaellehmkuhl rafaellehmkuhl merged commit b73b03a into bluerobotics:master Nov 17, 2023
8 checks passed
@rafaellehmkuhl rafaellehmkuhl deleted the choose_ice branch November 17, 2023 22:40
@ES-Alexander ES-Alexander added the docs-needed Change needs to be documented label Nov 24, 2023
ES-Alexander added a commit to ES-Alexander/ardusub-zola that referenced this pull request Nov 24, 2023
ES-Alexander added a commit to ES-Alexander/ardusub-zola that referenced this pull request Nov 24, 2023
ES-Alexander added a commit to bluerobotics/ardusub-zola that referenced this pull request Nov 24, 2023
ES-Alexander added a commit to bluerobotics/ardusub-zola that referenced this pull request Nov 24, 2023
ES-Alexander added a commit to bluerobotics/ardusub-zola that referenced this pull request Nov 24, 2023
@ES-Alexander ES-Alexander added docs-complete Change documentation has been completed and removed docs-needed Change needs to be documented labels Nov 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs-complete Change documentation has been completed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Strategies for a better video stream
3 participants