“stomp.py” is a Python client library for accessing messaging servers (such as ActiveMQ or RabbitMQ) using the STOMP protocol (versions 1.0, 1.1 and 1.2). It can also be run as a standalone, command-line client for testing.
Apache License 2.0
491
stars
167
forks
source link
Listener/transport modules too verbose when using heartbeat #400
Hello,
after migration my application from stomp.py version 4.1.21 to 7.0.0 and adopting hearbeat signs, I noticed that the listener and transport modules are extremely verbose, with messages like [1].
I would like to suggest to change the log level for those log records from INFO to DEBUG. I am about to create a PR with those changes in, in case you agree with this modification. Thanks
[1]
2022-09-09 20:47:08,879:139964385847040:INFO:listener:Starting heartbeat loop
2022-09-09 20:47:08,880:139965237520128:INFO:transport:attempt reconnection (True, None, 0)
2022-09-09 20:47:08,880:139965237520128:INFO:transport:Attempting connection to host xxx, port xxxx
2022-09-09 20:47:08,880:139965237520128:INFO:transport:Established connection to host xxx, port xxxx
2022-09-09 20:47:08,880:139964377454336:INFO:transport:Starting receiver loop (<Thread(Thread-14, started daemon 139964377454336)>)
2022-09-09 20:47:08,880:139965237520128:INFO:transport:Created thread <Thread(Thread-14, started daemon 139964377454336)> using func <function default_create_thread at 0x7f4c3faa1d30>
2022-09-09 20:47:08,881:139965237520128:INFO:transport:Sending frame: 'STOMP'
2022-09-09 20:47:08,882:139964377454336:INFO:transport:Received frame: 'CONNECTED', len(body)=0
2022-09-09 20:47:08,882:139964369061632:INFO:listener:Starting heartbeat loop
2022-09-09 20:47:08,882:139965237520128:INFO:transport:Sending frame: 'SEND'
Hello, after migration my application from stomp.py version 4.1.21 to 7.0.0 and adopting hearbeat signs, I noticed that the listener and transport modules are extremely verbose, with messages like [1]. I would like to suggest to change the log level for those log records from INFO to DEBUG. I am about to create a PR with those changes in, in case you agree with this modification. Thanks
[1]