From 984ea976d1ca308955650ed137118b5d1fbd493b Mon Sep 17 00:00:00 2001 From: Steve Kemp Date: Thu, 15 Nov 2012 10:28:17 +0000 Subject: A HTTP-body might be legitimately empty --- debian/changelog | 6 ++++++ lib/custodian/webfetch.rb | 7 ++++--- 2 files changed, 10 insertions(+), 3 deletions(-) diff --git a/debian/changelog b/debian/changelog index 77fd5a0..c87c33d 100644 --- a/debian/changelog +++ b/debian/changelog @@ -1,3 +1,9 @@ +custodian (2012:1115-3) stable; urgency=low + + * A HTTP-request might result in a legitimately empty body. + + -- Steve Kemp Thu, 15 Nov 2012 10:28:01 +0000 + custodian (2012:1115-2) stable; urgency=low * Throw exceptions on bogus configuration file entries. diff --git a/lib/custodian/webfetch.rb b/lib/custodian/webfetch.rb index 5c04b31..0b5f9c6 100755 --- a/lib/custodian/webfetch.rb +++ b/lib/custodian/webfetch.rb @@ -59,10 +59,11 @@ class WebFetch # - # If both files are size zero then we clearly failed. + # If the header was empty then we're a failure. # - if ( ( File.size( body ) == 0 ) || - ( File.size( head ) == 0 ) ) + # (A body might be legitimately empty.) + # + if ( File.size( head ) == 0 ) # # Cleanup -- cgit v1.2.1