iamamoose commented on code in PR #246:
URL: 
https://github.com/apache/incubator-ponymail-foal/pull/246#discussion_r1150439946


##########
tools/archiver.py:
##########
@@ -211,6 +211,11 @@ def __init__(self, part: email.message.Message):
                         break
                     except UnicodeDecodeError:
                         pass
+                    except:

Review Comment:
   I fed around 300k emails using import-mbox and had many hundreds of 
content-type header failures.  If there's a way for someone to make an invalid 
content-type header, I had a mail for it!  So after making this change I didn't 
capture all the other broken headers and figured the option should be "allow 
overiding any bad decoding"



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to