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

Add option to bind to interface #229

Merged
merged 2 commits into from
Jul 18, 2023
Merged

Add option to bind to interface #229

merged 2 commits into from
Jul 18, 2023

Conversation

Diederik98
Copy link
Contributor

@Diederik98 Diederik98 commented Apr 11, 2023

Bind immediately to all addresses of the given interface instead of a -p for each of them.
Restarting the dropbear application is no longer necessary when one of the addresses changes.

A new option -l allows to set the interface.

@mkj mkj merged commit 35fa447 into mkj:master Jul 18, 2023
19 of 32 checks passed
@mkj
Copy link
Owner

mkj commented Jul 18, 2023

Thanks, sorry for the delay getting to this. I'll add some guards so it's OK cross-platform.

One idea, I wonder if it's better to fail binding if the setsockopt() fails (because it isn't supported, most likely), rather than listening on wider interfaces. Any thoughts?

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