search

TimmyYu / javapns

Automatically exported from code.google.com/p/javapns
0 stars 0 forks source link

Javapns library ignoring JSONException #207

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
What steps will reproduce the problem?
1.Pass a valid json to the PushNotification payload but with the "aps" key
2. All the JSONExceptions are ignored by this class by simply printing 
e.stackTrace.
3. This will result in validation success( actuall an error ) in my app since 
JSONException is not thrown outsied
4. Please don't consume JSONException . Please throw it back.

What is the expected output? What do you see instead?
Don't consume JSONException or any RuntimeException which occurs due to api 
input. Throw it back

What version of the product are you using? On what operating system?
Mac Os 10.8

Please provide any additional information below.

Original issue reported on code.google.com by arun.geo...@gmail.com on 21 Nov 2013 at 9:07

GoogleCodeExporter commented 9 years ago 
Fixed in r389!

Original comment by sype...@gmail.com on 30 Sep 2014 at 3:22