-
Notifications
You must be signed in to change notification settings - Fork 22
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
Error when using with Express v5 #47
Comments
Seems like the WDYT? |
If possible, I'd love to see a major release supporting Express v5. It's not going to be widely adopted for a long time, but I've seen people (myself included) tinker with it on pet projects. |
Totally understand. I don't have much bandwidth though. I've asked people that opened issues/PRs if they'd like to become maintainers - is that something you might be interested in? |
Probably not, sorry. |
@evanshortiss I tinkered with it and the fix was quiet easy. I copied the approach Celebrate implemented years ago. Another approach would be to use |
When using with Express v5 you get an error like
Which leads here
I assume it happens due to sanitization. Anything that can be done about it? Thanks.
The text was updated successfully, but these errors were encountered: