technocloud-public/nginx-unit
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

Node.js: using low-case header names as key in req.headers.

Browse Source 
Node.js modules (body-parser, row-body) search low-cased names
('content-length', 'content-type' etc.) to properly assemble request body.

This closes #246 issue on GitHub.
This commit is contained in:
Max Romanov
2019-04-17 19:15:41 +03:00
parent 704529e4b1
commit c6e96647a1
1 changed files with 17 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

21
src/nodejs/unit-http/unit.cpp
View File

@@ -376,17 +376,30 @@ Unit::create_headers(nxt_unit_request_info_t *req, napi_value request)
} }
inline char
lowcase(char c)
{
return (c >= 'A' && c <= 'Z') ? (c | 0x20) : c;
}
inline void inline void
Unit::append_header(nxt_unit_field_t *f, napi_value headers, Unit::append_header(nxt_unit_field_t *f, napi_value headers,
napi_value raw_headers, uint32_t idx) napi_value raw_headers, uint32_t idx)
{ {
const char *name; char *name;
napi_value str, vstr; uint8_t i;
napi_value str, vstr;
name = (const char *) nxt_unit_sptr_get(&f->name); name = (char *) nxt_unit_sptr_get(&f->name);
str = create_string_latin1(name, f->name_length);
for (i = 0; i < f->name_length; i++) {
name[i] = lowcase(name[i]);
}
vstr = set_named_property(headers, name, f->value, f->value_length); vstr = set_named_property(headers, name, f->value, f->value_length);
str = create_string_latin1(name, f->name_length);
set_element(raw_headers, idx * 2, str); set_element(raw_headers, idx * 2, str);
set_element(raw_headers, idx * 2 + 1, vstr); set_element(raw_headers, idx * 2 + 1, vstr);