diff --git a/app/models/enterprise.rb b/app/models/enterprise.rb index 0ecca4734c..e7b81947ae 100644 --- a/app/models/enterprise.rb +++ b/app/models/enterprise.rb @@ -460,6 +460,7 @@ class Enterprise < ApplicationRecord return if white_label_logo_link.blank? + white_label_logo_link.strip! uri = URI(white_label_logo_link) self.white_label_logo_link = "http://#{white_label_logo_link}" if uri.scheme.nil? rescue URI::InvalidURIError diff --git a/spec/models/enterprise_spec.rb b/spec/models/enterprise_spec.rb index 1f624ba6a5..0eb4ec083b 100644 --- a/spec/models/enterprise_spec.rb +++ b/spec/models/enterprise_spec.rb @@ -332,6 +332,12 @@ describe Enterprise do expect(e.white_label_logo_link).to eq "http://www.example.com" end + it "ignores whitespace around the URL form copying and pasting" do + e = build(:enterprise, white_label_logo_link: ' www.example.com ') + expect(e).to be_valid + expect(e.white_label_logo_link).to eq "http://www.example.com" + end + it "does not validate if URL is invalid and can't be infered" do e = build(:enterprise, white_label_logo_link: 'with spaces') expect(e).to be_invalid