txAMQP

Merge lp://staging/~txamqpteam/txamqp/340115-fix-frame-length into lp://staging/txamqp

  1. 340115-fix-frame-length
  2. Merge into trunk
Proposed by Esteve Fernandez
Status: Merged
Merged at revision: not available
Proposed branch: lp://staging/~txamqpteam/txamqp/340115-fix-frame-length
Merge into: lp://staging/txamqp
Diff against target: None lines
To merge this branch: bzr merge lp://staging/~txamqpteam/txamqp/340115-fix-frame-length
Reviewer Review Type Date Requested Status
Dan Di Spaltro frame_max Approve
Review via email: mp+5428@code.staging.launchpad.net
To post a comment you must log in.
Revision history for this message
Dan Di Spaltro (dan-dispaltro) :
review: Approve (frame_max)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/txamqp/client.py'
2--- src/txamqp/client.py 2008-10-29 18:31:04 +0000
3+++ src/txamqp/client.py 2009-04-10 09:37:17 +0000
4@@ -25,6 +25,7 @@
5 locale=self.client.locale)
6
7 def connection_tune(self, ch, msg):
8+ self.client.MAX_LENGTH = msg.frame_max
9 ch.connection_tune_ok(*msg.fields)
10 self.client.started.set()
11
12
13=== modified file 'src/txamqp/protocol.py'
14--- src/txamqp/protocol.py 2009-02-05 19:22:56 +0000
15+++ src/txamqp/protocol.py 2009-04-10 09:37:17 +0000
16@@ -99,7 +99,7 @@
17 class FrameReceiver(protocol.Protocol, basic._PauseableMixin):
18
19 frame_mode = False
20- MAX_LENGTH = 16384
21+ MAX_LENGTH = 4096
22 HEADER_LENGTH = 1 + 2 + 4 + 1
23 __buffer = ''
24

Subscribers

People subscribed via source and target branches

to status/vote changes: