-
Notifications
You must be signed in to change notification settings - Fork 80
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
punycode is deprecated in Node v21 #315
Comments
There is already a pull request from 2022 fixing this issue. |
3ddec00 |
This is already fixed. We are just waiting for a new release. Patience is a virtue. |
Can you clarify why you suggest just waiting? Are you a maintainer and you are planning a release? It's 2 months ago since @lupomontero added a series of commits into the If you are not a maintainer, then perhaps @lupomontero could say if/when a new release is planned? |
Thank you for maintaining this package! I understand that it is a lot of work. It would be appreciated with a release that includes thix fix. |
Hi there, many apologies for taking so long to address this... I have just released a new version that should include a fix for this issue: https://github.com/lupomontero/psl/releases/tag/v1.10.0 |
Using punycode is deprecated in Node v21. See documentation https://nodejs.org/api/punycode.html. Maybe try replacing with NPM Package punycode
punycode is used here:
psl/index.js
Line 5 in 797ccb4
Fix:
npm i punycode
require("punycode")
torequire("punycode/")
AnWeber/httpyac#577
The text was updated successfully, but these errors were encountered: