From af4df47fe730e176785dc4ece6f256c965bc2ec1 Mon Sep 17 00:00:00 2001 From: Brian Warner Date: Fri, 30 Jun 2017 12:46:02 -0700 Subject: [PATCH] rendezvous: fix misleading log message We only log the internal (sqlite) ID of the nameplate, not the actual small-integer name. While investigating misbehavior due to overload, I was confused into thinking that users were getting nameplates in the 15000+ range, when in fact those were merely the internal database row ids. --- src/wormhole/server/rendezvous.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/wormhole/server/rendezvous.py b/src/wormhole/server/rendezvous.py index 50486aa..818c70c 100644 --- a/src/wormhole/server/rendezvous.py +++ b/src/wormhole/server/rendezvous.py @@ -468,10 +468,10 @@ class AppNamespace(object): mailbox_id = row["mailbox_id"] if mailbox_id in old_mailboxes: old_nameplates.add(npid) - log.msg(" 3: old_nameplates", old_nameplates) + log.msg(" 3: old_nameplates dbids", old_nameplates) for npid in old_nameplates: - log.msg(" deleting nameplate", npid) + log.msg(" deleting nameplate with dbid", npid) side_rows = db.execute("SELECT * FROM `nameplate_sides`" " WHERE `nameplates_id`=?", (npid,)).fetchall()