Fixed Bad Attachment Web/API Error; refs #168

apprise_api/api/tests/test_attachment.py

@@ -283,6 +283,10 @@ def test(*args, **kwargs):
         with self.assertRaises(ValueError):
             parse_attachments(attachment_payload, {})
 
+        # We allow empty entries, this is okay; there is just nothing
+        # returned at the end of the day
+        assert parse_attachments({''}, {}) == []
+
         # We can't parse entries that are not base64 but specified as
         # though they are
         attachment_payload = {

apprise_api/api/utils.py

@@ -178,7 +178,6 @@ def __init__(self, filename, delete=True, **kwargs):
                 'Could not prepare {} attachment in {}'.format(
                     filename, settings.APPRISE_ATTACH_DIR))
 
-
         # Prepare our item
         super().__init__(name=filename, **kwargs)
 
@@ -272,6 +271,14 @@ def parse_attachments(attachment_payload, files_request):
             # Prepare our Attachment
             #
             if isinstance(entry, str):
+                if not entry.strip():
+                    # ignore blank entries; these can come from using the
+                    # api/website and submitting without an element defined.
+                    # There is no need have a bad outcome; just decrement our
+                    # counter and move along
+                    count -= 1
+                    continue
+
                 if not re.match(r'^https?://.+', entry[:10], re.I):
                     # We failed to retrieve the product
                     raise ValueError(