Skip Menu |

This queue is for tickets about the WWW-Offliberty CPAN distribution.

Report information
The Basics
Id: 110057
Status: resolved
Priority: 0/
Queue: WWW-Offliberty

People
Owner: marius [...] ieval.ro
Requestors: SREZIC [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: 1.000
Fixed in: (no value)



Subject: t/50-network.t fails
Currently the t/50-network.t fails: # Failed test 'youtube, video_file => 1' # at t/50-network.t line 14. # got: '1' # expected: '2' # Looks like you failed 1 test of 2. t/50-network.t .. Dubious, test returned 1 (wstat 256, 0x100) Failed 1/2 subtests Don't know if this is a permanent or transient failure.
On 2015-12-03 22:23:38, SREZIC wrote: Show quoted text
> Currently the t/50-network.t fails: > > # Failed test 'youtube, video_file => 1' > # at t/50-network.t line 14. > # got: '1' > # expected: '2' > # Looks like you failed 1 test of 2. > t/50-network.t .. > Dubious, test returned 1 (wstat 256, 0x100) > Failed 1/2 subtests > > Don't know if this is a permanent or transient failure.
Tests that use the network should respect the various environment variables that users can set to opt out of this behaviour, and also be mindful that some installations occur behind firewalls. Test::RequiresInternet is a good module that handles these cases well, allowing network tests to be skipped gracefully.
Subject: Re: [rt.cpan.org #110057] t/50-network.t fails
Date: Fri, 04 Dec 2015 17:46:06 +0000
To: "Karen Etheridge via RT" <bug-WWW-Offliberty [...] rt.cpan.org>
From: Marius Gavrilescu <marius [...] ieval.ro>
"Karen Etheridge via RT" <bug-WWW-Offliberty@rt.cpan.org> writes: Show quoted text
> Queue: WWW-Offliberty > Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=110057 > > > On 2015-12-03 22:23:38, SREZIC wrote:
>> Currently the t/50-network.t fails: >> >> # Failed test 'youtube, video_file => 1' >> # at t/50-network.t line 14. >> # got: '1' >> # expected: '2' >> # Looks like you failed 1 test of 2. >> t/50-network.t .. >> Dubious, test returned 1 (wstat 256, 0x100) >> Failed 1/2 subtests >> >> Don't know if this is a permanent or transient failure.
> > > Tests that use the network should respect the various environment > variables that users can set to opt out of this behaviour, and also be > mindful that some installations occur behind firewalls. > > Test::RequiresInternet is a good module that handles these cases well, > allowing network tests to be skipped gracefully.
The test already uses Test::RequiresInternet. The cause of the bug is the addition of some robot protection in the Offliberty service. If the Referer header is unset, it does not work. I'll release a new version that fixes this (by setting the Referer header) in a few days. -- Marius Gavrilescu
Download signature.asc
application/pgp-signature 818b

Message body not shown because it is not plain text.

Fixed in 1.000001 by adding a Referer: header to the request.