[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