Bug #129909 for IO-Socket-ByteCounter: test calls $socket->send() with a TO parameter on a connected socket

Tue Jun 25 20:12:08 2019 TONYC [...] cpan.org - Ticket created

Subject:

test calls $socket->send() with a TO parameter on a connected socket

The tests for IO::Socket::ByteCounter call send() with a TO parameter on a connected socket. Recently in bleadperl, the IO::Socket send() method was changed to always supply the TO parameter to the underlying CORE::send() function if the TO parameter is supplied to IO::Socket::send(), since this works for at least UDP sockets on Linux. It doesn't work for UNIX sockets, and means the tests hang due to the failed send call (which the test doesn't check for.) I suggest removing the 'peer' parameter from both calls to send() in the test code. Tony reported against perl as https://rt.perl.org/Ticket/Display.html?id=134224

Wed Apr 01 07:35:55 2020 jkeenan [...] cpan.org - Correspondence added

On Tue Jun 25 20:12:08 2019, TONYC wrote: Show quoted text

> The tests for IO::Socket::ByteCounter call send() with a TO parameter > on a connected socket. > > Recently in bleadperl, the IO::Socket send() method was changed to > always supply the TO parameter to the underlying CORE::send() function > if the TO parameter is supplied to IO::Socket::send(), since this > works for at least UDP sockets on Linux. > > It doesn't work for UNIX sockets, and means the tests hang due to the > failed send call (which the test doesn't check for.) > > I suggest removing the 'peer' parameter from both calls to send() in > the test code. > > Tony > > reported against perl as > https://rt.perl.org/Ticket/Display.html?id=134224

Daniel Muey: The release of perl-5.32 is nearing, after which this module will fail its tests. Can you try out the fix recommended by Tony Cook above? Thank you very much. Jim Keenan

Wed Apr 01 07:35:55 2020 The RT System itself - Status changed from 'new' to 'open'

Wed Apr 01 09:21:14 2020 dmuey [...] cpan.org - Correspondence added

I'll try to prioritize this thanks! n Wed Apr 01 07:35:55 2020, JKEENAN wrote: Show quoted text

> On Tue Jun 25 20:12:08 2019, TONYC wrote:

> > The tests for IO::Socket::ByteCounter call send() with a TO parameter > > on a connected socket. > > > > Recently in bleadperl, the IO::Socket send() method was changed to > > always supply the TO parameter to the underlying CORE::send() > > function > > if the TO parameter is supplied to IO::Socket::send(), since this > > works for at least UDP sockets on Linux. > > > > It doesn't work for UNIX sockets, and means the tests hang due to the > > failed send call (which the test doesn't check for.) > > > > I suggest removing the 'peer' parameter from both calls to send() in > > the test code. > > > > Tony > > > > reported against perl as > > https://rt.perl.org/Ticket/Display.html?id=134224

> > Daniel Muey: The release of perl-5.32 is nearing, after which this > module will fail its tests. Can you try out the fix recommended by > Tony Cook above? > > Thank you very much. > Jim Keenan

Fri Oct 16 19:42:57 2020 TODDR [...] cpan.org - Correspondence added

On Wed Apr 01 09:21:14 2020, DMUEY wrote: Show quoted text

> I'll try to prioritize this thanks! >

I believe this is now broken on 5.32.0

Thu Oct 22 10:00:25 2020 dmuey [...] cpan.org - Status changed from 'open' to 'resolved'

Thu Oct 22 10:01:32 2020 dmuey [...] cpan.org - Correspondence added

Thanks! Resolved in v0.3 just uploaded to CPAN