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

feat(conn): expose socket timeout methods #86

Merged
merged 1 commit into from
Sep 9, 2024

Conversation

flotter
Copy link
Contributor

@flotter flotter commented Aug 23, 2024

Allow cancelling specific blocking netlink I/O operations.

This is available in other netlink derivative libraries such as https://github.com/jsimonetti/rtnetlink, and it can be used to implement request specific cancellation support without closing the file descriptor.

@flotter flotter force-pushed the expose-socket-timeout-methods branch from e55490a to 3c327c9 Compare August 30, 2024 15:08
@flotter
Copy link
Contributor Author

flotter commented Aug 30, 2024

I will rebase this on main, following acceptance of #87

@flotter flotter force-pushed the expose-socket-timeout-methods branch 2 times, most recently from f79fae7 to 1d10df9 Compare September 3, 2024 08:29
Allow cancelling specific blocking netlink I/O operations.

This is available in other netlink derivatives libraries such as
https://github.com/jsimonetti/rtnetlink, and it can be used to
implement request specific cancellation support without closing
the file descriptor.

Signed-off-by: Fred Lotter <[email protected]>
@flotter flotter force-pushed the expose-socket-timeout-methods branch from 1d10df9 to 13b037f Compare September 3, 2024 08:35
@flotter
Copy link
Contributor Author

flotter commented Sep 3, 2024

@SuperQ Ready for review. Thank you!

@flotter
Copy link
Contributor Author

flotter commented Sep 8, 2024

@mdlayher @SuperQ Any chance we could merge this? or any advice what I could do to move this forward please?

@SuperQ SuperQ merged commit e202ab5 into mdlayher:main Sep 9, 2024
6 checks passed
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