sift 1.1.6.0 → 1.1.6.2

checksums.yaml

@@ -1,15 +1,15 @@
 ---
 !binary "U0hBMQ==":
   metadata.gz: !binary |-
-    YzY5Mjg2YTQ4ZjRmYWRlOTQ4NTk4NzFmNjNiZjFmYmE3NTg3MDM2MQ==
+    MzdkOThmNmZiYWRmMDI3ODgyYjI4MjYyMjc2ZDg1YjhkOTdlNGU0MQ==
   data.tar.gz: !binary |-
-    ZGIzOTAwOWE4OTExMWQ0MzlkYTRiYjk4OTQzNmU0NzQ5YWVkMzg3OQ==
+    N2RiZGI4ZTFlZWI1YjBkNDQ1MDQyM2MxYTNmMDNlMzUxZjMyZDM1OA==
 SHA512:
   metadata.gz: !binary |-
-    NTM1ZTM3ZGU3ODI4ZmE5YzdiMTQ4ZWMxODUxZDhjZTAzZmZiYTAzMGZjMzZk
-    YzYxZjI5MDk1MjQ4ZWI1NDFhNWI1ODY2Y2MyYjk1MTI3YTA4ZjdmZjhjOTNl
-    NTc0ZGYzYzM5NmI0N2ZiM2JmMzgzMDg5M2ZhZWIwYjQwNWY1MGQ=
+    M2YzZTYyZmJhMzBjMDVmZmJiNWM5N2ZmNmJmNjkzOTMzN2ExODIxMTIwOGRh
+    OTBmYThlZDA1ZDVmMDYzZTI4MjIxZDcxYTFjYjZhMzc0MzlkYjk1ZThhZTMy
+    YjhlNGUwMzEwOTI2MjhlNGI5ODYwZTJmYmRkNGI4YzlkM2IzYTE=
   data.tar.gz: !binary |-
-    NzlhNGRiODU4M2UzNWQzZGRkM2U2OGM4MTk5MWVkZmE4ZjQ1YmJkYzcxZjI2
-    OWY1OGI5OWRjMzA4ODM1YWVkZGE4MGVhMzBhMjRkNDg3M2ZlMzNhY2M0YTQ4
-    OWJkNjkwN2EzNzIyOTRjMGNhOGNjNzIyMWYzNDdmN2Y3OWQ4ZTE=
+    NGVjMmZjYTJiOWI0NDA4N2Y5NjU3NGNkNDk1NTNjMDdhZGQ3NGZmNDEwNTYw
+    MTVmZGQ5ZDQ4NTY4ZmEyMTkwNjYwZDFhMTBmZDgwNDNiYWVjMWUyZWE0OTk1
+    NThlNTY5N2M2ZDAyZTBkMWI5Mjg4YzVhMmM2M2MyZDU5YjU1MDE=

data/lib/sift/client.rb

@@ -76,7 +76,7 @@ module Sift
       @path = path
       @timeout = timeout
 
-      
+
     end
 
     def api_key
@@ -108,7 +108,7 @@ module Sift
     #   Overrides the default API path with a different URL.
     #
     # return_score (optional)
-    #   Whether the API response should include a score for this user (the score will 
+    #   Whether the API response should include a score for this user (the score will
     #   be calculated using the submitted event
     #
     # == Returns:
@@ -117,9 +117,10 @@ module Sift
     #   the status message and status code. In general, you can ignore the returned
     #   result, though.
     #
-    def track(event, properties = {}, timeout = nil, path = nil, return_score = false)
+    def track(event, properties = {}, timeout = nil, path = nil, return_score = false, api_key = @api_key)
       raise(RuntimeError, "event must be a non-empty string") if (!event.is_a? String) || event.empty?
       raise(RuntimeError, "properties cannot be empty") if properties.empty?
+      raise(RuntimeError, "Bad api_key parameter") if api_key.empty?
       path ||= @path
       timeout ||= @timeout
       if return_score
@@ -127,7 +128,7 @@ module Sift
       end
       options = {
         :body => MultiJson.dump(delete_nils(properties).merge({"$type" => event,
-                                                               "$api_key" => @api_key})),
+                                                               "$api_key" => api_key})),
         :headers => {"User-Agent" => user_agent}
       }
       options.merge!(:timeout => timeout) unless timeout.nil?
