-
Notifications
You must be signed in to change notification settings - Fork 30
node:https.request
support
#323
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
Comments
Any update on this? |
I'm also interested in this. |
confirmed that it is not. Could you please expand on the use case? |
I'm quite new to Cloudflare workers and all so there might be a clear solution for my issue, however I haven't been able to troubleshoot myself to something. I'm building a Worker API that uses Firebase App Check for Auth. I'm getting the uenv not implemented error when using the App Check library. That's why I'm in need of it. Let me know if I can help by elaborating further. @vicb |
As you probably already knows Workers use the workerd Runtime that does not implement all of the Node.js APIs. However we are looking at supporting more and more of the Node libs. There are a few things that can help here:
So what you can do here to help is:
And we'll get that working. Thanks! |
Ideally, i would love to have a polyfill based on wintercg web sockets proposal which workerd currently support (startTLS) — and it should enable a universal / spec compatible solution in the future. Native workerd support would be cool but i think it is still viable to be discussed for unenv support :) —- I think we should also investigate firebase SDK issue separately, i guess it can be solved by making sure it leverages fetch for http calls (unless they really need TCP connect) |
I'm partially migrating from AWS to CF, at least for the resources it make sense to migrate from. And in doing this hybrid architecture, I'm trying to pushing some messages from CF Worker, to SQS (directly invoking the AWS API) (using the aws-lite library https://aws-lite.org/services/sqs) In doing so, I get the https.request not implemented error. Workaround 1: Workaround 2: |
I can definitely recommend aws4fetch here:
|
Relevant workerd discussion: cloudflare/workerd#3050 |
I followed your suggestion, it worked perfectly thank you. |
Edit - looks like Axios did add a fetch adapter recently - so that will probably alleviate these issues quite a bit. https://www.npmjs.com/package/axios#-fetch-adapter |
I guess cloudflare/workerd would be a good repo for that issue / discussion - there are probably existing ones |
Any update or workaround to this? This breaks many packages and makes it unusable on cloudflare / nuxthub Specifically I was depending on a package for twitter-api-v2 |
Major problem, tons of stuff broken, please fix |
For cloudflare users, this feature should be implemented in workerd (upstream tracker: cloudflare/workerd#3820). You can check the native support coverage from here. We might make a polyfill based on the new |
Originally posted by @jasnell in cloudflare/workerd#3050 (comment) |
I discovered this issue when migrating from pages to a worker, and the new vite/cloudflare plugin. |
Yes, I really need this feature |
Would love this feature |
Uh oh!
There was an error while loading. Please reload this page.
Describe the feature
Hi, just to make sure, is https.request implemented for cloudflare?
If not, do you think there an ETA for it? I'd love to help if possible.
Additional information
The text was updated successfully, but these errors were encountered: