pusher 1.2.0.rc3 → 1.2.0.rc4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: fc20df33b1edef141b85a41f44b5d48bd1a622ac
-  data.tar.gz: 1f8bb9ade6ae7531c9f780eb36bea7f5421da2de
+  metadata.gz: 8b9efdaf822f5abafd54a89499c0475fd75dfde4
+  data.tar.gz: 698e043ffa4116cde7f9951631c865a4c191fff6
 SHA512:
-  metadata.gz: 61130fb7ea38cea25bb9a7639c209bff888e722d2b2b488c5510b414a32b867cf91eb8be670d76767dd2a3b2d3f57a094a856a3011f86184b84943a18e99d3ec
-  data.tar.gz: 1a68e7135cdaf15f34864f836b20c2130a041512f2651eacfc76794d87cacf12421c9442456a79a7497ad0a0a7f6bd5420a3657fc5dbf43074803d4fab60c4e8
+  metadata.gz: 8aea046a40212d7b8bcc6467bea3e9ec142dce1917259e86dedadc3919e8cd396d4809bd18626d7ca392a0aea2878156802178d0e9af9a1ef5c9a190161b5d7b
+  data.tar.gz: fb590812921456d724f116482ca55195ef19af01eff2b6481a1030afe65064de9798f1c5038cb8f3b09dfc2721af8f3368e0446d50aa204266591786a8d787b2

data/lib/pusher/native_notification/client.rb

@@ -5,9 +5,6 @@ module Pusher
 
       API_PREFIX = "server_api"
       API_VERSION = "v1"
-      GCM_TTL = 241920
-      RESTRICTED_GCM_PAYLOAD_KEYS = [:to, :registration_ids]
-      WEBHOOK_LEVELS = ["DEBUG", "INFO"]
 
       def initialize(app_id, host, scheme, pusher_client)
         @app_id = app_id
@@ -45,14 +42,8 @@ module Pusher
       # @return [String]
       def payload(interests, data)
         interests = Array(interests).map(&:to_s)
-
         raise Pusher::Error, "Too many interests provided" if interests.length > 1
-
-        data = deep_symbolize_keys!(data)
-        validate_payload(data)
-
-        data.merge!(interests: interests)
-
+        data = deep_symbolize_keys!(data).merge(interests: interests)
         MultiJson.encode(data)
       end
 
@@ -60,54 +51,6 @@ module Pusher
         URI.parse("#{@scheme}://#{@host}/#{API_PREFIX}/#{API_VERSION}/apps/#{@app_id}#{path}")
       end
 
-      # Validate payload
-      # `time_to_live` -> value b/w 0 and 241920
-      # If the `notification` key is provided, ensure
-      # that there is an accompanying `title` and `icon`
-      # field
-      def validate_payload(payload)
-        unless (payload.has_key?(:apns) || payload.has_key?(:gcm))
-          raise Pusher::Error, "GCM or APNS data must be provided"
-        end
-
-        if (gcm_payload = payload[:gcm])
-          # Restricted keys
-          RESTRICTED_GCM_PAYLOAD_KEYS.each { |k| gcm_payload.delete(k) }
-          if (ttl = gcm_payload[:time_to_live])
-
-            if ttl.to_i < 0 || ttl.to_i > GCM_TTL
-              raise Pusher::Error, "Time to live must be between 0 and 241920 (4 weeks)"
-            end
-          end
-
-          # If the notification key is provided
-          # validate the `icon` and `title`keys
-          if (notification = gcm_payload[:notification])
-            notification_title, notification_icon = notification.values_at(:title, :icon)
-
-            if (!notification_title || notification_title.empty?)
-              raise Pusher::Error, "Notification title is a required field"
-            end
-
-            if (!notification_icon || notification_icon.empty?)
-              raise Pusher::Error, "Notification icon is a required field"
-            end
-          end
-        end
-
-        if (webhook_url = payload[:webhook_url])
-          raise Pusher::Error, "Webhook url is invalid" unless webhook_url =~ /\A#{URI::regexp(['http', 'https'])}\z/
-        end
-
-        if (webhook_level = payload[:webhook_level])
-          raise Pusher::Error, "Webhook level cannot be used without a webhook url" if !payload.has_key?(:webhook_url)
-
-          unless WEBHOOK_LEVELS.include?(webhook_level.upcase)
-            raise Pusher::Error, "Webhook level must either be INFO or DEBUG"
-          end
-        end
-      end
-
       # Symbolize all keys in the hash recursively
       def deep_symbolize_keys!(hash)
         hash.keys.each do |k|

data/lib/pusher/request.rb

@@ -85,7 +85,7 @@ module Pusher
       when 200
         return symbolize_first_level(MultiJson.decode(body))
       when 202
-        return true
+        return body.empty? ? true : symbolize_first_level(MultiJson.decode(body))
       when 400
         raise Error, "Bad request: #{body}"
       when 401

data/lib/pusher/version.rb

