arrest 0.0.13 → 0.0.14

data/lib/arrest/handler.rb

@@ -1,5 +1,9 @@
 module Arrest
 
+  RequestLog = Struct.new(:method, :url, :body)
+  ResponseLog = Struct.new(:status, :body)
+  CallLog = Struct.new(:request, :response)
+
   # The classes in this module supply default behaviour
   # for certain processing steps in the consumttion ot the 
   # rest api
@@ -39,6 +43,11 @@ module Arrest
       end
     end
 
+    class CallLogger
+      def self.log(requestLog, responseLog)
+      end
+    end
+
   end
 
 end

data/lib/arrest/http_source.rb

@@ -1,6 +1,8 @@
 require 'faraday'
+require 'arrest/handler'
 
 module Arrest
+
   class HttpSource
 
     def initialize base
@@ -22,6 +24,9 @@ module Arrest
         req.url sub, filter
         add_headers req.headers
       end
+      rql = RequestLog.new(:get, "#{sub}#{hash_to_query filter}", nil)
+      rsl = ResponseLog.new(response.env[:status], response.body)
+      Arrest::Source.call_logger.log(rql, rsl)
       if response.env[:status] != 200
         raise Errors::DocumentNotFoundError 
       end
@@ -33,6 +38,9 @@ module Arrest
         req.url sub, filter
         add_headers req.headers
       end
+      rql = RequestLog.new(:get, "#{sub}#{hash_to_query filter}", nil)
+      rsl = ResponseLog.new(response.env[:status], response.body)
+      Arrest::Source.call_logger.log(rql, rsl)
       response.body
     end
 
@@ -42,6 +50,9 @@ module Arrest
         req.url rest_resource.resource_location
         add_headers req.headers
       end
+      rql = RequestLog.new(:delete, rest_resource.resource_location, nil)
+      rsl = ResponseLog.new(response.env[:status], response.body)
+      Arrest::Source.call_logger.log(rql, rsl)
       response.env[:status] == 200
     end
 
@@ -50,11 +61,19 @@ module Arrest
       hash = rest_resource.to_jhash
       hash.delete(:id)
       hash.delete("id")
+      body = hash.to_json
 
       response = self.connection().put do |req|
         req.url rest_resource.resource_location
         add_headers req.headers
-        req.body = hash.to_json
+        req.body = body
+      end
+      rql = RequestLog.new(:put, rest_resource.resource_location, body)
+      rsl = ResponseLog.new(response.env[:status], response.body)
+      Arrest::Source.call_logger.log(rql, rsl)
+      if response.env[:status] != 200
+        err = Arrest::Source.error_handler.convert(response.body, response.env[:status])
+        rest_resource.errors.add(:base, err)
       end
       response.env[:status] == 200
     end
@@ -74,17 +93,17 @@ module Arrest
         add_headers req.headers
         req.body = body
       end
