From 51f174b97f686397bc7299c16a64fda14d5bc718 Mon Sep 17 00:00:00 2001
From: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
Date: Tue, 3 Jul 2012 18:52:48 +0300
Subject: [PATCH] fix port issue

---
 app/mailers/notify.rb             |  2 +-
 config/initializers/1_settings.rb | 14 +++++++++++---
 2 files changed, 12 insertions(+), 4 deletions(-)

diff --git a/app/mailers/notify.rb b/app/mailers/notify.rb
index ee738b4c..c673eb3d 100644
--- a/app/mailers/notify.rb
+++ b/app/mailers/notify.rb
@@ -4,7 +4,7 @@ class Notify < ActionMailer::Base
 
   default_url_options[:host]     = Gitlab.config.web_host
   default_url_options[:protocol] = Gitlab.config.web_protocol
-  default_url_options[:port]     = Gitlab.config.web_port
+  default_url_options[:port]     = Gitlab.config.web_port if Gitlab.config.web_custom_port?
 
   default from: Gitlab.config.email_from
 
diff --git a/config/initializers/1_settings.rb b/config/initializers/1_settings.rb
index fb039e9e..8b9ed8ae 100644
--- a/config/initializers/1_settings.rb
+++ b/config/initializers/1_settings.rb
@@ -20,17 +20,25 @@ class Settings < Settingslogic
 
     def web_port 
       if web.https
-        nil
+        web['port'] = 443
       else
         web['port'] ||= 80
-      end
+      end.to_i
+    end
+
+    def web_custom_port?
+      ![443, 80].include?(web_port)
     end
 
     def build_url
       raw_url = self.web_protocol
       raw_url << "://"
       raw_url << web_host
-      raw_url << ":#{web_port}" if web_port.to_i != 80
+
+      if web_custom_port?
+        raw_url << ":#{web_port}" 
+      end
+
       raw_url
     end