improve (fix) tnefparse logging

koodaamo / tnefparse

a TNEF decoding library written in python, without external dependencies

GNU Lesser General Public License v3.0

49 stars 37 forks source link

improve (fix) tnefparse logging #18

Closed petri closed 5 years ago

petri commented 6 years ago

Currently, logging in tnefparse is a bit of a mess and should be improved.

This issue is to gather input for that. If you don't like how tnefparse logs warnings etc. please submit a proposal under this issue.

jrideout commented 5 years ago

In general the logging isn't so bad. The problem are the things like WARNING Unknown TNEF Object: <TNEFObject 'Message ID'> that clutter things up and I think causes confusion (since this doesn't mean there is something wrong).

What we probably need to do is set all attributes we observe dynamically, not just the mapi attributes.

petri commented 5 years ago

I removed some of those more or less unnecessary warnings. Let's close this, we can reopen if needed.