[
https://issues.apache.org/jira/browse/AIRFLOW-3602?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16752685#comment-16752685
]
ASF subversion and git services commented on AIRFLOW-3602:
----------------------------------------------------------
Commit aed71a794423cfac819dd576d9111dc2f527e463 in airflow's branch
refs/heads/master from Felix
[ https://gitbox.apache.org/repos/asf?p=airflow.git;h=aed71a7 ]
[AIRFLOW-3602] Improve ImapHook handling of retrieving no attachments (#4475)
> Change ImapHook to raise an Exception when no attachments have been found
> -------------------------------------------------------------------------
>
> Key: AIRFLOW-3602
> URL: https://issues.apache.org/jira/browse/AIRFLOW-3602
> Project: Apache Airflow
> Issue Type: Improvement
> Affects Versions: 1.10.1
> Reporter: Felix Uellendall
> Assignee: Felix Uellendall
> Priority: Major
>
> At the moment the
> {code:python}
> def retrieve_mail_attachments(self, name, mail_folder='INBOX',
> check_regex=False,
> latest_only=False):
> {code}
> returns an empty list if no attachments have been found.
> And the
> {code:python}
> def download_mail_attachments(self, name, local_output_directory,
> mail_folder='INBOX',
> check_regex=False, latest_only=False):
> {code}
> also does not log anything if no attachment has been found.
> Is this useful? Or would it be better to raise an exception instead?
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
