oven-sh / bun

Incredibly fast JavaScript runtime, bundler, test runner, and package manager – all in one
https://bun.sh
Other
73.48k stars 2.71k forks source link

Implement `node:https` #1990

Open gaurishhs opened 1 year ago

gaurishhs commented 1 year ago

What is the problem this feature would solve?

https module. Currently the https module uses http.request under the hood. http module succeeds in making requests to a url with https protocol which it shouldn't https://github.com/oven-sh/bun/blob/main/src/js/node/http.ts#L1401 instead it should throw an invalid protocol error.

What is the feature you are proposing to solve the problem?

A https module.

What alternatives have you considered?

No response

birkskyum commented 1 year ago

It seems like the Node API compat docs here wrongly claim that node:https is fully implemented.