taskotron / resultsdb

#139 Log the reason ConnectionException
Merged by jskladan. Opened by pingou.
taskotron/ pingou/resultsdb better_debug  into  develop

Download 139.patch

ConnectionException seems to not print anything when printed as is
but it has a reason attribute which does provide more information
so print it directly.

Signed-off-by: Pierre-Yves Chibon pingou@pingoured.fr

+1

1 new commit added

  • Log when a message was successfully published

Added a second commit so we also have a clear log entry for when messages are successfully published :)

Someone who has permission to merge should review this.

It's not clear from ConnectionException documentation that it even has reason attribute (unless you look at the source code).

Also the documentation mentions logging the exception basically with str(e) which looks like a bug in fedora-messaging.

I'm trying to fix this in fedora-messaging.

Created PR for fedora-messaging: https://github.com/fedora-infra/fedora-messaging/pull/198

Pull-Request has been merged by jskladan
Metadata
None
No Tags
Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.
Red Hat Logo

Fedora is sponsored by Red Hat. Learn more about the relationship between Red Hat and Fedora »

© Red Hat, Inc. and others.