filbunke 1.1.4 → 1.1.5

data/VERSION

@@ -1 +1 @@
-1.1.4
+1.1.5

data/filbunke.gemspec

@@ -5,11 +5,11 @@
 
 Gem::Specification.new do |s|
   s.name = %q{filbunke}
-  s.version = "1.1.4"
+  s.version = "1.1.5"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["Wouter de Bie"]
-  s.date = %q{2010-11-10}
+  s.date = %q{2010-11-15}
   s.default_executable = %q{filbunked}
   s.description = %q{Filbunke client and library}
   s.email = %q{wouter@deltaprojects.se}

data/lib/filbunke/client.rb

@@ -1,6 +1,7 @@
 require 'json'
 require 'net/http'
 require 'fileutils'
+require 'digest/md5'
 
 module Filbunke
   class Client
@@ -25,19 +26,22 @@ module Filbunke
       failure = false
       updated_files.each do |raw_file|
         file = File.new(raw_file)
-        if file.url =~ /^http:\/\//
-          local_file_path = ::File.join(repository.local_path, file.path)
-          if update_http_file!(file, local_file_path) then
-            @callbacks.each do |callback|
-              callback.on_update(file)
+
+        local_file_path = ::File.join(repository.local_path, file.path)
+        if file_needs_update?(file, local_file_path)
+          if file.url =~ /^http:\/\//
+            if update_http_file!(file, local_file_path) then
+              @callbacks.each do |callback|
+                callback.on_update(file)
+              end
+              yield file
+            else
+              @logger.log "Unable to get file #{file.url}/#{file.path}!"
+              failure = true
             end
-            yield file
           else
-            @logger.log "Unable to get file #{file.url}/#{file.path}!"
-            failure = true
+            raise "Unsupported protocol for file: #{file.inspect}"
           end
-        else
-          raise "Unsupported protocol for file: #{file.inspect}"
         end
       end
       failure ? last_checkpoint : (updates["checkpoint"] || last_checkpoint)
@@ -74,19 +78,30 @@ module Filbunke
 
     private
 
+    def file_needs_update?(file, local_file_path)
+      return true if file.hash.nil || file.hash == ""
+
+      return true unless File.exists?(local_file_path)
+
+      local_hash = Digest::MD5.hexdigest(File.read(local_file_path))
+      local_hash != file.hash
+    end
+
     def get_updated_file_list(last_checkpoint)
+
       begin
         updates_http = Net::HTTP.new(@repository.host, @repository.port)
         updates_http.start do |http|
           begin
-          updates_path = "/#{UPDATES_ACTION}/#{@repository.name}?#{FROM_CHECKPOINT_KEY}=#{last_checkpoint}"
-          request = Net::HTTP::Get.new(updates_path)
-          response = http.request(request)
-          if response.code.to_i == 200
-            JSON.parse(response.body)
-          else
-            @logger.log "Failed to download updates for #{@repository.name}, error code = #{response.code}"
-          end
+            updates_path = "/#{UPDATES_ACTION}/#{@repository.name}?#{FROM_CHECKPOINT_KEY}=#{last_checkpoint}"
+            request = Net::HTTP::Get.new(updates_path)
+            response = http.request(request)
+            if response.code.to_i == 200
+              JSON.parse(response.body)
+            else
+              @logger.log "Failed to download updates for #{@repository.name}, error code = #{response.code}"
+              {}
+            end
           rescue StandardError => e
             @logger.log "Error getting file list: #{e.message}! Retrying later.."
             {}
@@ -100,25 +115,24 @@ module Filbunke
 
     def update_http_file!(file, local_file_path)
       begin
-      uri = URI.parse(file.url)
-      file_http=Net::HTTP.new(uri.host, uri.port)
-      file_http.start do |http|
-        request = Net::HTTP::Get.new(uri.path)
-        request.basic_auth @repository.user, @repository.pass if @repository.user
-        response = http.request(request)
-        if response.code.to_i == 200
-          write_file!(local_file_path, response.body)
-        elsif response.code.to_i == 404
-          delete_file!(local_file_path)
-        else
-          @logger.log "Failed to update file #{uri}, error code = #{response.code}"
+        uri = URI.parse(file.url)
+        file_http=Net::HTTP.new(uri.host, uri.port)
+        file_http.start do |http|
+          request = Net::HTTP::Get.new(uri.path)
+          request.basic_auth @repository.user, @repository.pass if @repository.user
+          response = http.request(request)
+          if response.code.to_i == 200
+            write_file!(local_file_path, response.body)
+          elsif response.code.to_i == 404
+            delete_file!(local_file_path)
+          else
+            @logger.log "Failed to update file #{uri}, error code = #{response.code}"
+          end
+          return true
         end
-        return true
-      end
       rescue StandardError => e
         return false
       end
-      
     end
 
     def write_file!(file_path, contents)

metadata

@@ -5,8 +5,8 @@ version: !ruby/object:Gem::Version
   segments: 
   - 1
   - 1
-  - 4
-  version: 1.1.4
+  - 5
+  version: 1.1.5
 platform: ruby
 authors: 
 - Wouter de Bie
@@ -14,7 +14,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2010-11-10 00:00:00 +01:00
+date: 2010-11-15 00:00:00 +01:00
 default_executable: filbunked
 dependencies: 
 - !ruby/object:Gem::Dependency