-
Notifications
You must be signed in to change notification settings - Fork 70
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
Update JWT dependency #99
Conversation
You want to get rid of the UTs altogether? |
Yes - if people need JWT support they can bring their own library, I don’t
want to support it or worry about vulnerabilities
…On Tue, Jul 16, 2024 at 07:25 Gagan Kalra ***@***.***> wrote:
Can we just remove the JWT functionality altogether?
You want to get rid of the UTs altogether?
—
Reply to this email directly, view it on GitHub
<#99 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AABZEIZXRTYMKYBMJXEH3CTZMUUN5AVCNFSM6AAAAABK6WXJRWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMZRGA2TONJYGA>
.
You are receiving this because you commented.Message ID:
***@***.***>
|
Cool, I'll remove the both UTs and update the PR. |
I have updated the code and even the mod file. |
Hey @kevinburke, |
This is an open source project and it's been one hour. I have a day job. |
This change has been merged |
Thank you 😅 |
This updates the deprecated jwt package with the new supported jwt package.