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

udp: add optional send/recv handler #602

Merged
merged 11 commits into from
Dec 8, 2022
Merged

Conversation

alfredh
Copy link
Contributor

@alfredh alfredh commented Nov 29, 2022

usecase: use udp_sock with external/custom sockets

@alfredh alfredh added this to the v2.11.0 milestone Nov 29, 2022
@alfredh alfredh marked this pull request as draft December 5, 2022 11:31
@alfredh alfredh removed this from the v2.11.0 milestone Dec 7, 2022
@alfredh
Copy link
Contributor Author

alfredh commented Dec 8, 2022

hi @sreimers

could you please take a look at this one

The usecase is to use external send handlers or external filedescriptors:

  1. Application creates a socket (fd) and then allocates udp_sock with that FD.
  2. Application creates udp_sock with custom send handler, and uses that one instead of FD.

Alfred

@alfredh alfredh marked this pull request as ready for review December 8, 2022 17:46
@sreimers
Copy link
Member

sreimers commented Dec 8, 2022

Looks good, thanks!

@sreimers sreimers merged commit 6c7f92a into main Dec 8, 2022
@sreimers sreimers deleted the udp_sock_external_sendrecv branch December 8, 2022 20:01
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