From: Trenton Holmes <797416+stumpylog@users.noreply.github.com>
Date: Mon, 26 Dec 2022 21:43:30 +0000 (-0800)
Subject: Fixes minor depracation I noticed
X-Git-Tag: v1.12.0-beta.rc0~55
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=efaa1c4dd77fd327a55a9c0d20a2c08125e0f56b;p=thirdparty%2Fpaperless-ngx.git

Fixes minor depracation I noticed
---

diff --git a/src/documents/signals/handlers.py b/src/documents/signals/handlers.py
index 4c2554f02e..cd42c90303 100644
--- a/src/documents/signals/handlers.py
+++ b/src/documents/signals/handlers.py
@@ -447,7 +447,7 @@ def update_filename_and_move_files(sender, instance, **kwargs):
             )
 
         except (OSError, DatabaseError, CannotMoveFilesException) as e:
-            logger.warn(f"Exception during file handling: {e}")
+            logger.warning(f"Exception during file handling: {e}")
             # This happens when either:
             #  - moving the files failed due to file system errors
             #  - saving to the database failed due to database errors
diff --git a/src/documents/views.py b/src/documents/views.py
index 147dd2e078..abd2e5f2fa 100644
--- a/src/documents/views.py
+++ b/src/documents/views.py
@@ -12,6 +12,7 @@ from time import mktime
 from unicodedata import normalize
 from urllib.parse import quote
 
+import pathvalidate
 from django.conf import settings
 from django.contrib.auth.models import User
 from django.db.models import Case
@@ -625,7 +626,7 @@ class PostDocumentView(GenericAPIView):
         os.makedirs(settings.SCRATCH_DIR, exist_ok=True)
 
         temp_file_path = Path(tempfile.mkdtemp(dir=settings.SCRATCH_DIR)) / Path(
-            doc_name,
+            pathvalidate.sanitize_filename(doc_name),
         )
 
         temp_file_path.write_bytes(doc_data)
@@ -634,8 +635,9 @@ class PostDocumentView(GenericAPIView):
 
         task_id = str(uuid.uuid4())
 
-        async_task = consume_file.delay(
-            temp_file_path,
+        consume_file.delay(
+            # Paths are not JSON friendly
+            str(temp_file_path),
             override_title=title,
             override_correspondent_id=correspondent_id,
             override_document_type_id=document_type_id,