Commit 9e52a2dc authored by Douwe Maan's avatar Douwe Maan

Don't use URI.regexp to validate since it doens't have start/end anchors.

parent cebb74a0
...@@ -24,7 +24,7 @@ class ApplicationSetting < ActiveRecord::Base ...@@ -24,7 +24,7 @@ class ApplicationSetting < ActiveRecord::Base
validates :home_page_url, validates :home_page_url,
allow_blank: true, allow_blank: true,
format: { with: URI::regexp(%w(http https)), message: "should be a valid url" }, format: { with: /\A#{URI.regexp(%w(http https))}\z/, message: "should be a valid url" },
if: :home_page_url_column_exist if: :home_page_url_column_exist
validates_each :restricted_visibility_levels do |record, attr, value| validates_each :restricted_visibility_levels do |record, attr, value|
......
...@@ -28,7 +28,7 @@ class WebHook < ActiveRecord::Base ...@@ -28,7 +28,7 @@ class WebHook < ActiveRecord::Base
default_timeout Gitlab.config.gitlab.webhook_timeout default_timeout Gitlab.config.gitlab.webhook_timeout
validates :url, presence: true, validates :url, presence: true,
format: { with: URI::regexp(%w(http https)), message: "should be a valid url" } format: { with: /\A#{URI.regexp(%w(http https))}\z/, message: "should be a valid url" }
def execute(data) def execute(data)
parsed_url = URI.parse(url) parsed_url = URI.parse(url)
......
...@@ -25,7 +25,7 @@ class BambooService < CiService ...@@ -25,7 +25,7 @@ class BambooService < CiService
validates :bamboo_url, validates :bamboo_url,
presence: true, presence: true,
format: { with: URI::regexp }, format: { with: /\A#{URI.regexp}\z/ },
if: :activated? if: :activated?
validates :build_key, presence: true, if: :activated? validates :build_key, presence: true, if: :activated?
validates :username, validates :username,
......
...@@ -18,7 +18,7 @@ class ExternalWikiService < Service ...@@ -18,7 +18,7 @@ class ExternalWikiService < Service
prop_accessor :external_wiki_url prop_accessor :external_wiki_url
validates :external_wiki_url, validates :external_wiki_url,
presence: true, presence: true,
format: { with: URI::regexp }, format: { with: /\A#{URI.regexp}\z/ },
if: :activated? if: :activated?
def title def title
......
...@@ -25,7 +25,7 @@ class TeamcityService < CiService ...@@ -25,7 +25,7 @@ class TeamcityService < CiService
validates :teamcity_url, validates :teamcity_url,
presence: true, presence: true,
format: { with: URI::regexp }, if: :activated? format: { with: /\A#{URI.regexp}\z/ }, if: :activated?
validates :build_type, presence: true, if: :activated? validates :build_type, presence: true, if: :activated?
validates :username, validates :username,
presence: true, presence: true,
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment