mirror of
				https://github.com/django/django.git
				synced 2025-10-26 07:06:08 +00:00 
			
		
		
		
	Fixed #3421 -- Added IP and localhost validation to newforms URLField. Thanks, SmileyChris.
git-svn-id: http://code.djangoproject.com/svn/django/trunk@6152 bcc190cf-cafb-0310-a4f2-bffc1f526a37
This commit is contained in:
		| @@ -335,12 +335,6 @@ class EmailField(RegexField): | |||||||
|         RegexField.__init__(self, email_re, max_length, min_length, |         RegexField.__init__(self, email_re, max_length, min_length, | ||||||
|             ugettext(u'Enter a valid e-mail address.'), *args, **kwargs) |             ugettext(u'Enter a valid e-mail address.'), *args, **kwargs) | ||||||
|  |  | ||||||
| url_re = re.compile( |  | ||||||
|     r'^https?://' # http:// or https:// |  | ||||||
|     r'(?:[A-Z0-9-]+\.)+[A-Z]{2,6}' # domain |  | ||||||
|     r'(?::\d+)?' # optional port |  | ||||||
|     r'(?:/?|/\S+)$', re.IGNORECASE) |  | ||||||
|  |  | ||||||
| try: | try: | ||||||
|     from django.conf import settings |     from django.conf import settings | ||||||
|     URL_VALIDATOR_USER_AGENT = settings.URL_VALIDATOR_USER_AGENT |     URL_VALIDATOR_USER_AGENT = settings.URL_VALIDATOR_USER_AGENT | ||||||
| @@ -399,6 +393,14 @@ class ImageField(FileField): | |||||||
|             raise ValidationError(ugettext(u"Upload a valid image. The file you uploaded was either not an image or a corrupted image.")) |             raise ValidationError(ugettext(u"Upload a valid image. The file you uploaded was either not an image or a corrupted image.")) | ||||||
|         return f |         return f | ||||||
|  |  | ||||||
|  | url_re = re.compile( | ||||||
|  |     r'^https?://' # http:// or https:// | ||||||
|  |     r'(?:(?:[A-Z0-9-]+\.)+[A-Z]{2,6}|' #domain... | ||||||
|  |     r'localhost|' #localhost... | ||||||
|  |     r'\d{1,3}\.\d{1,3}\.\d{1,3}\.\d{1,3})' # ...or ip | ||||||
|  |     r'(?::\d+)?' # optional port | ||||||
|  |     r'(?:/?|/\S+)$', re.IGNORECASE) | ||||||
|  |  | ||||||
| class URLField(RegexField): | class URLField(RegexField): | ||||||
|     def __init__(self, max_length=None, min_length=None, verify_exists=False, |     def __init__(self, max_length=None, min_length=None, verify_exists=False, | ||||||
|             validator_user_agent=URL_VALIDATOR_USER_AGENT, *args, **kwargs): |             validator_user_agent=URL_VALIDATOR_USER_AGENT, *args, **kwargs): | ||||||
|   | |||||||
| @@ -1607,10 +1607,18 @@ ValidationError: [u'This field is required.'] | |||||||
| Traceback (most recent call last): | Traceback (most recent call last): | ||||||
| ... | ... | ||||||
| ValidationError: [u'This field is required.'] | ValidationError: [u'This field is required.'] | ||||||
|  | >>> f.clean('http://localhost') | ||||||
|  | u'http://localhost' | ||||||
| >>> f.clean('http://example.com') | >>> f.clean('http://example.com') | ||||||
| u'http://example.com' | u'http://example.com' | ||||||
| >>> f.clean('http://www.example.com') | >>> f.clean('http://www.example.com') | ||||||
| u'http://www.example.com' | u'http://www.example.com' | ||||||
|  | >>> f.clean('http://www.example.com:8000/test') | ||||||
|  | u'http://www.example.com:8000/test' | ||||||
|  | >>> f.clean('http://200.8.9.10') | ||||||
|  | u'http://200.8.9.10' | ||||||
|  | >>> f.clean('http://200.8.9.10:8000/test') | ||||||
|  | u'http://200.8.9.10:8000/test' | ||||||
| >>> f.clean('foo') | >>> f.clean('foo') | ||||||
| Traceback (most recent call last): | Traceback (most recent call last): | ||||||
| ... | ... | ||||||
|   | |||||||
		Reference in New Issue
	
	Block a user