Skip to content

Commit 51d86e8

Browse files
tests fixed
1 parent b42a61c commit 51d86e8

3 files changed

Lines changed: 8 additions & 3 deletions

File tree

ChangeLog

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,6 @@
11
07/01/2026
2+
- propagate session destroy and closed-mode revocation store failures from
3+
logout and session start instead of continuing as if logout succeeded
24
- do not use the shared JWT verification cache for bearer JWT validation
35
calls that pass additional claim validators, preventing a token accepted
46
by one validator set from bypassing a stricter validator set on cache hit.

tests/Dockerfile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ RUN ["luarocks", "install", "lua-resty-jwt"]
77

88
# install test dependencies
99
RUN apt-get update \
10-
&& apt-get install -y --no-install-recommends git \
10+
&& apt-get install -y --no-install-recommends ca-certificates git \
1111
&& rm -rf /var/lib/apt/lists/*
1212
RUN ["luarocks", "install", "busted"]
1313
RUN ["luarocks", "install", "LuaSocket"]

tests/spec/logout_revocation_spec.lua

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
local http = require("socket.http")
2+
local ltn12 = require("ltn12")
23
local test_support = require("test_support")
34
require 'busted.runner'()
45

@@ -8,17 +9,19 @@ describe("when revocation_fail_mode is closed and the revocation store is unreac
89
fail_mode = "closed",
910
set_fails = true,
1011
},
11-
unauth_action = "deny",
1212
})
1313
teardown(test_support.stop_server)
1414

1515
local _, _, cookie = test_support.login()
1616

17-
local body, status = http.request({
17+
local response_body = {}
18+
local _, status = http.request({
1819
url = "http://127.0.0.1/default/logout",
1920
headers = { cookie = cookie },
2021
redirect = false,
22+
sink = ltn12.sink.table(response_body),
2123
})
24+
local body = table.concat(response_body)
2225

2326
it("propagates destroy failure", function()
2427
assert.are.equals(401, status)

0 commit comments

Comments
 (0)