@@ -1,3 +1,3 @@
 module Pusher
-  VERSION = '1.2.0.rc3'
+  VERSION = '1.2.0.rc4'
 end

data/spec/client_spec.rb

@@ -550,10 +550,6 @@ describe Pusher do
         expect(@client.notification_host).to eq(@client.notification_client.host)
       end
 
-      it "should raise an error if the gcm or apns key isn't provided in the payload" do
-        expect { @client.notify(["test"], { foo: "bar" }) }.to raise_error(Pusher::Error)
-      end
-
       it "should raise an error if more than one interest is provided" do
         payload = {
           gcm: {
@@ -567,70 +563,6 @@ describe Pusher do
         expect { @client.notify(["test1", "test2"], payload) }.to raise_error(Pusher::Error)
       end
 
-      it "should raise an error if the notification hash is missing the title field" do
-        payload = {
-          gcm: {
-            notification: {
-              icon: "someicon"
-            }
-          }
-        }
-
-        expect{ @client.notify(["test"], payload) }.to raise_error(Pusher::Error)
-      end
-
-      it "should raise an error if the notification title is empty" do
-        payload = {
-          gcm: {
-            notification: {
-              title: "",
-              icon: "myicon"
-            }
-          }
-        }
-
-        expect { @client.notify(["test"], payload) }.to raise_error(Pusher::Error)
-      end
-
-      it "should raise an error if the notification hash is missing the icon field" do
-        payload = {
-          gcm: {
-            notification: {
-              title: "sometitle"
-            }
-          }
-        }
-
-        expect{ @client.notify(["test"], payload) }.to raise_error(Pusher::Error)
-      end
-
-      it "should raise an error if the notification icon is empty" do
-        payload = {
-          gcm: {
-            notification: {
-              title: "title",
-              icon: ""
-            }
-          }
-        }
-
-        expect { @client.notify(["test"], payload) }.to raise_error(Pusher::Error)
-      end
-
-      it "should raise an error if the ttl field is provided and has an illegal value" do
-        payload = {
-          gcm: {
-            time_to_live: 98091283,
-            notification: {
-              title: "title",
-              icon: "icon",
-            }
-          }
-        }
-
-        expect{ @client.notify(["test"], payload) }.to raise_error(Pusher::Error)
-      end
-
       it "should send a request to the notifications endpoint" do
         notification_host_regexp = %r{nativepush-cluster1.pusher.com}
         payload = {
@@ -653,76 +585,8 @@ describe Pusher do
           :body => MultiJson.encode({ :foo => "bar" })
         })
 
-        @client.notify(["test"], payload)
-      end
-
-      it "should delete restricted gcm keys before sending a notification" do
-        notification_host_regexp = %r{nativepush-cluster1.pusher.com}
-        payload = {
-          interests: ["test"],
-          gcm: {
-            notification: {
-              title: "Hello",
-              icon: "icon",
-            }
-          }
-        }
-
-        stub_request(
-          :post,
-          notification_host_regexp,
-        ).with(
-          body: MultiJson.encode(payload)
-        ).to_return({
-          :status => 200,
-          :body => MultiJson.encode({ :foo => "bar" })
-        })
-
-        payload[:gcm].merge!(to: "blah", registration_ids: ["reg1", "reg2"])
-        @client.notify(["test"], payload)
-      end
-
-      it "should raise an error for an invalid webhook url field" do
-        payload = {
-          gcm: {
-            notification: {
-              title: "Hello",
-              icon: "icon"
-            }
-          },
-          webhook_url: "totallyinvalid"
-        }
-
-        expect { @client.notify(["test"], payload) }.to raise_error(Pusher::Error)
-      end
-
-      it "should raise an error if the webhook level is not supported" do
-        payload = {
-          gcm: {
-            notification: {
-              title: "Hello",
-              icon: "icon"
-            }
-          },
-          webhook_url: "http://test.com/wh",
-          webhook_level: "meh"
-        }
-
-        expect { @client.notify(["test"], payload) }.to raise_error(Pusher::Error)
-      end
-
-      it "should raise an error if the webhook level is used without the webhook url" do
-        payload = {
-          gcm: {
-            notification: {
-              title: "Hello",
-              icon: "icon"
-            }
-          },
-          webhook_level: "meh"
-        }
-
-        expect { @client.notify(["test"], payload) }.to raise_error(Pusher::Error)
+        res = @client.notify(["test"], payload)
+        expect(res).to eq({foo: "bar"})
       end
     end
   end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: pusher
 version: !ruby/object:Gem::Version
-  version: 1.2.0.rc3
+  version: 1.2.0.rc4
 platform: ruby
 authors:
 - Pusher
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-08-02 00:00:00.000000000 Z
+date: 2016-08-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: multi_json
@@ -186,7 +186,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: 1.3.1
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.5.1
+rubygems_version: 2.4.2
 signing_key: 
 specification_version: 4
 summary: Pusher API client