Skip to content
Browse files

Fixed #778 -- Improved isExistingURL validator not to raise Validatio…

…nError for URLs that exist but require authorization. Thanks for the report, lakin wrecker.

git-svn-id: bcc190cf-cafb-0310-a4f2-bffc1f526a37
  • Loading branch information...
1 parent 1bf6dd7 commit 0a74c68eeebb827736f2370dc67d0d72619025f2 @adrianholovaty adrianholovaty committed
Showing with 5 additions and 1 deletion.
  1. +5 −1 django/core/
6 django/core/
@@ -199,7 +199,11 @@ def isExistingURL(field_data, all_data):
u = urllib2.urlopen(field_data)
except ValueError:
raise ValidationError, _("Invalid URL: %s") % field_data
- except: # urllib2.HTTPError, urllib2.URLError, httplib.InvalidURL, etc.
+ except urllib2.HTTPError, e:
+ # 401s are valid; they just mean authorization is required.
+ if e.code not in ('401',):
+ raise ValidationError, _("The URL %s is a broken link.") % field_data
+ except: # urllib2.URLError, httplib.InvalidURL, etc.
raise ValidationError, _("The URL %s is a broken link.") % field_data
def isValidUSState(field_data, all_data):

0 comments on commit 0a74c68

Please sign in to comment.
Something went wrong with that request. Please try again.