+      rql = RequestLog.new(:post, rest_resource.resource_path, body)
+      rsl = ResponseLog.new(response.env[:status], response.body)
+      Arrest::Source.call_logger.log(rql, rsl)
       if (response.env[:status] == 201)
         location = response.env[:response_headers][:location]
         id = location.gsub(/^.*\//, '')
         rest_resource.id= id
         true
       else
-        if response.env[:status] != 201
-          err = Arrest::Source.error_handler.convert(response.body, response.env[:status])
-          rest_resource.errors.add(:base, err)
-        end
-        puts "unable to create: #{response.env[:response_headers]} body: #{response.body} "
+        err = Arrest::Source.error_handler.convert(response.body, response.env[:status])
+        rest_resource.errors.add(:base, err)
         false
       end
       
@@ -100,5 +119,19 @@ module Arrest
       end
     end
 
+    def hash_to_query hash
+      return "" if hash.empty?
+      r = ""
+      c = '?'
+      hash.each_pair do |k,v|
+        r << c
+        r << k.to_s
+        r << '='
+        r << v.to_s
+        c = '&'
+      end
+      r
+    end
+
   end
 end

data/lib/arrest/source.rb

@@ -15,6 +15,7 @@ module Arrest
       attr_accessor :json_key_converter
       attr_accessor :skip_validations
       attr_accessor :error_handler
+      attr_accessor :call_logger
 
       def source=(host=nil)
         if [nil, ""].include?(host)
@@ -54,6 +55,7 @@ module Arrest
   Source.debug = false
   Source.json_key_converter = Handlers::KeyConverter
   Source.error_handler = Handlers::ErrorHandler
+  Source.call_logger = Handlers::CallLogger
   Source.skip_validations = false
 
 end

data/lib/arrest/version.rb

@@ -1,3 +1,3 @@
 module Arrest
-  VERSION = "0.0.13"
+  VERSION = "0.0.14"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: arrest
 version: !ruby/object:Gem::Version
-  version: 0.0.13
+  version: 0.0.14
   prerelease: 
 platform: ruby
 authors:
@@ -9,11 +9,11 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-01-11 00:00:00.000000000Z
+date: 2012-01-12 00:00:00.000000000Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: json
-  requirement: &13647940 !ruby/object:Gem::Requirement
+  requirement: &18819600 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -21,10 +21,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *13647940
+  version_requirements: *18819600
 - !ruby/object:Gem::Dependency
   name: faraday
-  requirement: &13647020 !ruby/object:Gem::Requirement
+  requirement: &18818880 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - =
@@ -32,10 +32,10 @@ dependencies:
         version: 0.7.5
   type: :runtime
   prerelease: false
-  version_requirements: *13647020
+  version_requirements: *18818880
 - !ruby/object:Gem::Dependency
   name: activemodel
-  requirement: &13646240 !ruby/object:Gem::Requirement
+  requirement: &18818160 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -43,10 +43,10 @@ dependencies:
         version: '3'
   type: :runtime
   prerelease: false
-  version_requirements: *13646240
+  version_requirements: *18818160
 - !ruby/object:Gem::Dependency
   name: bundler
-  requirement: &13645500 !ruby/object:Gem::Requirement
+  requirement: &18817680 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -54,10 +54,10 @@ dependencies:
         version: 1.0.0
   type: :development
   prerelease: false
-  version_requirements: *13645500
+  version_requirements: *18817680
 - !ruby/object:Gem::Dependency
   name: rake
-  requirement: &13645080 !ruby/object:Gem::Requirement
+  requirement: &18817300 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -65,10 +65,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *13645080
+  version_requirements: *18817300
 - !ruby/object:Gem::Dependency
   name: rdoc
-  requirement: &13644600 !ruby/object:Gem::Requirement
+  requirement: &18816800 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -76,10 +76,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *13644600
+  version_requirements: *18816800
 - !ruby/object:Gem::Dependency
   name: rspec
-  requirement: &13644080 !ruby/object:Gem::Requirement
+  requirement: &18816280 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -87,10 +87,10 @@ dependencies:
         version: '2'
   type: :development
   prerelease: false
-  version_requirements: *13644080
+  version_requirements: *18816280
 - !ruby/object:Gem::Dependency
   name: rr
-  requirement: &13643640 !ruby/object:Gem::Requirement
+  requirement: &18815860 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -98,10 +98,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *13643640
+  version_requirements: *18815860
 - !ruby/object:Gem::Dependency
   name: simplecov
-  requirement: &13643160 !ruby/object:Gem::Requirement
+  requirement: &18815400 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -109,10 +109,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *13643160
+  version_requirements: *18815400
 - !ruby/object:Gem::Dependency
   name: rack
-  requirement: &13642740 !ruby/object:Gem::Requirement
+  requirement: &18814980 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -120,7 +120,7 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *13642740
+  version_requirements: *18814980
 description: Consume a rest API in a AR like fashion
 email:
 - axel.tetzlaff@fortytools.com