UDP socket improvements #4216

Merged
merged 12 commits into from Oct 11, 2016

Projects

None yet

2 participants

@etehtsea
Contributor

No description provided.

@etehtsea
Contributor
etehtsea commented Oct 10, 2016 edited

Ok, there is a regression that needs to be fixed.

headius and others added some commits Jan 2, 2016
@headius @etehtsea headius Clean up unused imports. 093d234
@headius @etehtsea headius Properly get address family for UDPSocket#initialize. 9bcfdd8
@etehtsea etehtsea Convert IPv6 addresses using SocketUtilsIPV6 3f36e15
@etehtsea etehtsea Refactor SocketUtilsIPV6 methods to be static 76120f9
@etehtsea etehtsea Add UDPSocket.new specs 6227536
@etehtsea etehtsea Remove ipv6 scope before parsing d73a809
@headius @etehtsea headius Fixes for UDPSocket.send based on rubysl-socket specs.
* Handle Addrinfo or packed sockaddr as destination.
* Try all addresses for a given hostname.
b56a192
@headius @etehtsea headius Last fixes for UDPSocket based on rubysl-socket specs.
* Handle family argument to initialize better.
* Try all addresses for connect when no explicit family specified.
* Raise proper error for wrong family.
9337fdd
@etehtsea etehtsea Untag related mri tests d659903
@etehtsea etehtsea Add spec for EAFNOSUPPORT in udp sockets 2c34eff
@etehtsea etehtsea Return short ipv6 adresses in unpack_sockaddr_in
4207acd
@etehtsea
Contributor
etehtsea commented Oct 10, 2016 edited

Ready for reviewing.

@etehtsea etehtsea Fix Addrinfo initialization
Fixed initialization with nil nodename
`Addrinfo.new(['AF_INET6', 59656, nil, '::1'], 30, 1, 6)`
a5bf7de
@headius headius added this to the JRuby 9.1.6.0 milestone Oct 11, 2016
@headius headius merged commit 1a6f27a into jruby:master Oct 11, 2016

0 of 2 checks passed

continuous-integration/appveyor/pr AppVeyor build failed
Details
continuous-integration/travis-ci/pr The Travis CI build is in progress
Details
@headius
Member
headius commented Oct 11, 2016

Thanks again :-)

@etehtsea etehtsea deleted the etehtsea:socket-udp-improvements branch Oct 11, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment