You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@apisix.apache.org by GitBox <gi...@apache.org> on 2022/11/05 13:27:22 UTC

[GitHub] [apisix] bzp2010 opened a new issue, #8251: bug: built-in variables are not supported in `_meta.filter`

bzp2010 opened a new issue, #8251:
URL: https://github.com/apache/apisix/issues/8251

   ### Current Behavior
   
   The filters in the plugin support only Nginx variables, not APISIX's built-in variables.
   
   For example, I can use the Nginx variable `http_xxx`, but I can't use APISIX custom variables like `arg_xxxx` or `graphql_xxxx`.
   
   ### Expected Behavior
   
   It can support all APISIX variable syntax, including `cookie_xxxx`, `arg_xxxx`, `post_arg_xxxx`, `graphql_xxxx`, etc.
   
   ### Error Logs
   
   _No response_
   
   ### Steps to Reproduce
   
   1. Set the filter in the plugin to use the above unsupported syntax.
   
   -------------
   
   **The reason is that we didn't pass in our ctx for that expr executor, and the expr uses its default `ngx.var`, which wouldn't support extended syntax.**
   
   https://github.com/apache/apisix/blob/master/apisix/plugin.lua#L437
   https://github.com/api7/lua-resty-expr/blob/main/lib/resty/expr/v1.lua#L376
   
   ### Environment
   
   - APISIX version (run `apisix version`): 3.0.0
   


-- 
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-unsubscribe@apisix.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [apisix] soulbird commented on issue #8251: bug: built-in variables are not supported in `_meta.filter`

Posted by GitBox <gi...@apache.org>.
soulbird commented on issue #8251:
URL: https://github.com/apache/apisix/issues/8251#issuecomment-1304549482

   I will fix it later.


-- 
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-unsubscribe@apisix.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [apisix] spacewander closed issue #8251: bug: built-in variables are not supported in `_meta.filter`

Posted by GitBox <gi...@apache.org>.
spacewander closed issue #8251: bug: built-in variables are not supported in `_meta.filter`
URL: https://github.com/apache/apisix/issues/8251


-- 
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-unsubscribe@apisix.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org