From b263f0cb0b6cab1605db3dbf44a10cc42348fc63 Mon Sep 17 00:00:00 2001
From: Dmitry Volyntsev <xeioex@nginx.com>
Date: Tue, 11 Jul 2023 22:50:44 -0700
Subject: [PATCH] HTTP: fixed setting of Last-Modified header.

Previously, r.headersOut['Last-Modified'] setter did not update
r->headers_out.last_modified. As a result a client might get two
Last-Modified headers.
---
 nginx/ngx_http_js_module.c | 38 ++++++++++++++++++++++++++++++++++++--
 nginx/t/js_headers.t       | 18 +++++++++++++++---
 2 files changed, 51 insertions(+), 5 deletions(-)

diff --git a/nginx/ngx_http_js_module.c b/nginx/ngx_http_js_module.c
index 595a813d..02245575 100644
--- a/nginx/ngx_http_js_module.c
+++ b/nginx/ngx_http_js_module.c
@@ -128,6 +128,9 @@ static njs_int_t ngx_http_js_content_type122(njs_vm_t *vm,
 static njs_int_t ngx_http_js_date122(njs_vm_t *vm, ngx_http_request_t *r,
     ngx_list_t *headers, njs_str_t *name, njs_value_t *setval,
     njs_value_t *retval);
+static njs_int_t ngx_http_js_last_modified122(njs_vm_t *vm,
+    ngx_http_request_t *r, ngx_list_t *headers, njs_str_t *name,
+    njs_value_t *setval, njs_value_t *retval);
 static njs_int_t ngx_http_js_location122(njs_vm_t *vm, ngx_http_request_t *r,
     ngx_list_t *headers, njs_str_t *name, njs_value_t *setval,
     njs_value_t *retval);
@@ -228,6 +231,9 @@ static njs_int_t ngx_http_js_content_type(njs_vm_t *vm, ngx_http_request_t *r,
 static njs_int_t ngx_http_js_date(njs_vm_t *vm, ngx_http_request_t *r,
     unsigned flags, njs_str_t *name, njs_value_t *setval,
     njs_value_t *retval);
+static njs_int_t ngx_http_js_last_modified(njs_vm_t *vm, ngx_http_request_t *r,
+    unsigned flags, njs_str_t *name, njs_value_t *setval,
+    njs_value_t *retval);
 static njs_int_t ngx_http_js_location(njs_vm_t *vm, ngx_http_request_t *r,
     unsigned flags, njs_str_t *name, njs_value_t *setval,
     njs_value_t *retval);
@@ -1535,7 +1541,7 @@ ngx_http_js_ext_header_out(njs_vm_t *vm, njs_object_prop_t *prop,
         { njs_str("Date"), ngx_http_js_date122 },
         { njs_str("Etag"), ngx_http_js_header_single },
         { njs_str("Expires"), ngx_http_js_header_single },
-        { njs_str("Last-Modified"), ngx_http_js_header_single },
+        { njs_str("Last-Modified"), ngx_http_js_last_modified122 },
         { njs_str("Location"), ngx_http_js_location122 },
         { njs_str("Set-Cookie"), ngx_http_js_header_array },
         { njs_str("Retry-After"), ngx_http_js_header_single },
@@ -1548,7 +1554,7 @@ ngx_http_js_ext_header_out(njs_vm_t *vm, njs_object_prop_t *prop,
         { njs_str("Date"), 0, ngx_http_js_date },
         { njs_str("Etag"), NJS_HEADER_SINGLE, ngx_http_js_header_out },
         { njs_str("Expires"), NJS_HEADER_SINGLE, ngx_http_js_header_out },
-        { njs_str("Last-Modified"), NJS_HEADER_SINGLE, ngx_http_js_header_out },
+        { njs_str("Last-Modified"), 0, ngx_http_js_last_modified },
         { njs_str("Location"), 0, ngx_http_js_location },
         { njs_str("Set-Cookie"), NJS_HEADER_ARRAY, ngx_http_js_header_out },
         { njs_str("Retry-After"), NJS_HEADER_SINGLE, ngx_http_js_header_out },
@@ -1970,6 +1976,14 @@ ngx_http_js_date122(njs_vm_t *vm, ngx_http_request_t *r,
 }
 
 
+static njs_int_t
+ngx_http_js_last_modified122(njs_vm_t *vm, ngx_http_request_t *r,
+    ngx_list_t *headers, njs_str_t *v, njs_value_t *setval, njs_value_t *retval)
+{
+    return ngx_http_js_last_modified(vm, r, 0, v, setval, retval);
+}
+
+
 static njs_int_t
 ngx_http_js_location122(njs_vm_t *vm, ngx_http_request_t *r,
     ngx_list_t *headers, njs_str_t *v, njs_value_t *setval, njs_value_t *retval)
@@ -3996,6 +4010,26 @@ ngx_http_js_date(njs_vm_t *vm, ngx_http_request_t *r, unsigned flags,
 }
 
 
+static njs_int_t
+ngx_http_js_last_modified(njs_vm_t *vm, ngx_http_request_t *r, unsigned flags,
+    njs_str_t *v, njs_value_t *setval, njs_value_t *retval)
+{
+    njs_int_t         rc;
+    ngx_table_elt_t  *h;
+
+    rc = ngx_http_js_header_out_special(vm, r, v, setval, retval, &h);
+    if (rc == NJS_ERROR) {
+        return NJS_ERROR;
+    }
+
+    if (setval != NULL || retval == NULL) {
+        r->headers_out.last_modified = h;
+    }
+
+    return NJS_OK;
+}
+
+
 static njs_int_t
 ngx_http_js_location(njs_vm_t *vm, ngx_http_request_t *r, unsigned flags,
     njs_str_t *v, njs_value_t *setval, njs_value_t *retval)
diff --git a/nginx/t/js_headers.t b/nginx/t/js_headers.t
index 57a8a808..f2f96569 100644
--- a/nginx/t/js_headers.t
+++ b/nginx/t/js_headers.t
@@ -88,6 +88,10 @@ http {
             js_content test.date;
         }
 
+        location /last_modified {
+            js_content test.last_modified;
+        }
+
         location /location {
             js_content test.location;
         }
@@ -254,6 +258,11 @@ $t->write_file('test.js', <<EOF);
         r.return(200);
     }
 
+    function last_modified(r) {
+        r.headersOut['Last-Modified'] = 'Sun, 09 Sep 2001 01:46:40 GMT';
+        r.return(200);
+    }
+
     function location(r) {
         if (njs.version_number >= 0x000705) {
             var lc = r.headersOut['Location'];
@@ -446,13 +455,13 @@ $t->write_file('test.js', <<EOF);
                     hdr_in, raw_hdr_in, hdr_sorted_keys, foo_in, ifoo_in,
                     hdr_out, raw_hdr_out, hdr_out_array, hdr_out_single,
                     hdr_out_set_cookie, ihdr_out, hdr_out_special_set,
-                    copy_subrequest_hdrs, subrequest, date, location,
-                    location_sr};
+                    copy_subrequest_hdrs, subrequest, date, last_modified,
+                    location, location_sr};
 
 
 EOF
 
-$t->try_run('no njs')->plan(45);
+$t->try_run('no njs')->plan(46);
 
 ###############################################################################
 
@@ -605,6 +614,9 @@ local $TODO = 'not yet' unless has_version('0.8.1');
 
 like(http_get('/date'), qr/Date: Sun, 09 Sep 2001 01:46:40 GMT/,
 	'set date');
+like(http_get('/last_modified'),
+	qr/Last-Modified: Sun, 09 Sep 2001 01:46:40 GMT/,
+	'set Last-Modified');
 
 }
 
-- 
2.47.3