Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

String index out of bounds in javapns.notification.PushedNotification.toString line 251 #219

Closed
GoogleCodeExporter opened this issue Sep 25, 2015 · 3 comments
Closed

String index out of bounds in javapns.notification.PushedNotification.toString line 251 #219

GoogleCodeExporter opened this issue Sep 25, 2015 · 3 comments
Labels
auto-migrated Priority-Medium Type-Defect

Comments

@GoogleCodeExporter
Copy link 

String index out of range: 64. Stacktrace follows:
Message: String index out of range: 64
    Line | Method
->> 1907 | substring        in java.lang.String
- - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - 
|    251 | toString         in javapns.notification.PushedNotification

Original issue reported on code.google.com by david.ma...@einsight.co.za on 8 Oct 2014 at 3:33
@GoogleCodeExporter
Copy link
Author 

As far as I can tell, this would happen if you had an invalid token that is not 
exactly 64-bytes long.

Original comment by sype...@gmail.com on 11 Oct 2014 at 12:28

@GoogleCodeExporter
Copy link
Author 

Improvement in r402: PushedNotification.toString will now check if the token is 
valid before using it in the toString.  If it is not valid, toString will 
include an INVALID_TOKEN[...] indication.

Original comment by sype...@gmail.com on 13 Oct 2014 at 9:00

  • Changed state: Fixed

@GoogleCodeExporter
Copy link
Author 

The notification is sent successfully. I am using groovy.

Original comment by david.ma...@einsight.co.za on 14 Oct 2014 at 6:08

GoogleCodeExporter pushed a commit that referenced this issue Sep 25, 2015
Fixed PushedNotification.toString when token is invalid (issue #219)
db94e88
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
auto-migrated Priority-Medium Type-Defect
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@GoogleCodeExporter