Jami Lindh added the comment:

In my opinion, this is still a clear case of "invalid input, raise an error", 
but I don't think AssertionError is the right one. Maybe just don't catch the 
unexpected binascii.Error and let it fly towards the user?

I might go even one step further and just let base64 library handle and raise 
all the necessary errors here, similarly to how your "encode_b" function works.

If you want more error resilience (meaning: email lib would not crash with 
invalid inputs), I would think returning an empty string is okay, since that 
base64 input is clearly not a valid base64 that is going to decode into 
anything.

----------

_______________________________________
Python tracker <rep...@bugs.python.org>
<http://bugs.python.org/issue27397>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

Reply via email to