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

[Bug]: Network filter does not apply to socket routing #1969

Open
3 tasks done
apurohitrivian opened this issue Apr 16, 2024 · 1 comment
Open
3 tasks done

[Bug]: Network filter does not apply to socket routing #1969

apurohitrivian opened this issue Apr 16, 2024 · 1 comment
Assignees
Labels
bug Something isn't working pending-research v1.7.3

Comments

@apurohitrivian
Copy link

Please confirm you have already done the following

  • I have searched the repository for related/existing bug reports
  • I have all the details the issue requires

Please answer the following prompt

  • This issue is replicable using the unmodified sample application

Describe the bug

  • The SDK provides a way to filter network interfaces so that ICE candidates are setup only on interfaces selected by the user.
  • However the socket created on the particular interface does not guarantee that packets are routed on the interface.

Did some more digging to find out that the socket creation is missing SO_BINDTODEVICE options.
https://stackoverflow.com/questions/14478167/bind-socket-to-network-interface

Expected Behavior

Packets for an ice candidate are routed over the interface for which ICE candidate is created.

Current Behavior

Packets route over the default interface of the device.

Reproduction Steps

Create a webrtc session with network interface filter not being the default route interface for the device.

WebRTC C SDK version being used

1.7.3

If it was working in a previous version, which one?

No response

Compiler and Version used

gcc-9.4.0

Operating System and version

linux-ubuntu

Platform being used

amd64; arm64

@apurohitrivian apurohitrivian added bug Something isn't working needs-triage labels Apr 16, 2024
@disa6302
Copy link
Contributor

@apurohitrivian ,

Thank you for the details. Will look into this and revert back.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working pending-research v1.7.3
Projects
None yet
Development

No branches or pull requests

3 participants