Changeset 9:69d5a3b74c6a in mailjam for postman/daemon.py

Timestamp:

May 19, 2012, 10:14:00 AM (12 years ago)

Author:

Francisco de Borja Lopez Rio <borja@…>

Branch:

default

Phase:

public

Message:

Added a new method to the config module that parses the configuration file and returns configuration parameters. Modified the daemon.Postman and daemon.PostmanDaemon so they load their configs from the config file

File:

• postman/daemon.py (modified) (3 diffs)

postman/daemon.py

@@ -12 +12 @@
 
     def __init__(self, configfile=None):
-        if not configfile:
-            # FIXME: This is not used right now, we are getting the stuff
-            # from postman.config, but we will move configurations to a
-            # external ini-style config file soon
-            configfile = os.path.join(os.path.dirname(__file__),
-                                      '../conf/postman.conf')
-        self.configfile = configfile
+        self.storage_config = config.get_config_parameters('storage',
+                                                           configfile)
+        self.archive_config = config.get_config_parameters('archive',
+                                                           configfile)
 
         # lists were the currently managed mailing lists information is going
@@ -27 +24 @@
         # the files were internal information (like active mailing lists,
         # members, etc) is saved
-        self.dbs = {'mailings': Storage(os.path.join(config.storage_path,
-                                                     'mailings.json')),
-                    'members': Storage(os.path.join(config.storage_path,
-                                                    'members.json'))}
+        self.dbs = {'mailings': Storage(self.storage_config['lists_db']),
+                    'members': Storage(self.storage_config['members_db'])}
 
     def save(self):
@@ -178 +173 @@
     def add(self, member_addr=None, list_addr=None):
         self.postman.add_mailing_member(member_addr, list_addr)
-                  
-
+    
+
+    
 class PostmanDaemon():
     def __init__(self, configfile=None):
-        if not configfile:
-            # FIXME: This is not used right now, we are getting the stuff
-            # from postman.config, but we will move configurations to a
-            # external ini-style config file soon
-            configfile = os.path.join(os.path.dirname(__file__),
-                                      '../conf/postman.conf')
-        self.configfile = configfile
+        self.config = config.get_config_parameters('xmlrpc_server', configfile)
 
         # FIXME: These should be loaded from a config file
-        self.address='localhost'
-        self.port = 9000
-        
-        self.logfile = os.path.join(os.path.dirname(__file__), 'server.log')
+        self.address = self.config.get('address', 'localhost')
+        self.port = int(self.config.get('port', 9876))
+        self.logfile = self.config.get('logfile',
+                                       os.path.join(os.path.dirname(__file__),
+                                                    'server.log'))
         logging.basicConfig(filename=self.logfile, level=logging.DEBUG)