This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author andrei.avk
Recipients andrei.avk, iritkatriel, jbw, kj, moi90, serhiy.storchaka
Date 2021-11-09.01:55:02
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1636422902.66.0.618024876663.issue43656@roundup.psfhosted.org>
In-reply-to
Content
Martin:

I have a couple of concerns:

 - Generally (AFAIK) Python is very conservative about silencing arbitrary exceptions. There are a few functions with args like `ignore_errors`, but those are for errors in the logic of respective functions. I don't recall examples where any error would be silenced via an argument, but if there are such cases, it would be interesting to look into how the design decision was made.

In this case of course arbitrary exceptions coming any objects' __repr__ may be silenced.

There is a clear and very explicit way to catch exceptions via try/except and as a dev, I would really want to be able to look at a module, look at all try/except clauses and be confident that exceptions are not silenced elsewhere.

 - You are targeting this fix to production use, but realistically, if merged, it will be used both in testing and production. Which means, by not seeing these exceptions in testing, you will have a higher chance of deploying them to production where they can surface in other circumstances.

IOW, as a dev I might prefer to see these errors early and often, rather than have a mechanism that ends up silencing errors more broadly than intended.

I'm not saying this fix should be rejected, but that there's a tricky balance here -- and I don't feel confident enough about this solution to approve the PR if I reviewed it.
History
Date User Action Args
2021-11-09 01:55:02andrei.avksetrecipients: + andrei.avk, serhiy.storchaka, iritkatriel, moi90, kj, jbw
2021-11-09 01:55:02andrei.avksetmessageid: <1636422902.66.0.618024876663.issue43656@roundup.psfhosted.org>
2021-11-09 01:55:02andrei.avklinkissue43656 messages
2021-11-09 01:55:02andrei.avkcreate