Skip Menu |

This queue is for tickets about the Net-Server-Coro CPAN distribution.

Report information
The Basics
Id: 81089
Status: resolved
Priority: 0/
Queue: Net-Server-Coro
People
Owner: Nobody in particular
Requestors: Xavier (no email address)
Cc:
AdminCc:
Bug Information
Severity: Unimportant
Broken in: 1.2
Fixed in: (no value)

History Show all quoted text
  Sat Nov 10 03:05:34 2012 x.guimard [...] free.fr - Ticket created
Subject: Little warning in tests
Hi, to avoid a little warning in tests, we use the following patch. Thanks, Xavier --- a/lib/Net/Server/Coro.pm +++ b/lib/Net/Server/Coro.pm @@ -102,7 +102,7 @@ my $self = shift; my ($host, $port, $proto) = @_; - my $is_ssl = ($proto eq "SSL" or $proto eq "SSLEAY"); + my $is_ssl = (defined($proto) and ($proto eq "SSL" or $proto eq "SSLEAY")); my $socket = $self->SUPER::proto_object( $host, $port, $is_ssl ? "TCP" : $proto );
  Mon Nov 12 03:31:22 2012 alex [...] chmrr.net - Correspondence added
Subject: Re: [rt.cpan.org #81089] Little warning in tests
Date: Mon, 12 Nov 2012 03:31:02 -0500
To: bug-Net-Server-Coro [...] rt.cpan.org
From: Alex Vandiver <alex [...] chmrr.net>
On Sat, 2012-11-10 at 03:05 -0500, Xavier Guimard via RT wrote: Show quoted text
> to avoid a little warning in tests, we use the following patch.
This is unfortunately not the correct fix -- this warning stems from an API change in Net::Server version 2.0 and above. I've released version 1.3 of Net::Server::Coro which is updated for the new API, and a correspondingly updated required version of Net::Server. - Alex
  Mon Nov 12 03:31:28 2012 The RT System itself - Status changed from 'new' to 'open'
  Mon Nov 12 03:32:57 2012 cpan [...] chmrr.net - Status changed from 'open' to 'resolved'