diff options
| author | Andrei Zeliankou <zelenkov@nginx.com> | 2024-01-25 12:49:47 +0000 |
|---|---|---|
| committer | andrey-zelenkov <xim.andrew@gmail.com> | 2024-01-26 15:17:00 +0000 |
| commit | 6452ca111c71188ab2813c763e6a0e86b48fbd56 (patch) | |
| tree | fc6517df5824a385a506c6830258801f7130834c /src/nodejs | |
| parent | ba56e50ee77d11c45f569ba8fbc95e6dadb323ef (diff) | |
| download | unit-6452ca111c71188ab2813c763e6a0e86b48fbd56.tar.gz unit-6452ca111c71188ab2813c763e6a0e86b48fbd56.tar.bz2 | |
Node.js: fixed "httpVersion" variable format
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
Diffstat (limited to 'src/nodejs')
| -rw-r--r-- | src/nodejs/unit-http/unit.cpp | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/src/nodejs/unit-http/unit.cpp b/src/nodejs/unit-http/unit.cpp index 7912d0ac..7d9395bb 100644 --- a/src/nodejs/unit-http/unit.cpp +++ b/src/nodejs/unit-http/unit.cpp @@ -581,6 +581,7 @@ Unit::get_server_object() void Unit::create_headers(nxt_unit_request_info_t *req, napi_value request) { + char *p; uint32_t i; napi_value headers, raw_headers; napi_status status; @@ -602,7 +603,12 @@ Unit::create_headers(nxt_unit_request_info_t *req, napi_value request) set_named_property(request, "headers", headers); set_named_property(request, "rawHeaders", raw_headers); - set_named_property(request, "httpVersion", r->version, r->version_length); + + // trim the "HTTP/" protocol prefix + p = (char *) nxt_unit_sptr_get(&r->version); + p += 5; + + set_named_property(request, "httpVersion", create_string_latin1(p, 3)); set_named_property(request, "method", r->method, r->method_length); set_named_property(request, "url", r->target, r->target_length); |