python gotcha: mutable type in __init__()

In Server.py:53

http://www.ferg.org/projects/python_gotchas.html

init arguments are parsed only once, so the same list will be shared with all instance objects.

added component::circumvention/bridgedb owner::aagbsn priority::medium resolution::fixed status::closed type::defect labels

Simple fix:

diff --git a/lib/bridgedb/Server.py b/lib/bridgedb/Server.py
index a1b837a..b67edf2 100644
--- a/lib/bridgedb/Server.py
+++ b/lib/bridgedb/Server.py
@@ -50,7 +50,7 @@ class WebResource(twisted.web.resource.Resource):
     def __init__(self, distributor, schedule, N=1, useForwardedHeader=False,
                  includeFingerprints=True,
                  useRecaptcha=False,recaptchaPrivKey='', recaptchaPubKey='',
-                 domains=[]): 
+                 domains=None): 
         """Create a new WebResource.
              distributor -- an IPBasedDistributor object
              schedule -- an IntervalSchedule object
@@ -63,6 +63,9 @@ class WebResource(twisted.web.resource.Resource):
         self.nBridgesToGive = N
         self.useForwardedHeader = useForwardedHeader
         self.includeFingerprints = includeFingerprints
+
+        # do not use mutable types as __init__ defaults!
+        if not domains: domains = []
         self.domains = domains
 
         # recaptcha options

https://gitweb.torproject.org/user/aagbsn/bridgedb.git/commit/aba290f8e3a1eaa38bfd580d350e0721911b49da

Trac:
Status: new to needs_review

Trac:
Status: needs_review to closed
Resolution: N/A to fixed

closed

moved to tpo/anti-censorship/bridgedb#5367 (closed)