-
Notifications
You must be signed in to change notification settings - Fork 210
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
Multiple nodejs_compat packages missing (crypto
, path
, etc)
#756
Comments
Hey! 👋 Yep, we'd be open to a PR, but as you point out this package is pretty-much deprecated. What are you using |
I'm creating a Knex client in Node that connects to our local database and then passing it to Miniflare:
However I think this would be fixed once Hyperdrive support (with local databases support) is added to Miniflare 3. |
Cool! Miniflare 3 does support Hyperdrive now, looks like this wasn't added to the API documentation though. const mf = new Miniflare({
hyperdrives: {
DB: "postgresql://user:password@localhost:5432/database"
}
}); |
Hey again! 👋 I'm going to close this issue since it looks like you've been able to switch to Miniflare 3, and we're getting very close to deprecating Miniflare 2 now that cloudflare/workers-sdk#4795 is up. 👍 |
When using nodejs_compat in Miniflare 2, only the following modules are available: assert, async_hooks, buffer, events, util
These ones are missing: crypto, diagnostics_channel, path, process, stream (and stream/promises, stream/consumers), string_decoder.
I'm willing to add these, but as this package is almost deprecated, would you be open for a PR? As we're using
globals
we can't migrate to Miniflare 3 yet.The text was updated successfully, but these errors were encountered: