Skip to content
This repository was archived by the owner on Sep 11, 2026. It is now read-only.

Add UDP notifier layer - #387

Merged
SeanTAllen merged 1 commit into
mainfrom
udp-notifier
Sep 2, 2026
Merged

SeanTAllen merged 1 commit into
mainfrom
udp-notifier

Conversation

@SeanTAllen

Copy link
Copy Markdown
Member

A convenience layer over lori's UDP socket, following the TCP notifier pattern. One trait (UDPSocketNotify) and one actor (UDPSocket) cover UDP without any actor boilerplate.

send_to is synchronous from callbacks so the caller can act on the result. write_to is a fire-and-forget behavior for sending from other actors — it calls send_to internally and discards the result. This differs from TCP's notifier, where write is always a behavior, because UDP has no send-completion callbacks and losing the result silently is a footgun.

Design: #384

A convenience layer over lori's UDP socket, following the TCP notifier
pattern. One trait (UDPSocketNotify) and one actor (UDPSocket) cover UDP
without any actor boilerplate.

send_to is synchronous from callbacks so the caller can act on the result.
write_to is a fire-and-forget behavior for sending from other actors.

Design: #384
@ponylang-main ponylang-main added the discuss during sync Should be discussed during an upcoming sync label Sep 2, 2026
@SeanTAllen
SeanTAllen merged commit 8dcb4fe into main Sep 2, 2026
12 checks passed
@SeanTAllen
SeanTAllen deleted the udp-notifier branch September 2, 2026 21:35
@ponylang-main ponylang-main removed the discuss during sync Should be discussed during an upcoming sync label Sep 2, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants