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

Wrap errors #41

Closed
tanner0101 opened this issue Jul 31, 2020 · 0 comments · Fixed by #107
Closed

Wrap errors #41

tanner0101 opened this issue Jul 31, 2020 · 0 comments · Fixed by #107
Assignees
@ptoffy
Labels
enhancement New feature or request
Projects
Vapor 4
Milestone
v5

Comments

@tanner0101
Copy link
Member

Check for errors being thrown by underlying calls that could benefit from being wrapped with more context.

For example, the signers.use(jwksJSON:) will currently throw a decoding error that lacks context about what was being decoded or why.
@tanner0101 tanner0101 added the enhancement New feature or request label Jul 31, 2020
@tanner0101 tanner0101 added this to To Do in Vapor 4 via automation Jul 31, 2020
@0xTim 0xTim added this to the v5 milestone Oct 9, 2023
@ptoffy ptoffy mentioned this issue Oct 28, 2023
Merged
16 tasks
@ptoffy ptoffy linked a pull request Oct 28, 2023 that will close this issue
V5 #107
Merged
16 tasks
@ptoffy ptoffy closed this as completed Feb 19, 2024
Vapor 4 automation moved this from To Do to Done Feb 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ptoffy ptoffy

Labels
enhancement New feature or request
Projects
Vapor 4
  
Done
Milestone
v5
Development

Successfully merging a pull request may close this issue.

V5 vapor/jwt-kit
3 participants
@tanner0101 @0xTim @ptoffy