use ipaddress package to test

moesifwsgi/client_ip.py

@@ -1,5 +1,6 @@
 import re
 import logging
+import ipaddress
 
 logger = logging.getLogger(__name__)
 
@@ -10,10 +11,12 @@ def __init__(self):
 
     @classmethod
     def is_ip(cls, value):
-        if not value is None:
-            ipv4 = r"^(?:(?:\d|[1-9]\d|1\d{2}|2[0-4]\d|25[0-5])\.){3}(?:\d|[1-9]\d|1\d{2}|2[0-4]\d|25[0-5])$"
-            ipv6 = r"^((?=.*::)(?!.*::.+::)(::)?([\dA-F]{1,4}:(:|\b)|){5}|([\dA-F]{1,4}:){6})((([\dA-F]{1,4}((?!\3)::|:\b|$))|(?!\2\3)){2}|(((2[0-4]|1\d|[1-9])?\d|25[0-5])\.?\b){4})$/i"
-            return re.match(ipv4, value) or re.match(ipv6, value)
+        # https://docs.python.org/3/library/ipaddress.html#ipaddress.ip_address
+        try:
+            ip = ipaddress.ip_address(value);
+            return True;
+        except ValueError:
+            return False;
 
     def getClientIpFromXForwardedFor(self, value):
         try:
@@ -98,4 +101,4 @@ def get_client_address(self, environ):
 
             return environ['REMOTE_ADDR']
         except KeyError:
-            return environ['REMOTE_ADDR']
+            return environ['REMOTE_ADDR']

moesifwsgi/middleware.py

@@ -104,7 +104,7 @@ def initialize_config(self):
             response_catcher = HttpResponseCatcher(self.DEBUG)
             self.api_client.http_call_back = response_catcher
         Configuration.BASE_URI = self.settings.get("BASE_URI") or self.settings.get("LOCAL_MOESIF_BASEURL", "https://api.moesif.net")
-        Configuration.version = "moesifwsgi-python/1.9.6"
+        Configuration.version = "moesifwsgi-python/1.9.7"
         if self.settings.get("CAPTURE_OUTGOING_REQUESTS", False):
             StartCapture().start_capture_outgoing(self.settings)
 

requirements.txt

@@ -5,3 +5,4 @@ moesifapi>=1.5.3
 moesifpythonrequest>=0.3.4
 readerwriterlock==1.0.9
 apscheduler==3.10.4
+ipaddress; python_version < '3.3'

setup.py

@@ -28,7 +28,7 @@
     # Versions should comply with PEP440.  For a discussion on single-sourcing
     # the version across setup.py and the project code, see
     # https://packaging.python.org/en/latest/single_source_version.html
-    version='1.9.6',
+    version='1.9.7',
 
     description='Moesif Middleware for Python WSGI based platforms (Flask, Bottle & Others)',
     long_description=long_description,
@@ -94,6 +94,7 @@
         'moesifpythonrequest>=0.3.4',
         'readerwriterlock>=1.0.9',
         'apscheduler>=3.10.4'
+        'ipaddress;python_version<"3.3"',
     ],
 
     # List additional groups of dependencies here (e.g. development