-
Notifications
You must be signed in to change notification settings - Fork 386
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
Edge getSession adds large joi-browser package #1581
Comments
Thanks for raising this @nordicgit70
Will take a look at upgrading to @hapi/[email protected] |
Thanks, it would great if the
|
Closed by mistake. |
It looks like When you say I also think we should not migrate to However, we might want to consider moving away from |
Hi, my main issue is with the impact on package size. If that could be resolved would be great. I do not have an opinion on the usage of Joi. |
Hi, is it planned to resolve this? |
There is currently nothing planned to move away from joi, no. |
How can I reduce the package size? The package |
Any current plans to look at this? A project I work on has recently upgraded from 1.9.2 to 3.5.0 and our bundle size has shot up. We're seeing this in our client-side bundle analyzer report. |
Checklist
Description
The Nextjs middleware package contains the joi-browser package when getSession is imported from
@auth0/nextjs-auth0/edge
. Joi-browser is deprecated and should not be part of the server bundle.It has a large impact on the Edge middleware package size.
Reproduction
Additional context
Reproduced with: https://github.com/nordicgit70/nextjs-app/tree/auth0-joi.
nextjs-auth0 version
3.3.0
Next.js version
14.0.2
Node.js version
18.18.2
The text was updated successfully, but these errors were encountered: