[ https://issues.apache.org/jira/browse/AIRFLOW-3602?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16752684#comment-16752684 ]
ASF subversion and git services commented on AIRFLOW-3602: ---------------------------------------------------------- Commit 055d86f53bc0fae3c2835720feeba6fb61160f8c in airflow's branch refs/heads/master from Felix [ https://gitbox.apache.org/repos/asf?p=airflow.git;h=055d86f ] [AIRFLOW-3602] Changes ImapHook to raise an exception on 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)