Solves #6045 by not throwing an exception when an invalid ip address is specified

Andras Gyomrey · lgandras · commit dac7fa778954 · 2017-09-09T20:58:18.000+02:00
diff --git a/cms/middleware/toolbar.py b/cms/middleware/toolbar.py
@@ -2,7 +2,9 @@
 """
 Edit Toolbar middleware
 """
+from django import forms
 from django.contrib.admin.models import LogEntry, ADDITION, CHANGE
+from django.core.exceptions import ValidationError
 from django.core.urlresolvers import resolve
 from django.http import HttpResponse
 
@@ -27,8 +29,13 @@ def is_cms_request(self, request):
 
         if internal_ips:
             client_ip = get_request_ip(request)
-            if client_ip not in internal_ips:
+            try:
+                client_ip = forms.GenericIPAddressField().clean(client_ip)
+            except ValidationError:
                 return False
+            else:
+                if client_ip not in internal_ips:
+                    return False
 
         if not toolbar_hide:
             return True
