specter/mastodon
1
0
Fork 1
forked from fedi/mastodon
Code Pull requests Activity

Strip spaces around URL when adding a relay (#22655)

Browse source 
* Strip spaces around URL when adding a relay

Fixes #22650

* Gracefuly handle URL parsing errors in URL validator
This commit is contained in:
Claire 2023-01-05 13:33:33 +01:00 committed by talon
parent 2a8be395fb
commit fddaa40a03
2 changed files with 7 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
app/models/relay.rb
View file

@ -18,6 +18,7 @@ class Relay < ApplicationRecord
scope :enabled, -> { accepted }
before_validation :strip_url
before_destroy :ensure_disabled
alias enabled? accepted?
@ -74,4 +75,8 @@ class Relay < ApplicationRecord
def ensure_disabled
disable! if enabled?
end
def strip_url
inbox_url&.strip!
end
end

2
app/validators/url_validator.rb
View file

@ -10,5 +10,7 @@ class URLValidator < ActiveModel::EachValidator
def compliant?(url)
parsed_url = Addressable::URI.parse(url)
parsed_url && %w(http https).include?(parsed_url.scheme) && parsed_url.host
rescue Addressable::URI::InvalidURIError
false
end
end