Skip Menu |
 

This queue is for tickets about the HTTP-Proxy CPAN distribution.

Report information
The Basics
Id: 27922
Status: resolved
Priority: 0/
Queue: HTTP-Proxy

People
Owner: Nobody in particular
Requestors: sshah [...] RisingEdge.org
Cc:
AdminCc:

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



Subject: wildcard records break tests in 22http.t and 22transparent.t
Date: Mon, 2 Jul 2007 22:46:32 -0700
To: <bug-HTTP-Proxy [...] rt.cpan.org>
From: "Steve Shah" <sshah [...] RisingEdge.org>
Download (untitled) / with headers
text/plain 386b
In the test cases, there is reference to "www.example.zzz". Note that anyone whose resolv.conf cites a domain that has a wildcard record, e.g., *.planetoid.org and is running a web server will cause the test to incorrectly fail. Change "www.example.zzz" to "www.example.zzz." (note the extra period at the end) to make your tests work as intended. -- Steve Shah sshah@planetoid.org
Subject: evil DNS breaks tests in 22http.t and 22transparent.t
RT-Send-CC: book [...] cpan.org
Download (untitled) / with headers
text/plain 543b
On Tue Jul 03 01:46:59 2007, sshah@RisingEdge.org wrote: Show quoted text
> Change "www.example.zzz" to "www.example.zzz." (note the > extra period at the end) to make your tests work as intended.
That doesn't work for me (verizon's evil DNS servers want to sell me a domain or something), but removing the "www." does work here. This test should probably just accept 404 and move on (diag() a rant if you like) because the marketing people aren't going to quit breaking DNS anytime soon and in the meantime it makes the module a pain to install. --Eric
From: book [...] cpan.org
Download (untitled) / with headers
text/plain 705b
On Sat Feb 09 18:39:51 2008, EWILHELM wrote: Show quoted text
> On Tue Jul 03 01:46:59 2007, sshah@RisingEdge.org wrote:
> > Change "www.example.zzz" to "www.example.zzz." (note the > > extra period at the end) to make your tests work as intended.
> > That doesn't work for me (verizon's evil DNS servers want to sell me a > domain or something), but removing the "www." does work here. > > This test should probably just accept 404 and move on (diag() a rant > if you like) because the marketing people aren't going to quit > breaking DNS anytime soon and in the meantime it makes the module a > pain to install.
My ISP broke the DNS too, since last August. I guess I'll have to change those tests, yes. -- BooK
Download (untitled) / with headers
text/plain 267b
On Sat Feb 09 19:23:58 2008, BOOK wrote: Show quoted text
> > My ISP broke the DNS too, since last August. > > I guess I'll have to change those tests, yes. >
The tests now do a DNS request, and if it doesn't fail as expected, they skip the test. This will be in 0.21. -- BooK


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.