diff --git a/sat/plugins/plugin_misc_attach.py b/sat/plugins/plugin_misc_attach.py index 8314e695..7512bfc4 100644 --- a/sat/plugins/plugin_misc_attach.py +++ b/sat/plugins/plugin_misc_attach.py @@ -59,15 +59,19 @@ class AttachPlugin: for attachment in attachments: try: - path = Path(attachment["path"]) + # we pop path because we don't want it to be stored, as the image can be + # only in a temporary location + path = Path(attachment.pop("path")) except KeyError: log.warning("no path in attachment: {attachment}") to_delete.append(attachment) continue if "url" in attachment: - log.warning("invalid attachment: unknown URL") - to_delete.append(attachment) + url = attachment.pop('url') + log.warning( + f"unexpected URL in attachment: {url!r}\nattachment: {attachment}" + ) try: name = attachment["name"]