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

No audio after UPDATE packages behind NAT #37

Merged
merged 8 commits into from
Aug 28, 2024
Merged

No audio after UPDATE packages behind NAT #37

merged 8 commits into from
Aug 28, 2024

Conversation

Stell0
Copy link
Collaborator

@Stell0 Stell0 commented Aug 28, 2024

Audio get routed to the wrong address when Asterisk send an UPDATE packet while being behind a NAT.

In this scenario, the UPDATE packets were inadvertently modifying dlg_var(direction) variable within Kamailio. This variable is used to track the direction of the media flow (RTP packets) and is integral to the business logic that manages media handling through rtpengine_manage.

When dlg_var(direction) was altered by the UPDATE packets, it caused inconsistencies in how media flow direction was being handled, leading to disruptions in the call handling process. Essentially, the logic that determined the direction of RTP streams was getting confused, which could result in issues like incorrect media routing and dropped calls after 30 seconds.

https://evoseed.atlassian.net/servicedesk/customer/portal/5/EA-29

Paolo and others added 8 commits July 19, 2024 20:18
The variable is used to determine if a device is present on the local
network. If the device is on the local network, Kamailio must present
itself to the device with its private IP address. If the device is
located outside on the internet, Kamailio must use its public IP
address.

Additional local networks can be added using the `local_networks` field
of the JSON object passed to the `configure-module` action.
The field is only used in a NAT scenario when a public IP is also set.
@Stell0 Stell0 merged commit a18abc2 into main Aug 28, 2024
1 check passed
@Stell0 Stell0 deleted the EA-29 branch August 28, 2024 12:38
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