From: robk-tahoe <robk-tahoe@allmydata.com>
Date: Fri, 1 Feb 2008 02:11:31 +0000 (-0700)
Subject: stats_gatherer: reject "<unauth>" as a tubid, to avoid screwing up the data.
X-Git-Url: https://git.rkrishnan.org/pf/content/en/seg/rgr-080307.php?a=commitdiff_plain;h=0700ccabaa6bbeb01ba147759bff93b0123646ba;p=tahoe-lafs%2Ftahoe-lafs.git

stats_gatherer: reject "<unauth>" as a tubid, to avoid screwing up the data.
---

diff --git a/src/allmydata/stats.py b/src/allmydata/stats.py
index fd6d4b27..dbdaf79e 100644
--- a/src/allmydata/stats.py
+++ b/src/allmydata/stats.py
@@ -120,6 +120,10 @@ class StatsGatherer(foolscap.Referenceable, service.MultiService):
 
     def remote_provide(self, provider, nickname):
         tubid = self.get_tubid(provider)
+        if tubid == '<unauth>':
+            print "WARNING: failed to get tubid for %s (%s)" % (provider, nickname)
+            # don't add to clients to poll (polluting data) don't care about disconnect
+            return
         self.clients[tubid] = provider
         self.nicknames[tubid] = nickname
         provider.notifyOnDisconnect(self.lost_client, tubid)