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 fd6d4b27530366eea7ebf25e5b207de5ef9bd5b3..dbdaf79e5990268c3250b6714102164a1a4cadcc 100644 (file)
--- 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)