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 2021/01/12 09:17:16 UTC

[GitHub] [apisix-dashboard] imjoey edited a comment on issue #1275: bug: the script field in Route is limited to map, instead of string

imjoey edited a comment on issue #1275:
URL: https://github.com/apache/apisix-dashboard/issues/1275#issuecomment-758519103


   > oh, please have a look at test cases for plugin orchestration:
   > https://github.com/apache/apisix-dashboard/blob/master/api/test/e2e/route_with_plugin_orchestration_test.go
   > 
   > and you will know why.
   
   @nic-chen
   
   Thanks for pointing this out. I guess you mean the `Script` of `map` type is only for plugin orchestration in FE, just as the comment **-- The 'script' fields below are used by dashboard for plugin orchestration** in https://github.com/apache/apisix/blob/3e19b06293463f52c8d26f0958542fc475ff0fcd/apisix/schema_def.lua#L455 ? Is that correct?
   
   If yes, may I ask that how we could add a script like `"local _M = {} \n function _M.access(api_ctx) \n ngx.log(ngx.INFO,\"hit access phase\") \n end \nreturn _M"` in Route via manager-api, to implement the serverless services? 
   
   Please let me know if I missed anything important. 😄  Much appreciated. 
   
   


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

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