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

If connect fails let other side accept connection and come back #29

Closed
wants to merge 1 commit into from

Conversation

e4t
Copy link
Contributor

@e4t e4t commented Jul 18, 2018

The UNIX socket code uses inotify to get notified when a socket
it monitors appears. The inotify is triggered when the other side
does bind(). However, we are not able to connect until the other
side calls listen(). Therefore, we need to let the other side continue
if the connect fails and should come back at the earliest time to
attempt a connect() again.
For this we just reset the poll() delay.
This fixes issue #28.

Signed-off-by: Egbert Eich eich@suse.com

The UNIX socket code uses inotify to get notified when a socket
it monitors appears. The inotify is triggered when the other side
does bind(). However, we are not able to connect until the other
side calls listen(). Therefore, we need to let the other side continue
if the connect fails and should come back at the earliest time to
attempt a connect() again.
For this we just reset the poll() delay.

Signed-off-by: Egbert Eich <eich@suse.com>
@dun
Copy link
Owner

dun commented Jul 21, 2018

Just a quick ack... I'm hoping to have time to look at this next week.

@dun dun added the bug label Jul 28, 2018
@dun dun added this to the 0.3.0 milestone Jul 28, 2018
@dun dun closed this in 8f5bd09 Jul 28, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants