-
Notifications
You must be signed in to change notification settings - Fork 156
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.
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
ArrayIndexOutOfBoundsException issues #43
Comments
I also ran into this today for the first time. I definitely think that |
I'll check on these today. I'll keep you posted on that. |
I fixed it. Instead throwing an exception, I return an empty array |
I have found two other occurrences where I get ArrayIndexOutOfBoundsException from deep in the HashIds code. and
Like the original submitter above, these aren't valid hashes, but I'm running into these errors in my web app because someone COULD put these characters in as an ID in a URL accidentally or on purpose. My app is just passing in these characters directly into HashIds. I would expect something other than ArrayIndexOutOfBoundsExceptions. Please let me know if I should open a new bug. I figured this is directly related to this issue so I'd comment here. |
Hi @kmandeville , I'll look at this issue, but for the sake of organization, could you file another issue? Just to relate it in the commit. |
@kmandeville , never mind, I just created it(Issue #45) and fixed in master. Thanks! |
Thank you! |
Both:
and:
are causing exceptions in various places.
I realize that they are not valid hashes, but the exception should either be declared or handled.
Hope this helps.
The text was updated successfully, but these errors were encountered: