Fixes stderr appearing to have content when it doesn't

docs/advanced_usage.rst

@@ -149,6 +149,9 @@ which will in turn call `pdf2pdfocr.py`_ on your document, which will then
 overwrite the file with an OCR'd version of the file and exit.  At which point,
 the consumption process will begin with the newly modified file.
 
+The script's stdout and stderr will be logged line by line to the webserver log, along
+with the exit code of the script.
+
 .. _pdf2pdfocr.py: https://github.com/LeoFCardoso/pdf2pdfocr
 
 .. _advanced-post_consume_script:
@@ -178,6 +181,10 @@ example, you can take a look at `post-consumption-example.sh`_ in this project.
 
 The post consumption script cannot cancel the consumption process.
 
+The script's stdout and stderr will be logged line by line to the webserver log, along
+with the exit code of the script.
+
+
 Docker
 ------
 Assumed you have ``/home/foo/paperless-ngx/scripts/post-consumption-example.sh``.

src/documents/consumer.py

@@ -537,11 +537,19 @@ class Consumer(LoggingMixin):
         )
 
         # Decode the output (if any)
-        stdout_str = completed_process.stdout.decode("utf8", errors="ignore").split(
+        stdout_str = (
-            "\n",
+            completed_process.stdout.decode("utf8", errors="ignore")
+            .strip()
+            .split(
+                "\n",
+            )
         )
-        stderr_str = completed_process.stderr.decode("utf8", errors="ignore").split(
+        stderr_str = (
-            "\n",
+            completed_process.stderr.decode("utf8", errors="ignore")
+            .strip()
+            .split(
+                "\n",
+            )
         )
 
         if len(stdout_str):