unit.git/src/nodejs, branch 1.35.0 Universal Web Application Server node.js: Fixed issue with duplicate headers in response 2025-05-28T16:27:04+00:00 skokalin sergey.kokalin@gmail.com 2025-05-28T13:47:42+00:00 9ae106730e28f89aed7987d40c9387057df4784e It fixes losing context in response in cases when there are 2 or more headers with the same name. The prev implementation used to use foreach function which uses local lexical environment and did not find this.headers_len locally, which causes crash of the http server module. It was replaced with a for loop in order to make access for this.headers_len variable and improve performance of calculation. Closes: https://github.com/nginx/unit/issues/1621 Signed-off-by: Andrew Clayton <a.clayton@nginx.com> 
It fixes losing context in response in cases when there are 2 or more
headers with the same name.  The prev implementation used to use foreach
function which uses local lexical environment and did not find
this.headers_len locally, which causes crash of the http server module.

It was replaced with a for loop in order to make access for this.headers_len
variable and improve performance of calculation.

Closes: https://github.com/nginx/unit/issues/1621
Signed-off-by: Andrew Clayton <a.clayton@nginx.com>
Add additional replace rules for node:* modules 2024-02-20T12:35:07+00:00 Gabor Javorszky g.javorszky@f5.com 2023-12-07T15:07:24+00:00 d24ae5a9a4b1140695e027087e72dcfdeb484ec0 In that particular issue the compiled nuxt files end up importing the http module as node:http rather than http only. This bypasses unit's custom loader implementation which only check for the http or unit-http modules, and their websocket counterparts. This changeset adds replace sources for both the node:http and node:websocket import signatures. Closes: https://github.com/nginx/unit/issues/1013 Reviewed-by: Andrew Clayton <a.clayton@nginx.com> 
In that particular issue the compiled nuxt files end up importing the
http module as node:http rather than http only. This bypasses unit's
custom loader implementation which only check for the http or unit-http
modules, and their websocket counterparts.

This changeset adds replace sources for both the node:http and
node:websocket import signatures.

Closes: https://github.com/nginx/unit/issues/1013
Reviewed-by: Andrew Clayton <a.clayton@nginx.com>
Node.js: Use console.warn instead of stderr.write 2024-02-19T14:56:57+00:00 Dan Callahan d.callahan@f5.com 2024-02-19T12:22:05+00:00 756feafd541160e97edfb6faf13e9942271629ce Functionally identical, but marginally more idiomatic. Refines: fbeb2065b180e2376088387ee150d3975dc08cd5 
Functionally identical, but marginally more idiomatic.

Refines: fbeb2065b180e2376088387ee150d3975dc08cd5
fix: Take options as well as requestListener (#1091) 2024-02-14T18:16:01+00:00 Gabor Javorszky g.javorszky@f5.com 2024-02-14T18:16:01+00:00 fbeb2065b180e2376088387ee150d3975dc08cd5 * Take options as well as requestListener Unit-http have not kept up with the signature of nodejs's http package development. Nodejs allows an optional `options` object to be passed to the `createServer` function, we didn't. This resulted in function signature errors when user code that did make use of the options arg tried to call unit's replaced function. This change changes the signature to be more in line with how nodejs does it discarding it and printing a message to stdout. * Add test file to start node application with options * Add changes to docs/changes.xml Closes: https://github.com/nginx/unit/issues/1043 
* Take options as well as requestListener

Unit-http have not kept up with the signature of nodejs's http package
development. Nodejs allows an optional `options` object to be passed to
the `createServer` function, we didn't. This resulted in function
signature errors when user code that did make use of the options arg
tried to call unit's replaced function.

This change changes the signature to be more in line with how nodejs
does it discarding it and printing a message to stdout.

* Add test file to start node application with options

* Add changes to docs/changes.xml

Closes: https://github.com/nginx/unit/issues/1043
 Node.js: fixed "httpVersion" variable format 2024-01-26T15:17:00+00:00 Andrei Zeliankou zelenkov@nginx.com 2024-01-25T12:49:47+00:00 6452ca111c71188ab2813c763e6a0e86b48fbd56 According to the Node.js documenation this variable should only include numbering scheme. Thanks to @dbit-xia. Closes: https://github.com/nginx/unit/issues/1085 
According to the Node.js documenation this variable
should only include numbering scheme.

Thanks to @dbit-xia.

Closes: https://github.com/nginx/unit/issues/1085
White space formatting fixes 2024-01-16T15:37:07+00:00 Andrei Zeliankou zelenkov@nginx.com 2024-01-16T15:37:07+00:00 a1e00b4e28d56365b4b5cc4aa44185c4b53f5c33 Closes: <https://github.com/nginx/unit/pull/1062> 
Closes: <https://github.com/nginx/unit/pull/1062>
Node.js: ServerResponse.flushHeaders() implemented. 2023-11-17T17:27:31+00:00 Andrei Zeliankou zelenkov@nginx.com 2023-11-17T17:27:31+00:00 1443d623d4b5d59e4463e025b4125be9a5aa3436 This closes #1006 issue on GitHub. Reviewed-by: Andrew Clayton <a.clayton@nginx.com> 
This closes #1006 issue on GitHub.

Reviewed-by: Andrew Clayton <a.clayton@nginx.com>
Node.js: provide reasonable default paths for macOS. 2023-09-26T23:14:21+00:00 Konstantin Pavlov thresh@nginx.com 2023-09-26T23:14:21+00:00 f1ce2a5ac2ba3266d525544445366fae538d7194 






Node.js: ServerRequest.destroy() implemented.
2023-09-26T11:49:39+00:00

Andrei Zeliankou
zelenkov@nginx.com

2023-09-26T11:49:39+00:00

2d0e502d2a69ad490e0633c59636e1115afa983d

This closes #871 issue on GitHub.





This closes #871 issue on GitHub.







Node.js: response body chunk can now be a Uint8Array.
2023-09-26T11:49:39+00:00

Andrei Zeliankou
zelenkov@nginx.com

2023-09-26T11:49:39+00:00

e0c26757740fa7974af6e6592e35b5f2e00339fe

Starting from Node.js 15.0.0 the chunk parameter of the response.write()
can be a Uint8Array.

This closes #870 issue on GitHub.





Starting from Node.js 15.0.0 the chunk parameter of the response.write()
can be a Uint8Array.

This closes #870 issue on GitHub.