-
Notifications
You must be signed in to change notification settings - Fork 5.4k
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
Add Node.js native module polyfills to std/node #3403
Comments
I'm working on the 'os' polyfill, though as discussed in #3802 there's a lot of functionality Deno is missing. |
@kevinkassimo - That's the 'os' polyfill taken as far as I can, so please mark this as partly completed. Now moving onto 'events'. |
@cknight Thanks so much! Marked |
@kevinkassimo - That's the 'events' polyfill complete now (#3944, #3960, #4016). I'm going to attempt to pick up fs next, though will progress this in small chunks due to its large size. |
Nice one @dubiousjim. I've not yet looked at detail into what Deno can offer, but at a glance it looks like your pending PRs will get us a lot closer to a full polyfill. |
Is there consensus on if or how much of node's crypto module to reimplement? It's got a pretty big API surface and openssl-isms leak through in many places but it should be possible to make basic functionality (cipher/decipher/hash/hmac/sign/verify) work for the most common ciphers and hashes using RustCrypto. |
currently working on os stuff impl. |
@kevinkassimo #4370 took care of |
@kevinkassimo i'm diving back into Deno after not working on it for a while (and I've only really contributed to the TypeScript portion). Which of these would be a good first issues? |
@acconrad Good for starters would be |
@Soremwar cool! |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed in 7 days if no further activity occurs. Thank you for your contributions. |
assert? |
Assert will be there as well @lygstate denoland/std#705 |
Good to see |
|
Many Node.js native module polyfills are needed in
std/node
, to support loading npm packages (such that we are able to leverage some of the tools while Deno specific versions are not available yet).Note some of the modules might not yet have a corresponding Deno API available. In that case, adding a new native binding for Deno or adding a new module in
std
modules might be reasonable.#3382 is an example how to inject a polyfill for
require()
.Some of the polyfill tasks can be Good First Issue.
buffer
child_process
cluster
(maybe low priority?)crypto
dns
events
fs
(partially, high priority)http
/https
/http2
/dgram
/net
/tls
(maybe low priority?)modules
os
(more bindings needed, #3802)path
querystring
Add node querystring polyfill #4370readline
repl
stream
tty
url
util
(partially)The text was updated successfully, but these errors were encountered: