node.py: tolerate a UTF-8 BOM at the start of tahoe.cfg. fixes #1470

diff --git a/src/allmydata/node.py b/src/allmydata/node.py
index 06b707de1c50d86804a208a97e8657ea87cb3572..a77fdadca0489439c18b1fc4bd3de4b45be09735 100644 (file)
--- a/src/allmydata/node.py
+++ b/src/allmydata/node.py
@@ -122,7 +122,21 @@ class Node(service.MultiService):
     def read_config(self):
         self.error_about_old_config_files()
         self.config = ConfigParser.SafeConfigParser()
-        self.config.read([os.path.join(self.basedir, "tahoe.cfg")])
+
+        tahoe_cfg = os.path.join(self.basedir, "tahoe.cfg")
+        try:
+            f = open(tahoe_cfg, "rb")
+            try:
+                # Skip any initial Byte Order Mark. Since this is an ordinary file, we
+                # don't need to handle incomplete reads, and can assume seekability.
+                if f.read(3) != '\xEF\xBB\xBF':
+                    f.seek(0)
+                self.config.readfp(f)
+            finally:
+                f.close()
+        except EnvironmentError:
+            if os.path.exists(tahoe_cfg):
+                raise
 
         cfg_tubport = self.get_config("node", "tub.port", "")
         if not cfg_tubport: