Fallback to GET when HEAD is not allowed (nanoc#274)

Author: cakebaker

lib/nanoc/extra/checking/checks/external_links.rb

@@ -103,6 +103,9 @@ def validate(href)
         begin
           Timeout::timeout(10) do
             res = request_url_once(url)
+            if res.code == '405'
+              res = request_url_once(url, Net::HTTP::Get)
+            end
           end
         rescue => e
           return Result.new(href, e.message)
@@ -133,9 +136,9 @@ def validate(href)
       raise 'should not have gotten here'
     end
 
-    def request_url_once(url)
+    def request_url_once(url, req_method = Net::HTTP::Head)
       path = (url.path.nil? || url.path.empty? ? '/' : url.path)
-      req = Net::HTTP::Head.new(path)
+      req = req_method.new(path)
       http = Net::HTTP.new(url.host, url.port)
       if url.instance_of? URI::HTTPS
         http.use_ssl = true

test/extra/checking/checks/test_external_links.rb

@@ -38,4 +38,18 @@ def check.request_url_once(url)
     end
   end
 
+  def test_fallback_to_get_when_head_is_not_allowed
+    with_site do |site|
+      #Create check
+      check = Nanoc::Extra::Checking::Checks::ExternalLinks.new(site)
+      def check.request_url_once(url, req_method = Net::HTTP::Head)
+        Net::HTTPResponse.new('1.1', (req_method == Net::HTTP::Head || url.path == '/405') ? '405' : '200', 'okay')
+      end
+
+      #Test
+      assert_nil check.validate('http://127.0.0.1:9204')
+      refute_nil check.validate('http://127.0.0.1:9204/405')
+    end
+  end
+
 end