Skip to content

Commit a6e7c3a

Browse files
tests: fixed test failure.
1 parent 3cf21a6 commit a6e7c3a

File tree

3 files changed

+28
-14
lines changed

3 files changed

+28
-14
lines changed

.travis.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,7 @@ env:
4545
- TEST_NGINX_RANDOMIZE=1
4646
- LUACHECK_VER=0.21.1
4747
matrix:
48-
- NGINX_VERSION=1.21.4 OPENSSL_VER=1.1.1s OPENSSL_PATCH_VER=1.1.1f
48+
- NGINX_VERSION=1.21.4 OPENSSL_VER=1.1.1u OPENSSL_PATCH_VER=1.1.1f
4949

5050
services:
5151
- memcache

t/ctx.t

Lines changed: 11 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -608,8 +608,10 @@ GET /t
608608
--- grep_error_log eval
609609
qr/(received: \w+|lua release ngx.ctx at ref \d+)/
610610
--- grep_error_log_out eval
611-
["",
612-
"lua release ngx.ctx at ref 2
611+
["lua release ngx.ctx at ref 1
612+
",
613+
"lua release ngx.ctx at ref 1
614+
lua release ngx.ctx at ref 2
613615
received: 42
614616
received: 1
615617
lua release ngx.ctx at ref 2
@@ -674,8 +676,10 @@ closed
674676
--- grep_error_log eval
675677
qr/lua release ngx.ctx at ref \d+/
676678
--- grep_error_log_out eval
677-
["",
679+
["lua release ngx.ctx at ref 1
680+
",
678681
"lua release ngx.ctx at ref 1
682+
lua release ngx.ctx at ref 1
679683
"]
680684
--- no_error_log
681685
[error]
@@ -772,15 +776,15 @@ GET /t
772776
qr/(received: \w+|lua release ngx.ctx at ref \d+)/
773777
--- grep_error_log_out eval
774778
["lua release ngx.ctx at ref 2
775-
received: 102
779+
received: 112
776780
lua release ngx.ctx at ref 2
777-
received: 102
781+
received: 112
778782
lua release ngx.ctx at ref 1
779783
",
780784
"lua release ngx.ctx at ref 2
781-
received: 102
785+
received: 112
782786
lua release ngx.ctx at ref 2
783-
received: 102
787+
received: 112
784788
lua release ngx.ctx at ref 1
785789
lua release ngx.ctx at ref 2
786790
received: 112

t/ssl-session-fetch.t

Lines changed: 16 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -97,7 +97,7 @@ qr/ssl_session_fetch_by_lua\(nginx.conf:\d+\):\d+: session id: [a-fA-f\d]+/s
9797
9898
--- grep_error_log_out eval
9999
[
100-
'',
100+
qr/ssl_session_fetch_by_lua\(nginx.conf:\d+\):4: session id: [a-fA-f\d]+/s,
101101
qr/ssl_session_fetch_by_lua\(nginx.conf:\d+\):4: session id: [a-fA-f\d]+/s,
102102
qr/ssl_session_fetch_by_lua\(nginx.conf:\d+\):4: session id: [a-fA-f\d]+/s,
103103
]
@@ -184,7 +184,8 @@ qr/ssl_session_fetch_by_lua:\d: session size: [a-fA-f\d]+|get session error: bad
184184
185185
--- grep_error_log_out eval
186186
[
187-
'',
187+
'get session error: bad session in lua context
188+
',
188189
'get session error: bad session in lua context
189190
',
190191
'get session error: bad session in lua context
@@ -220,7 +221,11 @@ In practice, never store session in plaintext on persistent storage.
220221
local ssl = require "ngx.ssl.session"
221222
local sid = ssl.get_session_id()
222223
print("session id: ", sid)
223-
local f = assert(io.open("$TEST_NGINX_SERVER_ROOT/html/session.tmp"))
224+
local f = io.open("$TEST_NGINX_SERVER_ROOT/html/session.tmp")
225+
if f == nil then
226+
return
227+
end
228+
224229
local sess = f:read("*a")
225230
f:close()
226231
ssl.set_serialized_session(sess)
@@ -383,7 +388,10 @@ qr/failed to resume session: failed to de-serialize session|ssl_session_(fetch|s
383388
384389
--- grep_error_log_out eval
385390
[
386-
qr/^ssl_session_store_by_lua\(nginx.conf:\d+\):5: session id: [a-fA-F\d]+$/s,
391+
qr/^ssl_session_fetch_by_lua\(nginx.conf:\d+\):4: session id: [a-fA-F\d]+
392+
failed to resume session: failed to de-serialize session
393+
ssl_session_store_by_lua\(nginx.conf:\d+\):5: session id: [a-fA-F\d]+
394+
$/s,
387395
qr/^ssl_session_fetch_by_lua\(nginx.conf:\d+\):4: session id: [a-fA-F\d]+
388396
failed to resume session: failed to de-serialize session
389397
ssl_session_store_by_lua\(nginx.conf:\d+\):5: session id: [a-fA-F\d]+
@@ -393,7 +401,6 @@ failed to resume session: failed to de-serialize session
393401
ssl_session_store_by_lua\(nginx.conf:\d+\):5: session id: [a-fA-F\d]+
394402
$/s,
395403
]
396-
397404
--- no_error_log
398405
[alert]
399406
[emerg]
@@ -527,7 +534,10 @@ $/s,
527534
528535
ssl_session_fetch_by_lua_block {
529536
local ssl = require "ngx.ssl.session"
530-
local f = assert(io.open("$TEST_NGINX_SERVER_ROOT/html/session.tmp"))
537+
local f = io.open("$TEST_NGINX_SERVER_ROOT/html/session.tmp")
538+
if f == nil then
539+
return
540+
end
531541
local sess = f:read("*a")
532542
f:close()
533543
ssl.set_serialized_session(sess)

0 commit comments

Comments
 (0)