Skip to content
This repository was archived by the owner on Feb 19, 2021. It is now read-only.

Commit e0da952

Browse files
authored
Merge pull request #539 from bauerj/subdirectories
Warn if consume directory contains subdirectories
2 parents 5317019 + 22c7f30 commit e0da952

File tree

2 files changed

+11
-0
lines changed

2 files changed

+11
-0
lines changed

src/documents/consumer.py

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -94,6 +94,11 @@ def consume_new_files(self):
9494
ignored_files.append(file)
9595
else:
9696
files.append(file)
97+
else:
98+
self.logger.warning(
99+
"Skipping %s as it is not a file",
100+
entry.path
101+
)
97102

98103
if not files:
99104
return

src/documents/management/commands/document_consumer.py

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -26,6 +26,7 @@ class Command(BaseCommand):
2626
def __init__(self, *args, **kwargs):
2727

2828
self.verbosity = 0
29+
self.logger = logging.getLogger(__name__)
2930

3031
self.file_consumer = None
3132
self.mail_fetcher = None
@@ -138,6 +139,11 @@ def loop_inotify(self, mail_delta):
138139
file = os.path.join(directory, event.name)
139140
if os.path.isfile(file):
140141
self.file_consumer.try_consume_file(file)
142+
else:
143+
self.logger.warning(
144+
"Skipping %s as it is not a file",
145+
file
146+
)
141147
else:
142148
break
143149

0 commit comments

Comments
 (0)