From e1fcad34a9e3c3fd22b3bee66b82d85bd03629fb Mon Sep 17 00:00:00 2001 From: Eugen Rochko Date: Mon, 31 Jul 2017 05:06:20 +0200 Subject: Fix length validator counting things that look like URIs like URLs (#4462) URI.extract is too strong, not limited to URLs, matched real text. Same issue was present in language detector. --- app/validators/status_length_validator.rb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'app/validators') diff --git a/app/validators/status_length_validator.rb b/app/validators/status_length_validator.rb index abf250d65..77be3f1f5 100644 --- a/app/validators/status_length_validator.rb +++ b/app/validators/status_length_validator.rb @@ -24,7 +24,7 @@ class StatusLengthValidator < ActiveModel::Validator def countable_text(status) status.text.dup.tap do |new_text| - URI.extract(new_text).each { |url| new_text.gsub!(url, 'x' * 23) } + new_text.gsub!(FetchLinkCardService::URL_PATTERN, 'x' * 23) new_text.gsub!(Account::MENTION_RE, '@\2') end end -- cgit