discourse/lib/validators/upload_url_validator.rb

class UploadUrlValidator < ActiveModel::EachValidator
  def validate_each(record, attribute, value)
    if value.present?
      uri =
        begin
          URI.parse(value)
        rescue URI::Error
        end

      unless uri && Upload.exists?(url: value)
        record.errors[attribute] << (options[:message] || I18n.t('errors.messages.invalid'))
      end
    end
  end
end
SECURITY: Make sure uploaded_urls have corresponding upload records 2016-07-29 01:54:17 +08:00			`class UploadUrlValidator < ActiveModel::EachValidator`
			`def validate_each(record, attribute, value)`
			`if value.present?`
Remove use of `rescue nil`. * `rescue nil` is a really bad pattern to use in our code base. We should rescue errors that we expect the code to throw and not rescue everything because we're unsure of what errors the code would throw. This would reduce the amount of pain we face when debugging why something isn't working as expexted. I've been bitten countless of times by errors being swallowed as a result during debugging sessions. 2018-03-28 16:20:08 +08:00			`uri =`
			`begin`
			`URI.parse(value)`
FIX: store the topic links using the cooked upload url 2018-08-14 18:23:32 +08:00			`rescue URI::Error`
Remove use of `rescue nil`. * `rescue nil` is a really bad pattern to use in our code base. We should rescue errors that we expect the code to throw and not rescue everything because we're unsure of what errors the code would throw. This would reduce the amount of pain we face when debugging why something isn't working as expexted. I've been bitten countless of times by errors being swallowed as a result during debugging sessions. 2018-03-28 16:20:08 +08:00			`end`
SECURITY: Make sure uploaded_urls have corresponding upload records 2016-07-29 01:54:17 +08:00
Merge diffs from master 2019-02-01 06:24:35 +08:00			`unless uri && Upload.exists?(url: value)`
SECURITY: Make sure uploaded_urls have corresponding upload records 2016-07-29 01:54:17 +08:00			`record.errors[attribute] << (options[:message] \|\| I18n.t('errors.messages.invalid'))`
			`end`
			`end`
			`end`
			`end`