pinger: Get whole url, don't just head.

Asking for only the head worked in my tests, but I've found a site where it
didn't -- apparently ikiwiki didn't get a chance to do or finish the
refresh when HEADed. Getting the whole url, waiting for ikiwiki to finish,
avoided the update problem.

diff --git a/IkiWiki/Plugin/pinger.pm b/IkiWiki/Plugin/pinger.pm
index 4a808866164d7e2f1e6e185b97b37e50cf0c307b..c20ecb5d4ef3a663f5496135f56ddb5369c5dd87 100644 (file)
--- a/IkiWiki/Plugin/pinger.pm
+++ b/IkiWiki/Plugin/pinger.pm
@@ -106,7 +106,7 @@ sub ping {
                        # will still be avoided.
                        next if $url=~/^\Q$config{cgiurl}\E/;
                        
                        # will still be avoided.
                        next if $url=~/^\Q$config{cgiurl}\E/;
                        

-                       $ua->head($url);
+                       $ua->get($url);

                }
                
                exit 0;
                }
                
                exit 0;

diff --git a/debian/changelog b/debian/changelog
index 13ef9afed30dbb39ea6e347b0213fdbdea9fc5e1..6772aded9f5a32c14c2a1f595c57d3d3dfb0e8d7 100644 (file)
--- a/debian/changelog
+++ b/debian/changelog
@@ -12,6 +12,7 @@ ikiwiki (3.02) UNRELEASED; urgency=low
     that take a page name parameter. Previously, match_created_before(),
     match_created_after(), match_sourcepage(), and match_destpage()
     did not support that, and the docs were not clear.
     that take a page name parameter. Previously, match_created_before(),
     match_created_after(), match_sourcepage(), and match_destpage()
     did not support that, and the docs were not clear.

+  * pinger: Get whole url, don't just head.

 
  -- Joey Hess <joeyh@debian.org>  Tue, 06 Jan 2009 15:02:52 -0500
 
 
  -- Joey Hess <joeyh@debian.org>  Tue, 06 Jan 2009 15:02:52 -0500