Fix duplicate web archive URL when retrying

fixes #46

Author: StrawberryMaster

Gemfile

@@ -1,4 +1,4 @@
 source "https://rubygems.org"
 
-gem 'concurrent-ruby', '~> 1.3', '>= 1.3.4'
+gem 'concurrent-ruby', '~> 1.3', '>= 1.3.6'
 gemspec

lib/wayback_machine_downloader.rb

@@ -1122,10 +1122,14 @@ def safe_display_url(url)
   def download_with_retry(file_path, file_url, file_timestamp, connection, redirect_count = 0)
     retries = 0
     begin
-      wayback_url = if @rewritten
-        "https://web.archive.org/web/#{file_timestamp}/#{file_url}"
+      if redirect_count > 0 && file_url =~ %r{\Ahttps?://}
+        wayback_url = file_url
       else
-        "https://web.archive.org/web/#{file_timestamp}id_/#{file_url}"
+        wayback_url = if @rewritten
+          "https://web.archive.org/web/#{file_timestamp}/#{file_url}"
+        else
+          "https://web.archive.org/web/#{file_timestamp}id_/#{file_url}"
+        end
       end
 
       # Escape characters that are not valid in URI()

wayback_machine_downloader.gemspec

@@ -10,7 +10,7 @@ Gem::Specification.new do |s|
   s.homepage    = "https://github.com/StrawberryMaster/wayback-machine-downloader"
   s.license     = "MIT"
   s.required_ruby_version = ">= 3.4.3"
-  s.add_runtime_dependency "concurrent-ruby", "~> 1.3", ">= 1.3.4"
+  s.add_runtime_dependency "concurrent-ruby", "~> 1.3", ">= 1.3.6"
   s.add_development_dependency "rake", "~> 12.2"
   s.add_development_dependency "minitest", "~> 5.2"
 end