FIX: Don't rate limit gravatar downloads

app/models/user_avatar.rb

@@ -23,7 +23,8 @@ class UserAvatar < ActiveRecord::Base
         tempfile = FileHelper.download(
           gravatar_url,
           max_file_size: SiteSetting.max_image_size_kb.kilobytes,
-          tmp_file_name: "gravatar"
+          tmp_file_name: "gravatar",
+          skip_rate_limit: true
         )
         if tempfile
           upload = UploadCreator.new(tempfile, 'gravatar.png', origin: gravatar_url, type: "avatar").create_for(user_id)

lib/file_helper.rb

@@ -7,11 +7,21 @@ class FileHelper
     filename =~ images_regexp
   end
 
-  def self.download(url, max_file_size:, tmp_file_name:, follow_redirect: false, read_timeout: 5)
+  def self.download(url,
+                    max_file_size:,
+                    tmp_file_name:,
+                    follow_redirect: false,
+                    read_timeout: 5,
+                    skip_rate_limit: false)
+
     url = "https:" + url if url.start_with?("//")
     raise Discourse::InvalidParameters.new(:url) unless url =~ /^https?:\/\//
 
-    uri = FinalDestination.new(url, max_redirects: follow_redirect ? 5 : 1).resolve
+    uri = FinalDestination.new(
+      url,
+      max_redirects: follow_redirect ? 5 : 1,
+      skip_rate_limit: skip_rate_limit
+    ).resolve
     return unless uri.present?
 
     extension = File.extname(uri.path)

lib/final_destination.rb

@@ -99,7 +99,9 @@ class FinalDestination
     end
 
     # Rate limit how often this IP can be crawled
+    unless @opts[:skip_rate_limit]
       RateLimiter.new(nil, "crawl-destination-ip:#{address_s}", 100, 1.hour).performed!
+    end
 
     true
   rescue RateLimiter::LimitExceeded