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

Implement ICE Enforcement Configuration #340

Merged
merged 5 commits into from
Dec 5, 2024
Merged

Implement ICE Enforcement Configuration #340

merged 5 commits into from
Dec 5, 2024

Conversation

Amygos
Copy link
Member

@Amygos Amygos commented Nov 25, 2024

This pull request adds the ability to enforce ICE candidates with the proxy IP
address. It introduces a new environment variable, ICEENFORCE, which is set
to the proxy IP address. The janus.service is also updated to include the
--ice-enforce-list option, allowing the enforcement of ICE candidates on
specific interfaces or IP addresses. Additionally, the create-module and
update-module actions are modified to include the ICEENFORCE variable
enforcement behavior.

NethServer/dev#7099

@Amygos Amygos force-pushed the iceenforce branch 3 times, most recently from 34bf0ea to 7edd171 Compare November 26, 2024 15:35
Introduce ICEENFORCE variable to control ICE
enforcement behavior.
Introduce a new script step to configure ICE to enforce the proxy IP
address. This step retrieves the proxy service records and updates the
environment variable `ICEENFORCE` with the correct address. If the
address is changed, it triggers a restart of relevant services to apply
the new configuration.
Use the ICEENFORCE environment variable to populate the list of the only
interfaces that Janus will use for ICE gathering.
Add documentation for the `ICEENFORCE` environment variable in the
README.md. This variable allows users to specify a list of interfaces or
IP addresses to enforce during ICE candidates gathering. By default, it
is set to 'none'.
On the event `nethvoice-proxy-settings-changed`, update the ICEENFORCE
environment variable and also restart the janus service.
@Amygos Amygos merged commit 4a09116 into main Dec 5, 2024
4 checks passed
@Amygos Amygos deleted the iceenforce branch December 5, 2024 16:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants