chore(deps): bump ngx_wasm_module to 91d447ffd0e9bb08f11cc69d1aa9128e…

…c36b4526 (#12941) (#8959) Co-authored-by: Hisham Muhammad <[email protected]> Co-authored-by: Rob Serafini <[email protected]>
Kong · Apr 29, 2024 · 8418af1 · 8418af1
1 parent 8c3a358
commit 8418af1
Show file tree

Hide file tree

Showing 5 changed files with 10 additions and 8 deletions.
diff --git a/.requirements b/.requirements
@@ -20,7 +20,7 @@ SNAPPY=23b3286820105438c5dbb9bc22f1bb85c5812c8a # 1.2.0
 # Please leave this untouched and use KONG_ADMIN in EE version instead
 KONG_MANAGER=DO_NOT_USE
 
-NGX_WASM_MODULE=3bd94e61c55415ccfb0f304fa51143a7d630d6ae
+NGX_WASM_MODULE=91d447ffd0e9bb08f11cc69d1aa9128ec36b4526
 WASMER=3.1.1
 WASMTIME=19.0.0
 V8=12.0.267.17

diff --git a/changelog/unreleased/kong/bump-ngx-wasm-module.yml b/changelog/unreleased/kong/bump-ngx-wasm-module.yml
@@ -1,2 +1,2 @@
-message: "Bump `ngx_wasm_module` to `3bd94e61c55415ccfb0f304fa51143a7d630d6ae`"
+message: "Bump `ngx_wasm_module` to `91d447ffd0e9bb08f11cc69d1aa9128ec36b4526`"
 type: dependency
diff --git a/spec/02-integration/20-wasm/04-proxy-wasm_spec.lua b/spec/02-integration/20-wasm/04-proxy-wasm_spec.lua
@@ -288,7 +288,7 @@ describe("proxy-wasm filters (#wasm) (#" .. strategy .. ")", function()
       assert.res_status(200, res)
       assert.response(res).has.no.header("x-via")
       assert.logfile().has.line([[testing in "Log"]])
-      assert.logfile().has.line("cannot add response header: headers already sent")
+      assert.logfile().has.line("can only set response headers before \"on_response_body\"")
     end)
 
     pending("throw a trap", function()

diff --git a/spec/fixtures/proxy_wasm_filters/tests/src/routines.rs b/spec/fixtures/proxy_wasm_filters/tests/src/routines.rs
@@ -18,7 +18,6 @@ pub(crate) fn add_response_header(ctx: &mut TestHttp) {
         let (name, value) = header.split_once('=').unwrap();
 
         ctx.add_http_response_header(name, value);
-        ctx.set_http_request_header(HEADER_NAME, None)
     }
 
     const CONFIG_HEADER_NAME: &str = "X-PW-Resp-Header-From-Config";

diff --git a/spec/fixtures/proxy_wasm_filters/tests/src/test_http.rs b/spec/fixtures/proxy_wasm_filters/tests/src/test_http.rs
@@ -98,11 +98,14 @@ impl TestHttp {
         if cur_phase == on_phase {
             info!("[proxy-wasm] testing in \"{:?}\"", on_phase);
 
-            self.set_http_request_header(INPUT_HEADER_NAME, None);
-            self.set_http_request_header(TEST_HEADER_NAME, None);
-            self.set_http_request_header(PHASE_HEADER_NAME, None);
+            if cur_phase == TestPhase::RequestHeaders || cur_phase == TestPhase::RequestBody {
+                self.set_http_request_header(INPUT_HEADER_NAME, None);
+                self.set_http_request_header(TEST_HEADER_NAME, None);
+                self.set_http_request_header(PHASE_HEADER_NAME, None);
+
+                add_request_header(self);
+            }
 
-            add_request_header(self);
             add_response_header(self);
 
             if let Some(test) = opt_test {