Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Feature net::UdpSocket connect, send+to, recv+from, etc. #33

Merged
merged 23 commits into from
Jun 27, 2022

Conversation

pinkforest
Copy link
Contributor

@pinkforest pinkforest commented Jun 25, 2022

Working on #17

Implements

  • connect()
  • send(), send_to()
  • recv(), recv_from()
  • set_ttl(), ttl()
  • set_broadcast, broadcast()
  • set, get, use timeouts
  • dummy fn's set_nonblocking & take_error

Notes

Moar Notes

  • UdpSocket IPv6 addr bind accidentally called tcp_bind 😱 🤦‍♀️ - fixed it 😰 🥵
  • As previous consider in future to refactor for addr in addr.to_socket_addrs() like std net's each_addr

@pinkforest pinkforest changed the title feat/udp connect Feature net::UdpSocket connect Jun 25, 2022
@pinkforest pinkforest mentioned this pull request Jun 25, 2022
@pinkforest pinkforest changed the title Feature net::UdpSocket connect Feature net::UdpSocket connect, send, recv, peer_addr (?) Jun 25, 2022
@pinkforest pinkforest mentioned this pull request Jun 26, 2022
@pinkforest pinkforest changed the title Feature net::UdpSocket connect, send, recv, peer_addr (?) Feature net::UdpSocket connect, send+to, recv+from Jun 26, 2022
@pinkforest pinkforest changed the title Feature net::UdpSocket connect, send+to, recv+from Feature net::UdpSocket connect, send+to, recv+from, ttl Jun 26, 2022
@pinkforest pinkforest changed the title Feature net::UdpSocket connect, send+to, recv+from, ttl Feature net::UdpSocket connect, send+to, recv+from, etc. Jun 26, 2022
@bkolobara
Copy link
Contributor

That's awesome work @pinkforest. Thank you so much for the contribution!

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants