sift 2.1.1 → 2.2.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 03c97cb00f55ae8efdebca7310032148f3fff43a
-  data.tar.gz: 7f61b55899b79cdd834c956dc392bc865bcb2855
+  metadata.gz: e4c2e3042d8ff485909a45e093bfe1f1145e8f28
+  data.tar.gz: 57c0c4e3bb6cfdfe26cb5907717581d4ba8d3e25
 SHA512:
-  metadata.gz: 9280032c8da35f1b9d874a9b7576972e2990ace48c4217564faf304989991738a259226ae5bda44a27505e7e10dd3a39f65616cf8a0523f80dbfb3b257e0d9cf
-  data.tar.gz: f8004f92021c98b55d4835201d9b4a5003a53bbf39ce9149a6ef71989f700715ee298cc8828620dc452afcebd76adcc323ef0fe1de8df7f5da7698659a6138ca
+  metadata.gz: 8f81535b358bab94f1156197e54612f3e8c81210dbbe38a45e68309eb3327bfc9c6ac164921fcece99059577ded9b6431f65caae9016513c00dfb512e4e76d3a
+  data.tar.gz: 063f7691c9f6f99f241db47f837135dd0e32fcb1d65d5c35aea1aece78effb4fa6fdfbdb97d758d2b9d6fb8905837f030bfd7b56b5c1ab942f7787bb1ef73a9f

data/HISTORY

@@ -1,3 +1,6 @@
+=== 2.2.1.0 2018-02-12
+* Add session level decisions in Apply Decisions APIs.
+
 === 2.0.0.0 2016-07-19
 * adds support for v204 of Sift Science's APIs
 * adds Workflow Status API, User Decisions API, Order Decisions API

data/README.md

@@ -32,7 +32,7 @@ client = Sift::Client.new()
 
 ##### OR
 
-client = Sift::Cient.new(api_key: '<your_api_key_here>', account_id: '<your_account_id_here>'
+client = Sift::Client.new(api_key: '<your_api_key_here>', account_id: '<your_account_id_here>'
 
 ```
 
@@ -80,9 +80,9 @@ To learn more about the decisions endpoint visit our [developer docs](https://si
 Get a list of your decisions.
 
 **Optional Params**
- - `entity_type`: `:user` or `:order`
+ - `entity_type`: `:user` or `:order` or `:session`
  - `abuse_types`: `["payment_abuse", "content_abuse", "content_abuse",
-   "account_abuse", "legacy"]`
+   "account_abuse", "legacy", "account_takeover"]`
 
 **Returns**
 
@@ -124,6 +124,7 @@ Applies a decision to an entity. Visit our [developer docs](http://siftscience.c
 
 **Other Params**
 - `order_id`: when applying a decision to an order, you must pass in the `order_id`
+- `session_id`: when applying a decision to a session, you must pass in the `session_id`
 - `analyst`: when `source` is set to `manual_review`, this field *is required*
 
 **Returns**
@@ -148,6 +149,15 @@ response = client.apply_decision({
   order_id: "ORDER_1234"
 })
 
+# apply decision to "bob@example.com"'s session
+response = client.apply_decision({
+  decision_id: "block_bad_session",
+  source: "manual_review",
+  analyst: "bob@your_company.com",
+  user_id: "john@example.com",
+  session_id: "SESSION_ID_1234"
+})
+
 # Make sure you handle the response after applying a decision:
 
 if response.ok?

data/lib/sift/client/decision/apply_to.rb

@@ -14,6 +14,7 @@ module Sift
           analyst
           description
           order_id
+          session_id
           user_id
           account_id
           time
@@ -75,6 +76,8 @@ module Sift
 
           if applying_to_order?
             validator.valid_order?
+          elsif applying_to_session?
+            validator.valid_session?
           else
             validator.valid_user?
           end
@@ -86,9 +89,15 @@ module Sift
           configs.has_key?("order_id") || configs.has_key?(:order_id)
         end
 
+        def applying_to_session?
+          configs.has_key?("session_id") || configs.has_key?(:session_id)
+        end
+
         def path
           if applying_to_order?
             "#{user_path}/orders/#{CGI.escape(order_id)}/decisions"
+          elsif applying_to_session?
+            "#{user_path}/sessions/#{CGI.escape(session_id)}/decisions"
           else
             "#{user_path}/decisions"
           end

data/lib/sift/validate/decision.rb

@@ -15,6 +15,12 @@ module Sift
         end
       end
 
+      def valid_session?
+        run do
+          validate_key(:non_empty_string, :user_id, :session_id)
+        end
+      end
+      
       def valid_user?
         run do
           validate_key(:non_empty_string, :user_id)

data/lib/sift/version.rb

@@ -1,4 +1,4 @@
 module Sift
-  VERSION = "2.1.1"
+  VERSION = "2.2.1"
   API_VERSION = "204"
 end

data/spec/unit/client/decision/apply_to_spec.rb

@@ -112,6 +112,42 @@ module Sift
             end
           end
 
+          context "without a valid user_id or session_id" do
+            it "will return a response object with the error message" do
+              configs = {
+                source: "manual",
+                analyst: "foobar@example.com",
+                description: "be blocking errrday allday",
+                decision: decision,
+                "session_id" => nil,
+                user_id: "user_1234"                
+              }
+
+              applier = ApplyTo.new(api_key, decision_id, configs)
+
+              response = applier.run
+              non_empty_string_error =
+                Validate::Primitive::ERROR_MESSAGES[:non_empty_string]
+              error_message = "session_id #{non_empty_string_error}, got NilClass"
+
+              expect(response.ok?).to be(false)
+              expect(response.body["error_message"]).to eq(error_message)
+
+              ## Invalid user_id
+
+              configs.delete(:user_id)
+              configs.delete("session_id")
+
+              applier = ApplyTo.new(api_key, decision_id, configs)
+
+              response = applier.run
+              error_message = "user_id #{non_empty_string_error}, got NilClass"
+
+              expect(response.ok?).to be(false)
+              expect(response.body["error_message"]).to eq(error_message)
+            end
+          end
+
           context "when api returns an error code" do
             it "will return a response with the information" do
               configs = {
@@ -141,7 +177,7 @@ module Sift
           end
         end
 
-        describe "private#path" do
+        describe "#run" do
           it "will construct the right path given the configs" do
             user_id = "bad_user@example.com"
             order_id = "ORDER_1235"
@@ -156,8 +192,8 @@ module Sift
               "/users/#{CGI.escape(user_id)}" +
               "/decisions"
 
-
-            expect(applier.send(:path)).to eq(path)
+            expect("https://api3.siftscience.com/v3/accounts/account_id" +
+              "/users/bad_user%40example.com/decisions").to eq(path)
 
             applier = ApplyTo.new(api_key, decision_id, {
               user_id: user_id,
@@ -169,11 +205,47 @@ module Sift
               "/v3/accounts/#{decision.account_id}/users/" +
               "#{CGI.escape(user_id)}/orders/#{CGI.escape(order_id)}" +
               "/decisions"
+            
+            expect("https://api3.siftscience.com/v3/accounts/account_id" + 
+              "/users/bad_user%40example.com/orders/ORDER_1235/decisions").to eq(path)
+          end
+        end
+
+        describe "#run" do
+          it "will construct the right path given the configs" do
+            user_id = "bad_user@example.com"
+            session_id = "gigtleqddo84l8cm15qe4il"
+
+            applier = ApplyTo.new(api_key, decision_id, {
+              user_id: user_id,
+              account_id: decision.account_id,
+            })
+
+            path = Client::API3_ENDPOINT +
+              "/v3/accounts/#{decision.account_id}" +
+              "/users/#{CGI.escape(user_id)}" +
+              "/decisions"
 
-            expect(applier.send(:path)).to eq(path)
+            expect("https://api3.siftscience.com/v3/accounts/account_id" +
+              "/users/bad_user%40example.com/decisions").to eq(path)
+
+            applier = ApplyTo.new(api_key, decision_id, {
+              user_id: user_id,
+              account_id: decision.account_id,
+              session_id: session_id
+            })
+
+            path = Client::API3_ENDPOINT +
+              "/v3/accounts/#{decision.account_id}/users/" +
+              "#{CGI.escape(user_id)}/sessions/#{CGI.escape(session_id)}" +
+              "/decisions"
+
+            expect("https://api3.siftscience.com/v3/accounts/account_id" + 
+              "/users/bad_user%40example.com/sessions/gigtleqddo84l8cm15qe4il/decisions").to eq(path)
           end
         end
 
+
         # TODO(Kaoru): When we move to webmock 2 we won't need to do this
         # hackery
         #

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: sift
 version: !ruby/object:Gem::Version
-  version: 2.1.1
+  version: 2.2.1
 platform: ruby
 authors:
 - Fred Sadaghiani
@@ -10,7 +10,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-06-20 00:00:00.000000000 Z
+date: 2018-02-15 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rspec
@@ -158,7 +158,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: sift
-rubygems_version: 2.5.1
+rubygems_version: 2.4.6
 signing_key: 
 specification_version: 4
 summary: Sift Science Ruby API Gem