example client ends with exception

Bug #407654 reported by ceelian
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
txAMQP
Fix Released
Undecided
Unassigned

Bug Description

The following exception happens when i try to start the example client:

christian@charly:~/tmp/txamqp/src/examples$ python client.py myamqpserver.com 5672 / test test ../specs/standard/amqp0-8.xml
Got results to add()
3
Ping received
Got results to calculate()
-1
Got results to calculate()
0
Got results to calculate()
5
Got results to calculate()
6
Got an error
integer division or modulo by zero
Unhandled error in Deferred:
Traceback (most recent call last):
  File "/usr/local/lib/python2.6/dist-packages/txAMQP-0.2-py2.6.egg/txamqp/queue.py", line 47, in close
    self.put(TimeoutDeferredQueue.END)
  File "/usr/lib/python2.6/dist-packages/twisted/internet/defer.py", line 992, in put
    self.waiting.pop(0).callback(obj)
  File "/usr/lib/python2.6/dist-packages/twisted/internet/defer.py", line 243, in callback
    self._startRunCallbacks(result)
  File "/usr/lib/python2.6/dist-packages/twisted/internet/defer.py", line 312, in _startRunCallbacks
    self._runCallbacks()
--- <exception caught here> ---
  File "/usr/lib/python2.6/dist-packages/twisted/internet/defer.py", line 328, in _runCallbacks
    self.result = callback(self.result, *args, **kw)
  File "/usr/local/lib/python2.6/dist-packages/txAMQP-0.2-py2.6.egg/txamqp/queue.py", line 32, in _raiseIfClosed
    raise Closed()
txamqp.queue.Closed:

Revision history for this message
Jonathan Stoppani (jonathan-stoppani) wrote :

As posted in https://answers.launchpad.net/txamqp/+question/79275:

"Looking at the source code it seems that someone is listening on incoming messages using a TimeoutDeferredQueue but is not catching Closed exceptions."

Revision history for this message
Esteve Fernandez (esteve) wrote :

I just linked this bug to that question. I introduced that error while trying to fix #366674, I'll try to fix both today.

Thanks!

Changed in txamqp:
status: New → Confirmed
Revision history for this message
Jonathan Stoppani (jonathan-stoppani) wrote :

I just proposed a merge which contains a fix to both.

Revision history for this message
Esteve Fernandez (esteve) wrote :

I just merged your fix into the trunk. Thanks Jonathan!

Changed in txamqp:
status: Confirmed → Fix Committed
Changed in txamqp:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.