Merge pull request #198 from pusher/increase-size-check

increase the size of the local payload check

Author: fbenevides

pusher/pusher_client.py

@@ -91,7 +91,7 @@ def trigger(self, channels, event_name, data, socket_id=None):
             raise ValueError("event_name too long")
 
         data = data_to_string(data, self._json_encoder)
-        if sys.getsizeof(data) > 10240:
+        if sys.getsizeof(data) > 30720:
             raise ValueError("Too much data")
 
         channels = list(map(validate_channel, channels))

pusher_tests/test_pusher_client.py

@@ -207,23 +207,23 @@ def test_trigger_with_private_encrypted_channel_string_fail_case_no_encryption_m
     def test_trigger_too_much_data(self):
         pc = PusherClient(app_id=u'4', key=u'key', secret=u'secret', ssl=True)
 
-        self.assertRaises(ValueError, lambda: pc.trigger(u'private-tst', u'some_event', u'a' * 10241))
+        self.assertRaises(ValueError, lambda: pc.trigger(u'private-tst', u'some_event', u'a' * 30721))
 
     def test_trigger_batch_too_much_data(self):
         pc = PusherClient(app_id=u'4', key=u'key', secret=u'secret', ssl=True)
 
         self.assertRaises(ValueError, lambda: pc.trigger_batch(
-            [{u'channel': u'private-tst', u'name': u'some_event', u'data': u'a' * 10241}]))
+            [{u'channel': u'private-tst', u'name': u'some_event', u'data': u'a' * 30721}]))
 
-    def test_trigger_str_shorter_than_10240_but_more_than_10kb_raising(self):
+    def test_trigger_str_shorter_than_30720_but_more_than_3kb_raising(self):
         pc = PusherClient(app_id=u'4', key=u'key', secret=u'secret', ssl=True)
 
-        self.assertRaises(ValueError, lambda: pc.trigger.make_request(u'private-tst', u'some_event', u'你' * 10000))
+        self.assertRaises(ValueError, lambda: pc.trigger.make_request(u'private-tst', u'some_event', u'你' * 30000))
 
-    def test_trigger_batch_str_shorter_than_10240_but_more_than_10kb_raising(self):
+    def test_trigger_batch_str_shorter_than_30720_but_more_than_30kb_raising(self):
         pc = PusherClient(app_id=u'4', key=u'key', secret=u'secret', ssl=True)
 
-        self.assertRaises(ValueError, lambda: pc.trigger_batch.make_request([{u'channel': u'private-tst', u'name': u'some_event', u'data': u'你' * 10000}]))
+        self.assertRaises(ValueError, lambda: pc.trigger_batch.make_request([{u'channel': u'private-tst', u'name': u'some_event', u'data': u'你' * 30000}]))
 
     def test_trigger_with_public_channel_with_encryption_master_key_specified_success(self):
         json_dumped = u'{"message": "something"}'