[Checkins] SVN: zc.monitor/trunk/src/zc/monitor/ Bugs Fixed: The z3monitor server lacked a handle_close method, which caused
Jim Fulton
jim at zope.com
Mon Sep 15 18:27:28 EDT 2008
Log message for revision 91183:
Bugs Fixed: The z3monitor server lacked a handle_close method, which caused
errors to get logged when users closed connections before giving
commands.
Changed:
U zc.monitor/trunk/src/zc/monitor/CHANGES.txt
U zc.monitor/trunk/src/zc/monitor/README.txt
U zc.monitor/trunk/src/zc/monitor/__init__.py
-=-
Modified: zc.monitor/trunk/src/zc/monitor/CHANGES.txt
===================================================================
--- zc.monitor/trunk/src/zc/monitor/CHANGES.txt 2008-09-15 22:24:22 UTC (rev 91182)
+++ zc.monitor/trunk/src/zc/monitor/CHANGES.txt 2008-09-15 22:27:28 UTC (rev 91183)
@@ -2,6 +2,13 @@
Change History
==============
+0.1.2 (2008-09-15)
+------------------
+
+- Bugfix: The z3monitor server lacked a handle_close method, which
+ caused errors to get logged when users closed connections before
+ giving commands.
+
0.1.1 (2008-09-14)
------------------
Modified: zc.monitor/trunk/src/zc/monitor/README.txt
===================================================================
--- zc.monitor/trunk/src/zc/monitor/README.txt 2008-09-15 22:24:22 UTC (rev 91182)
+++ zc.monitor/trunk/src/zc/monitor/README.txt 2008-09-15 22:27:28 UTC (rev 91183)
@@ -142,3 +142,9 @@
TypeError: hello() takes at most 2 arguments (3 given)
<BLANKLINE>
-> CLOSE
+
+.. Edge cases
+
+ Closing the connection:
+
+ >>> connection.test_close('test')
Modified: zc.monitor/trunk/src/zc/monitor/__init__.py
===================================================================
--- zc.monitor/trunk/src/zc/monitor/__init__.py 2008-09-15 22:24:22 UTC (rev 91182)
+++ zc.monitor/trunk/src/zc/monitor/__init__.py 2008-09-15 22:27:28 UTC (rev 91183)
@@ -65,7 +65,10 @@
if self.mode is QUIT_MARKER:
connection.write(zc.ngi.END_OF_DATA)
+ def handle_close(self, connection, reason):
+ pass # Don't care
+
def start(port):
"""start monitor server.
More information about the Checkins
mailing list