-
Notifications
You must be signed in to change notification settings - Fork 629
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
Add support for SO_BINDTODEVICE option #408
Conversation
Any idea why CI jobs got stuck here? |
3b24f10
to
9da06a8
Compare
I just tweaked your commit and force pushed it just for the effect of poking CI in the eye with a sharp stick. It wasn't giving me the option to restart jobs either. Maybe they will run now. |
9da06a8
to
6fabbb3
Compare
6fabbb3
to
e485f75
Compare
@alerque Ah thanks :) Yeah that seemed to work, I just added a fix to handle the call macOS where the option does not exist. Should pass the CI now. |
Please feel free to poke it again in case it does not start in a bit, perhaps it only triggers on commits signed by maintainers. |
e485f75
to
7084840
Compare
This PR has broken my static Windows build, because |
This PR adds support for setting
SO_BINDTODEVICE
throughmaster:setoption()
.In combination with
:bind()
this allows for binding to unix tunnel interfaces (e.g. an OpenVPN connection), aka source based split tunneling.Example