New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
test_urllibnet.test_bad_address() fails when using OpenDNS #47833
Comments
OpenDNS has a "feature" where if you enter an address that doesn't Probably should just change the test such that if no exception is |
Are many people using OpenDNS? Is there a way to detect that OpenDNS is |
On Mon, Aug 18, 2008 at 3:52 AM, Antoine Pitrou <report@bugs.python.org> wrote:
That's also a possibility. Out of curiosity, what HTTP response should be received? |
Le lundi 18 août 2008 à 17:19 +0000, Brett Cannon a écrit :
There shouldn't be an HTTP response at all. If DNS lookup fails, |
I got a similar problem but here the ISP returned an error/search page >>> import http.client
>>> conn = http.client.HTTPConnection("www.somerandominvalidpage.edu")
>>> conn.request("GET", "/")
>>> r1 = conn.getresponse()
>>> r1.status, r1.reason
(302, 'Found') test_xmlrpc_net failed for the same reason (see bpo-6027). |
I think the best we can do here is add a message explaining that the error may be due to a broken DNS server (one with a wildcard dns record for all non-existent top level domains). However, assertRaises, even in context manager form, doesn't take a msg argument (yet). I've opened an issue with a feature request to fix that and made it a dependency of this issue. Note that the test uses a domain name ending in ".d", and for a while before that used '.invalid', so the test should not fail if the ISP is only capturing valid top level domains with wildcards, something that seems to be far more common than catching invalid domains. |
New changeset acce13a6e728 by Brett Cannon in branch 'default': |
Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.
Show more details
GitHub fields:
bugs.python.org fields:
The text was updated successfully, but these errors were encountered: