From 06e42791c49da9b6b8baf74d54bf977b28e1b9ae Mon Sep 17 00:00:00 2001 From: Arne Allisat Date: Fri, 6 Nov 2020 09:21:17 +0100 Subject: [PATCH 1/2] Limit filename length to 255 characters https://github.com/domainaware/parsedmarc/issues/197 Since there is a limit for filename length in some OS, filename length should be limited. I propose limit it to 255 characters. --- parsedmarc/utils.py | 2 ++ 1 file changed, 2 insertions(+) diff --git a/parsedmarc/utils.py b/parsedmarc/utils.py index de42e7c..5808716 100644 --- a/parsedmarc/utils.py +++ b/parsedmarc/utils.py @@ -395,6 +395,8 @@ def get_filename_safe_string(string): string = string.replace(char, "") string = string.rstrip(".") + string = (string[:255]) if len(string) > 255 else string + return string From 87092cf4caa0d765fa997ea300280807a8959bb8 Mon Sep 17 00:00:00 2001 From: Arne Allisat Date: Fri, 6 Nov 2020 14:16:09 +0100 Subject: [PATCH 2/2] Reduce number of characters in filename from 255 to 100 Reduce number of characters in filename from 255 to 100 --- parsedmarc/utils.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/parsedmarc/utils.py b/parsedmarc/utils.py index 5808716..5be154d 100644 --- a/parsedmarc/utils.py +++ b/parsedmarc/utils.py @@ -395,7 +395,7 @@ def get_filename_safe_string(string): string = string.replace(char, "") string = string.rstrip(".") - string = (string[:255]) if len(string) > 255 else string + string = (string[:100]) if len(string) > 100 else string return string