From 3152e35ee0f7df305763af6b693e0d5e1180e4a4 Mon Sep 17 00:00:00 2001 From: Dmitry Volyntsev Date: Tue, 7 Jun 2022 21:30:57 -0700 Subject: [PATCH] HTTP: returing undefined for Content-Type when the header is absent. This unifies empty response value type for r.headersOut. --- nginx/ngx_http_js_module.c | 12 ++++++++++++ 1 file changed, 12 insertions(+) diff --git a/nginx/ngx_http_js_module.c b/nginx/ngx_http_js_module.c index cc3ece3d..0c0d9114 100644 --- a/nginx/ngx_http_js_module.c +++ b/nginx/ngx_http_js_module.c @@ -1978,6 +1978,12 @@ ngx_http_js_content_type(njs_vm_t *vm, ngx_http_request_t *r, if (retval != NULL && setval == NULL) { hdr = &r->headers_out.content_type; + + if (hdr->len == 0) { + njs_value_undefined_set(retval); + return NJS_OK; + } + return njs_vm_value_string_set(vm, retval, hdr->data, hdr->len); } @@ -3764,6 +3770,12 @@ ngx_http_js_content_type(njs_vm_t *vm, ngx_http_request_t *r, if (retval != NULL && setval == NULL) { hdr = &r->headers_out.content_type; + + if (hdr->len == 0) { + njs_value_undefined_set(retval); + return NJS_OK; + } + return njs_vm_value_string_set(vm, retval, hdr->data, hdr->len); } -- 2.47.3