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

Explicitly import sign/verify methods #84

Closed
wants to merge 1 commit into from

Conversation

danawoodman
Copy link

Current examples don't work as is, instead import the methods that are called directly

Current examples don't work as is, instead import the methods that are called directly
@tsndr
Copy link
Owner

tsndr commented Oct 27, 2024

Thank you for letting me know, and while your change is correct, I'd rather go with the jwt.sign() syntax, for readablitly reasons. Really appreciate your help though :)

@tsndr tsndr closed this Oct 27, 2024
@danawoodman
Copy link
Author

no problem

fwiw won't that mean larger bundles if you're not using both methods (tree shaking)?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants