Allow extended characters in ID of email address (#693)

apprise/plugins/NotifyForm.py

@@ -129,7 +129,8 @@ class NotifyForm(NotifyBase):
         },
     }
 
-    def __init__(self, headers=None, method=None, payload=None, params=None, **kwargs):
+    def __init__(self, headers=None, method=None, payload=None, params=None,
+                 **kwargs):
         """
         Initialize Form Object
 

apprise/plugins/NotifyXML.py

@@ -136,7 +136,8 @@ class NotifyXML(NotifyBase):
         },
     }
 
-    def __init__(self, headers=None, method=None, payload=None, params=None, **kwargs):
+    def __init__(self, headers=None, method=None, payload=None, params=None,
+                 **kwargs):
         """
         Initialize XML Object
 

apprise/utils.py

@@ -153,10 +153,10 @@ URL_DETAILS_RE = re.compile(
 #   - user@example.com
 #   - label+user@example.com
 GET_EMAIL_RE = re.compile(
-    r'(([\s"\']+)?(?P<name>[^:<"\']+)?[:<\s"\']+)?'
+    r'(([\s"\']+)?(?P<name>[^:<\'"]+)?[:<\s\'"]+)?'
     r'(?P<full_email>((?P<label>[^+]+)\+)?'
-    r'(?P<email>(?P<userid>[a-z0-9$%=_~-]+'
-    r'(?:\.[a-z0-9$%+=_~-]+)'
+    r'(?P<email>(?P<userid>[a-z0-9_!#$%&*/=?%`{|}~^-]+'
+    r'(?:\.[a-z0-9_!#$%&\'*/=?%`{|}~^-]+)'
     r'*)@(?P<domain>('
     r'(?:[a-z0-9](?:[a-z0-9_-]*[a-z0-9])?\.)+'
     r'[a-z0-9](?:[a-z0-9_-]*[a-z0-9]))|'
@@ -188,7 +188,7 @@ URL_DETECTION_RE = re.compile(
 
 EMAIL_DETECTION_RE = re.compile(
     r'[\s,]*([^@]+@.*?)(?=$|[\s,]+'
-    + r'(?:[^:<]+?[:<\s]+?)?'
+    r'(?:[^:<]+?[:<\s]+?)?'
     r'[^@\s,]+@[^\s,]+)',
     re.IGNORECASE)
 

test/test_apprise_utils.py

@@ -1361,6 +1361,30 @@ def test_is_email():
     assert utils.is_email("Just A Name") is False
     assert utils.is_email("Name <bademail>") is False
 
+    # Extended valid emails
+    #
+    # The first + denotes our label, so this test really validates
+    # that there is a correct split and parsing of our email
+    results = utils.is_email('a-z0-9_!#$%&*+/=?%`{|}~^.-@gmail.com')
+    assert '' == results['name']
+    assert 'a-z0-9_!#$%&*' == results['label']
+    assert '/=?%`{|}~^.-@gmail.com' == results['email']
+    assert 'a-z0-9_!#$%&*+/=?%`{|}~^.-@gmail.com' == results['full_email']
+    assert 'gmail.com' == results['domain']
+    assert '/=?%`{|}~^.-' == results['user']
+
+    # A similar test without '+' (use of a label)
+
+    # The first + denotes our label, so this test really validates
+    # that there is a correct split and parsing of our email
+    results = utils.is_email('a-z0-9_!#$%&*/=?%`{|}~^.-@gmail.com')
+    assert '' == results['name']
+    assert '' == results['label']
+    assert 'a-z0-9_!#$%&*/=?%`{|}~^.-@gmail.com' == results['email']
+    assert 'a-z0-9_!#$%&*/=?%`{|}~^.-@gmail.com' == results['full_email']
+    assert 'gmail.com' == results['domain']
+    assert 'a-z0-9_!#$%&*/=?%`{|}~^.-' == results['user']
+
 
 def test_is_call_sign_no():
     """

test/test_config_base.py

@@ -504,6 +504,7 @@ urls:
     - invalid: test
   - sns://T1JJ3T3L2/:
     - invalid: test
+    - _invalid: Token can not start with an underscore
 
   # some strangeness
   -

test/test_plugin_email.py

@@ -191,7 +191,7 @@ TEST_URLS = (
         'response': False,
     }),
     # Valid URL, but can't structure a proper email
-    ('mailtos://nuxref.com?user=%20!&pass=.', {
+    ('mailtos://nuxref.com?user=%20"&pass=.', {
         'instance': TypeError,
     }),
     # Invalid From (and To) Address

test/test_plugin_mailgun.py

@@ -58,7 +58,7 @@ apprise_url_tests = (
         'instance': TypeError,
     }),
     # Invalid from email address
-    ('mailgun://!@localhost.localdomain/{}-{}-{}'.format(
+    ('mailgun://"@localhost.localdomain/{}-{}-{}'.format(
         'a' * 32, 'b' * 8, 'c' * 8), {
         'instance': TypeError,
     }),

test/test_plugin_smtp2go.py

@@ -58,7 +58,7 @@ apprise_url_tests = (
         'instance': TypeError,
     }),
     # Invalid from email address
-    ('smtp2go://!@localhost.localdomain/{}-{}-{}'.format(
+    ('smtp2go://"@localhost.localdomain/{}-{}-{}'.format(
         'a' * 32, 'b' * 8, 'c' * 8), {
         'instance': TypeError,
     }),

test/test_plugin_sparkpost.py

@@ -59,7 +59,7 @@ apprise_url_tests = (
         'instance': TypeError,
     }),
     # Invalid from email address
-    ('sparkpost://!@localhost.localdomain/{}'.format('b' * 32), {
+    ('sparkpost://"@localhost.localdomain/{}'.format('b' * 32), {
         'instance': TypeError,
     }),
     # No To email address, but everything else is valid