Skip to content

Commit

Permalink
tests: ssl.t: silenced the _G write guard log by declaring a couple o…
Browse files Browse the repository at this point in the history 
…f variables as local.
  • Loading branch information
@thibaultcha
thibaultcha committed Jul 18, 2019
1 parent e624e51 commit 63a5f7a
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 12 deletions.
12 changes: 6 additions & 6 deletions t/ssl.t
View file
Original file line number Diff line number Diff line change
Expand Up @@ -926,13 +926,13 @@ lua ssl server name: "test.com"
local cert_data = f:read("*a")
f:close()

cert_data, err = ssl.cert_pem_to_der(cert_data)
if not cert_data then
local cert, err = ssl.cert_pem_to_der(cert_data)
if not cert then
ngx.log(ngx.ERR, "failed to convert pem cert to der cert: ", err)
return
end

local ok, err = ssl.set_der_cert(cert_data)
local ok, err = ssl.set_der_cert(cert)
if not ok then
ngx.log(ngx.ERR, "failed to set DER cert: ", err)
return
Expand Down Expand Up @@ -1446,13 +1446,13 @@ ssl cert by lua done
local cert_data = f:read("*a")
f:close()

cert_data, err = ssl.cert_pem_to_der(cert_data)
if not cert_data then
local cert, err = ssl.cert_pem_to_der(cert_data)
if not cert then
ngx.log(ngx.ERR, "failed to convert pem cert to der cert: ", err)
return
end

local ok, err = ssl.set_der_cert(cert_data)
local ok, err = ssl.set_der_cert(cert)
if not ok then
ngx.log(ngx.ERR, "failed to set DER cert: ", err)
return
Expand Down
12 changes: 6 additions & 6 deletions t/stream/ssl.t
View file
Original file line number Diff line number Diff line change
Expand Up @@ -716,13 +716,13 @@ lua ssl server name: "test.com"
local cert_data = f:read("*a")
f:close()

cert_data, err = ssl.cert_pem_to_der(cert_data)
if not cert_data then
local cert, err = ssl.cert_pem_to_der(cert_data)
if not cert then
ngx.log(ngx.ERR, "failed to convert pem cert to der cert: ", err)
return
end

local ok, err = ssl.set_der_cert(cert_data)
local ok, err = ssl.set_der_cert(cert)
if not ok then
ngx.log(ngx.ERR, "failed to set DER cert: ", err)
return
Expand Down Expand Up @@ -1154,13 +1154,13 @@ ssl cert by lua done
local cert_data = f:read("*a")
f:close()

cert_data, err = ssl.cert_pem_to_der(cert_data)
if not cert_data then
local cert, err = ssl.cert_pem_to_der(cert_data)
if not cert then
ngx.log(ngx.ERR, "failed to convert pem cert to der cert: ", err)
return
end

local ok, err = ssl.set_der_cert(cert_data)
local ok, err = ssl.set_der_cert(cert)
if not ok then
ngx.log(ngx.ERR, "failed to set DER cert: ", err)
return
Expand Down

0 comments on commit 63a5f7a

Please sign in to comment.