gajim can now connect to custom hostname and port [TODO: patch xmpppy to do ssl in any port I ask]

src/common/config.py

@@ -119,6 +119,7 @@ class Config:
 		'accounts': ({
 			'name': [ opt_str, '' ],
 			'hostname': [ opt_str, '' ],
+			'port': [ opt_int, 5222 ],
 			'savepass': [ opt_bool, False ],
 			'password': [ opt_str, '' ],
 			'resource': [ opt_str, 'gajim' ],

src/common/connection.py

@@ -594,6 +594,7 @@ class Connection:
 		"""Connect and authenticate to the Jabber server"""
 		name = gajim.config.get_per('accounts', self.name, 'name')
 		hostname = gajim.config.get_per('accounts', self.name, 'hostname')
+		port = gajim.config.get_per('accounts', self.name, 'port')
 		resource = gajim.config.get_per('accounts', self.name, 'resource')
 		usessl = gajim.config.get_per('accounts', self.name, 'usessl')
 
@@ -613,14 +614,11 @@ class Connection:
 			con = common.xmpp.Client(hostname)
 		else:
 			con = common.xmpp.Client(hostname, debug = [])
-			#debug = [common.jabber.DBG_ALWAYS], log = sys.stderr, \
+		common.xmpp.dispatcher.DefaultTimeout = 45 # wait 45 seconds until you timeout connection
-			#connection=common.xmlstream.TCP_SSL, port=5223, proxy = proxy)
-		common.xmpp.dispatcher.DefaultTimeout = 45
 		con.UnregisterDisconnectHandler(con.DisconnectHandler)
 		con.RegisterDisconnectHandler(self._disconnectedCB)
-		port = 5222
+
-		if usessl:
+		#pass ssl optional arg if neccessary when client.py is patched
-			port = 5223
 		con_type = con.connect((hostname, port), proxy = proxy) #FIXME: blocking
 		if not con_type:
 			gajim.log.debug("Couldn't connect to %s" % self.name)
@@ -896,7 +894,8 @@ class Connection:
 		common.xmpp.dispatcher.DefaultTimeout = 45
 		c.UnregisterDisconnectHandler(c.DisconnectHandler)
 		c.RegisterDisconnectHandler(self._disconnectedCB)
-		port = 5222
+		port = gajim.config.get_per('accounts', self.name, 'port')
+		#FIXME: use ssl
 		#if usessl:
 			#port = 5223
 		#FIXME: blocking

src/config.py

@@ -1056,6 +1056,9 @@ class AccountModificationWindow:
 		usessl = gajim.config.get_per('accounts', self.account, 'usessl')
 		self.xml.get_widget('use_ssl_checkbutton').set_active(usessl)
 
+		port = gajim.config.get_per('accounts', self.account, 'port')
+		self.xml.get_widget('custom_port_entry').set_text(str(port))
+
 		gpg_key_label = self.xml.get_widget('gpg_key_label')
 		if gajim.config.get('usegpg'):
 			self.init_account_gpg()
@@ -1130,7 +1133,19 @@ _('To change the account name, it must be disconnected.')).get_response()
 		config['proxy'] = proxy
 		
 		config['usessl'] = self.xml.get_widget('use_ssl_checkbutton').get_active()
+
+		if self.xml.get_widget('custom_host_port_checkbutton').get_active():
+			config['name'] = jid.split('@')[0]
+			config['hostname'] = self.xml.get_widget('custom_host_entry').get_text()
+			config['port'] = int(self.xml.get_widget('custom_port_entry').get_text())
+		else:
 			(config['name'], config['hostname']) = jid.split('@')
+			if config['usessl']:
+				port = 5223 #FIXME: better way
+			else:
+				port = 5222
+			config['port'] = port
+
 		config['keyname'] = self.xml.get_widget('gpg_name_label').get_text()
 		if config['keyname'] == '': #no key selected
 			config['keyid'] = ''
@@ -1303,6 +1318,13 @@ _('There was a problem retrieving your GPG secret keys.')).get_response()
 			if not widget.get_active():
 				w.set_text('')
 
+	def on_use_ssl_checkbutton_toggled(self, widget):
+		isactive = widget.get_active()
+		if isactive:
+			self.xml.get_widget('custom_port_entry').set_text('5223')
+		else:
+			self.xml.get_widget('custom_port_entry').set_text('5222')
+
 	def on_custom_host_port_checkbutton_toggled(self, widget):
 		isactive = widget.get_active()
 		self.xml.get_widget('custom_host_port_hbox').set_sensitive(isactive)

src/gtkgui.glade

@@ -1322,6 +1322,7 @@
 			      <property name="active">False</property>
 			      <property name="inconsistent">False</property>
 			      <property name="draw_indicator">True</property>
+				  <signal name="toggled" handler="on_use_ssl_checkbutton_toggled" last_modification_time="Mon, 14 Jun 2005 10:44:56 GMT"/>
 			    </widget>
 			    <packing>
 			      <property name="padding">0</property>