[GitHub] [apisix] tokers commented on a change in pull request #5947: chore(jwt-auth): get JWT by ctx.var.cookie_jwt instead of resty.cookie

[GitBox]

tokers commented on a change in pull request #5947:
URL: https://github.com/apache/apisix/pull/5947#discussion_r776125596



##########
File path: apisix/plugins/jwt-auth.lua
##########
@@ -344,8 +341,8 @@ end
 function _M.rewrite(conf, ctx)
     local jwt_token, err = fetch_jwt_token(ctx)
     if not jwt_token then
-        if err and err:sub(1, #"no cookie") ~= "no cookie" then
-            core.log.error("failed to fetch JWT token: ", err)
+        if err then

Review comment:
       Do we still need this check?




-- 
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