SkyeYoung commented on code in PR #12603:
URL: https://github.com/apache/apisix/pull/12603#discussion_r2347730711
##########
t/plugin/jwt-auth.t:
##########
@@ -1001,25 +1001,20 @@ base64_secret required but the secret is not in base64
format
--- config
location /t {
content_by_lua_block {
- local t = require("lib.test_admin").test
- local code, body, res = t('/apisix/admin/consumers',
- ngx.HTTP_PUT,
- [[{
- "username": "kerouac",
- "plugins": {
- "jwt-auth": {
- "key": "exp-not-set",
- "secret": "my-secret-key"
- }
- }
- }]]
- )
-
- res = require("toolkit.json").decode(res)
- assert(res.value.plugins["jwt-auth"].exp == 86400)
+ local core = require("apisix.core")
Review Comment:
Because the original method can no longer detect whether the `exp` variable
exists, it's changed to direct validation through the schema, according to the
internal logic.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscr...@apisix.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
