X-Git-Url: https://git.ucc.asn.au/?p=uccvend-vendserver.git;a=blobdiff_plain;f=sql-edition%2Fservers%2FVendServer.py;h=85f42f4546ae50e84095b8ef73055d0ab8450106;hp=4a601b759a967a06edfbd01cc05ba7c63b351a46;hb=d09660203257449733853893a4b154bf6be6808a;hpb=c25d6ce856cda7b8ea20355472c3821f66c7b78f

diff --git a/sql-edition/servers/VendServer.py b/sql-edition/servers/VendServer.py
index 4a601b7..85f42f4 100755
--- a/sql-edition/servers/VendServer.py
+++ b/sql-edition/servers/VendServer.py
@@ -3,11 +3,12 @@
 
 USE_DB = 0
 
+import ConfigParser
 import sys, os, string, re, pwd
 if USE_DB: import pg
 from time import time, sleep
 from popen2 import popen2
-from LATClient import LATClient
+from LATClient import LATClient, LATClientException
 from VendingMachine import VendingMachine, VendingException
 from HorizScroll import HorizScroll
 from random import random, seed
@@ -197,11 +198,11 @@ class MessageKeeper:
 	def done(self):
 		return len(self.scrolling_message) == 0
 
-def run_forever(rfh, wfh):
+def run_forever(rfh, wfh, options, cf):
 	v = VendingMachine(rfh, wfh)
 	print 'PING is', v.ping()
 
-	if USE_DB: db = DispenseDatabase(v, DBServer, DBName, DBUser, DBPassword)
+	if USE_DB: db = DispenseDatabase(v, cf.DBServer, cf.DBName, cf.DBUser, cf.DBPassword)
 	cur_user = ''
 	cur_pin = ''
 	cur_selection = ''
@@ -363,11 +364,10 @@ def run_forever(rfh, wfh):
 					cur_selection = ''
 					time_to_autologout = time() + 8
 
-# FIXME: a less ugly way of passing all these options would be nice
-def connect_to_vend(options, DBServer, DBName, DBUser, DBPassword, ServiceName, ServicePassword, ServerName, ConnectPassword, PrivPassword):
-	# Open vending machine via LAT
+def connect_to_vend(options, cf):
+	# Open vending machine via LAT?
 	if options.use_lat:
-		latclient = LATClient(service = ServiceName, password = ServicePassword, server_name = ServerName, connect_password = ConnectPassword, priv_password = PrivPassword)
+		latclient = LATClient(service = cf.ServiceName, password = cf.ServicePassword, server_name = cf.ServerName, connect_password = cf.ConnectPassword, priv_password = cf.PrivPassword)
 		rfh, wfh = latclient.get_fh()
 	else:
 		#(rfh, wfh) = popen2('../../virtualvend/vvend.py')
@@ -379,8 +379,7 @@ def connect_to_vend(options, DBServer, DBName, DBUser, DBPassword, ServiceName,
 		
 	return rfh, wfh
 
-if __name__ == '__main__':
-	from ConfigParser import ConfigParser
+def parse_args():
 	from optparse import OptionParser
 
 	op = OptionParser(usage="%prog [OPTION]...")
@@ -393,33 +392,56 @@ if __name__ == '__main__':
 	if len(args) != 0:
 		op.error('extra command line arguments: ' + ' '.join(args))
 
-	cp = ConfigParser()
-	cp.read(options.config_file)
-	DBServer = cp.get('Database', 'Server')
-	DBName = cp.get('Database', 'Name')
-	DBUser = cp.get('VendingMachine', 'DBUser')
-	DBPassword = cp.get('VendingMachine', 'DBPassword')
+	return options
+
+config_options = {
+	'DBServer': ('Database', 'Server'),
+	'DBName': ('Database', 'Name'),
+	'DBUser': ('VendingMachine', 'DBUser'),
+	'DBPassword': ('VendingMachine', 'DBPassword'),
+	
+	'ServiceName': ('VendingMachine', 'ServiceName'),
+	'ServicePassword': ('VendingMachine', 'Password'),
+	
+	'ServerName': ('DecServer', 'Name'),
+	'ConnectPassword': ('DecServer', 'ConnectPassword'),
+	'PrivPassword': ('DecServer', 'PrivPassword'),
+	}
+
+class VendConfigFile:
+	def __init__(self, config_file, options):
+		try:
+			cp = ConfigParser.ConfigParser()
+			cp.read(config_file)
+
+			for option in options:
+				section, name = options[option]
+				value = cp.get(section, name)
+				self.__dict__[option] = value
+		
+		except ConfigParser.Error, e:
+			print "Error reading config file "+config_file+": " + str(e)
+			sys.exit(1)
+
 
-	ServiceName = cp.get('VendingMachine', 'ServiceName')
-	ServicePassword = cp.get('VendingMachine', 'Password')
+if __name__ == '__main__':
 
-	ServerName = cp.get('DecServer', 'Name')
-	ConnectPassword = cp.get('DecServer', 'ConnectPassword')
-	PrivPassword = cp.get('DecServer', 'PrivPassword')
+	options = parse_args()
+	config_opts = VendConfigFile(options.config_file, config_options)
 
 	while True:
 		try:
-			rfh, wfh = connect_to_vend(options, DBServer, DBName, DBUser, DBPassword, ServiceName, ServicePassword, ServerName, ConnectPassword, PrivPassword)
-		except Exception:
+			rfh, wfh = connect_to_vend(options, config_opts)
+		except (LATClientException, socket.error), e:
 			(exc_type, exc_value, exc_traceback) = sys.exc_info()
-			print "Connection error ("+str(exc_type)+"):"
-			print_tb(exc_traceback)
-			del exc_traceback
+			print
+			print "Connection error: "+str(exc_type)+" "+str(e)
 			print "Trying again in 5 seconds."
 			sleep(5)
 			continue
 		try:
-			run_forever(rfh, wfh)
+			run_forever(rfh, wfh, options, config_opts)
 		except VendingException:
+			print
 			print "Connection died, trying again..."