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: connect: Add source address option #145

Merged
merged 1 commit into from
Mar 21, 2023

Conversation

kgraefe
Copy link
Contributor

@kgraefe kgraefe commented Mar 17, 2023

This option allows to specify the source address and port for outgoing UDP packets.

Resolves #130

@kgraefe kgraefe changed the title udp: Add source address option udp: connect: Add source address option Mar 17, 2023
This option allows to specify the source address and port for outgoing
UDP packets.

Signed-off-by: Konrad Gräfe <kgraefe@paktolos.net>
Copy link
Owner

@lemunozm lemunozm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great! Thanks again for your awesome and clean work @kgraefe

I think when the config gets longer, we could define it with the builder pattern. For now, it's ok!

@lemunozm lemunozm merged commit 40d6f2b into lemunozm:master Mar 21, 2023
@lemunozm lemunozm mentioned this pull request Mar 21, 2023
3 tasks
@kgraefe
Copy link
Contributor Author

kgraefe commented Mar 21, 2023

I think when the config gets longer, we could define it with the builder pattern. For now, it's ok!

Oh yes, I'd like that!

@kgraefe kgraefe deleted the udp-source-address branch March 21, 2023 11:49
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.

Modify some parameters of udp connection
2 participants