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.

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

The exception try(){}catch{} suggession #436

Closed
ballchencz opened this issue Aug 21, 2020 · 2 comments
Closed

The exception try(){}catch{} suggession #436

ballchencz opened this issue Aug 21, 2020 · 2 comments
Assignees
Labels
waiting for customer This issue is waiting for a response from the issue or PR author

Comments

@ballchencz
Copy link

When I removed several bits of a signed token,My system throws an "IllegalArgumentException", but I hope it throws "JWTDecodeException", When I checked the source code, I found that the "JWTDecoder" constructor only caught a null pointer exception and threw "JWTDecoderException". Therefore, I hope to catch the runtime exception here, so that I can handle the token parsing failure in the exception handling instance of my system

@lbalmaceda lbalmaceda added the needs investigation An issue that has more questions to answer or otherwise needs work to fully understand the issue label Sep 8, 2020
@lbalmaceda lbalmaceda self-assigned this Sep 8, 2020
@jimmyjames
Copy link
Contributor

Hi @ballchencz,

We do have a fix for this issue that has been merged, but not yet released. We're planning to do the release this week.

Here is the change, if you'd like to try it out and verify that it addresses the issue, that would be great. Thanks!

@jimmyjames jimmyjames added waiting for customer This issue is waiting for a response from the issue or PR author and removed needs investigation An issue that has more questions to answer or otherwise needs work to fully understand the issue labels Sep 14, 2020
@lbalmaceda
Copy link
Contributor

Fixed in 3.11.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
waiting for customer This issue is waiting for a response from the issue or PR author
Projects
None yet
Development

No branches or pull requests

3 participants