Skip to content

Commit

Permalink
chore(deps): bump ngx_wasm_module to 91d447ffd0e9bb08f11cc69d1aa9128e…
Browse files Browse the repository at this point in the history
…c36b4526 (#12941)

Changes since 3bd94e61c55415ccfb0f304fa51143a7d630d6ae:

* 91d447ff - feat(wasmtime) new 'cache_config' directive for compilation cache
* 4ed38176 - feat(proxy-wasm) consistent context checks for all header setters
* ee5890dd - feat(proxy-wasm) allow setting ':status' response pseudo-header
* bb139e16 - hotfix(http) fix an expression in postpone checks
* d70c87d4 - refactor(wasm) fix conflicting values in our '*_CONF' definitions
* 7ee4eb45 - tests(backtraces) update backtrace formats for latest Rust toolchain
* 1d4534d0 - feat(http) implement 'postpone_rewrite' + 'postpone_access' directives
* 473a6705 - refactor(wasmx) store ngx_wa_conf_t as our core module context
* fa9bc6b9 - chore(deps) bump Nginx to 1.25.5
* faf302c5 - chore(deps) bump OpenSSL to 3.3.0
  • Loading branch information
hishamhm authored Apr 26, 2024
1 parent 7b5add3 commit a5a61d6
Show file tree
Hide file tree
Showing 5 changed files with 10 additions and 8 deletions.
2 changes: 1 addition & 1 deletion .requirements
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ ATC_ROUTER=ffd11db657115769bf94f0c4f915f98300bc26b6 # 1.6.2
SNAPPY=23b3286820105438c5dbb9bc22f1bb85c5812c8a # 1.2.0

KONG_MANAGER=nightly
NGX_WASM_MODULE=3bd94e61c55415ccfb0f304fa51143a7d630d6ae
NGX_WASM_MODULE=91d447ffd0e9bb08f11cc69d1aa9128ec36b4526
WASMER=3.1.1
WASMTIME=19.0.0
V8=12.0.267.17
Expand Down
2 changes: 1 addition & 1 deletion changelog/unreleased/kong/bump-ngx-wasm-module.yml
Original file line number Diff line number Diff line change
@@ -1,2 +1,2 @@
message: "Bump `ngx_wasm_module` to `3bd94e61c55415ccfb0f304fa51143a7d630d6ae`"
message: "Bump `ngx_wasm_module` to `91d447ffd0e9bb08f11cc69d1aa9128ec36b4526`"
type: dependency
2 changes: 1 addition & 1 deletion spec/02-integration/20-wasm/04-proxy-wasm_spec.lua
Original file line number Diff line number Diff line change
Expand Up @@ -281,7 +281,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()
Expand Down
1 change: 0 additions & 1 deletion spec/fixtures/proxy_wasm_filters/tests/src/routines.rs
Original file line number Diff line number Diff line change
Expand Up @@ -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";
Expand Down
11 changes: 7 additions & 4 deletions spec/fixtures/proxy_wasm_filters/tests/src/test_http.rs
Original file line number Diff line number Diff line change
Expand Up @@ -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 {
Expand Down

1 comment on commit a5a61d6

@github-actions
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bazel Build

Docker image available kong/kong:a5a61d645aad3107d4c742cf9537bb4d673c8fbe
Artifacts available https://github.com/Kong/kong/actions/runs/8845726933

Please sign in to comment.