nginx / unit

NGINX Unit - universal web app server - a lightweight and versatile open source server that simplifies the application stack by natively executing application code across eight different programming language runtimes.
https://unit.nginx.org
Apache License 2.0
5.27k stars 324 forks source link

npm: unit-http: should we replace the https package as well? #1025

Open javorszky opened 7 months ago

javorszky commented 7 months ago

Issue #1013 revealed a bug where imports with signature node:http weren't replaced.

The current npm package unit uses as a custom loader looks for the http and websocket packages. Node applications do have the option to import the https package as well, which also has a .listen() method.

Do we need to replace the built in node https package with a unit specific https package, or will the underlying http package take care of the unit specific functionality as well?