@@ -153,15 +154,16 @@ module Sift
     #   A Response object is returned and captures the status message and
     #   status code. In general, you can ignore the returned result, though.
     #
-    def score(user_id, timeout = nil)
+    def score(user_id, timeout = nil, api_key = @api_key)
 
       raise(RuntimeError, "user_id must be a non-empty string") if (!user_id.is_a? String) || user_id.to_s.empty?
+      raise(RuntimeError, "Bad api_key parameter") if api_key.empty?
       timetout ||= @timeout
 
       options = { :headers => {"User-Agent" => user_agent} }
       options.merge!(:timeout => timeout) unless timeout.nil?
 
-      response = self.class.get("/v#{API_VERSION}/score/#{user_id}/?api_key=#{@api_key}", options)
+      response = self.class.get("/v#{API_VERSION}/score/#{user_id}/?api_key=#{api_key}", options)
       Response.new(response.body, response.code)
 
     end
@@ -185,12 +187,12 @@ module Sift
     #   A Response object is returned and captures the status message and
     #   status code. In general, you can ignore the returned result, though.
     #
-    def label(user_id, properties = {}, timeout = nil)
+    def label(user_id, properties = {}, timeout = nil, api_key = @api_key)
 
       raise(RuntimeError, "user_id must be a non-empty string") if (!user_id.is_a? String) || user_id.to_s.empty?
 
       path = Sift.current_users_label_api_path(user_id)
-      track("$label", delete_nils(properties), timeout, path)
+      track("$label", delete_nils(properties), timeout, path, false, api_key)
     end
 
     private

data/lib/sift/version.rb

@@ -1,4 +1,4 @@
 module Sift
-  VERSION = "1.1.6.0"
+  VERSION = "1.1.6.2"
   API_VERSION = "203"
 end

data/spec/unit/client_spec.rb

@@ -141,6 +141,25 @@ describe Sift::Client do
     response.api_error_message.should eq("OK")
   end
 
+  it "Successfully submits event with overriden key" do
+    response_json = { :status => 0, :error_message => "OK"}
+    stub_request(:post, "https://api.siftscience.com/v203/events").
+      with { | request|
+        parsed_body = JSON.parse(request.body)
+        parsed_body.should include("$buyer_user_id" => "123456")
+        parsed_body.should include("$api_key" => "overridden")
+      }.to_return(:status => 200, :body => MultiJson.dump(response_json), :headers => {})
+
+    api_key = "foobar"
+    event = "$transaction"
+    properties = valid_transaction_properties
+
+    response = Sift::Client.new(api_key).track(event, properties, nil, nil, false, "overridden")
+    response.ok?.should eq(true)
+    response.api_status.should eq(0)
+    response.api_error_message.should eq("OK")
+  end
+
   it "Successfuly scrubs nils" do
 
     response_json = { :status => 0, :error_message => "OK" }
@@ -183,6 +202,23 @@ describe Sift::Client do
     response.body["score"].should eq(0.93)
   end
 
+  it "Successfully fetches a score with an overridden key" do
+
+    api_key = "foobar"
+    response_json = score_response_json
+
+    stub_request(:get, "https://api.siftscience.com/v203/score/247019/?api_key=overridden").
+      to_return(:status => 200, :body => MultiJson.dump(response_json), :headers => {})
+
+    response = Sift::Client.new(api_key).score(score_response_json[:user_id], nil, "overridden")
+    response.ok?.should eq(true)
+    response.api_status.should eq(0)
+    response.api_error_message.should eq("OK")
+
+    response.body["score"].should eq(0.93)
+  end
+
+
   it "Successfuly make a sync score request" do
 
     api_key = "foobar"
@@ -204,4 +240,6 @@ describe Sift::Client do
     response.body["score_response"]["score"].should eq(0.93)
   end
 
+
+
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: sift
 version: !ruby/object:Gem::Version
-  version: 1.1.6.0
+  version: 1.1.6.2
 platform: ruby
 authors:
 - Fred Sadaghiani
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-09-03 00:00:00.000000000 Z
+date: 2014-09-19 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rspec
@@ -124,7 +124,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: sift
-rubygems_version: 2.2.2
+rubygems_version: 2.1.11
 signing_key: 
 specification_version: 4
 summary: Sift Science Ruby API Gem