On Thu Jul 02 20:35:00 2009, todd_rinaldo wrote:
Show quoted text> Right now, you're doing the tests if gethostbyname("www.netscape.com")
> succeeds.
>
> For some strange reason, we're behind a firewall, but netscape is
> resolving.
>
> I suggest this patch so it's disabled if proxy is enabled.
>
> - if (gethostbyname("www.netscape.com") && !gethostbyname
> ("example.xyz"))
> + if (gethostbyname("www.netscape.com") && !gethostbyname
> ("example.xyz") && !$ENV{http_proxy} & !$ENV{HTTP_PROXY})
>
>
Got this as well today. Strangely it seemed fine the first time or two
I installed it, but have been testing a deployment script and wiping it
clean after each go. Suddenly I got this error.
Now, this particular error appears to be the cause of most of the cpan
failures reported, and considering how many other packages depend on URI
now would be great if we could do something...
I'm not sure exactly what this test is trying to do, but it's reliance
of some dubious hosts names is problematic. I guess when this test was
written the author assumed 'www.netscape.com' was pretty safe. I'd say
change it to www.google.com, but that would mess up since so many
countries block google from time to time.
I don't completely understand what this test is hoping to achieve, but
it seems totally wrong to say:
$URI::Heuristic::MY_COUNTRY = "bv";
and then try to match on a .com or something. Maybe .bv was handled
differently all those years ago. The docs say something about if the
country is two letters, assume is the default country, but that is not
clear at all to me what it means.
We should probably just skip this test for now since I don't see what is
going to get hurt. My guess is all the passes have more to do with
netscape.com being unresolvable for most people. Or we can set the dns
lookup to an ENV var and skip the test if it's not present. That way
the authors can set it for developing if there is some logic there which
is important to fix.