Firstsawyou commented on a change in pull request #3971: URL: https://github.com/apache/apisix/pull/3971#discussion_r606085000
########## File path: docs/en/latest/plugins/serverless.md ########## @@ -80,7 +82,7 @@ curl -i http://127.0.0.1:9080/apisix/admin/routes/1 -H 'X-API-KEY: edd1c9f03433 "plugins": { "serverless-pre-function": { "phase": "rewrite", - "functions" : ["return function() ngx.log(ngx.ERR, \"serverless pre function\"); end"] + "functions" : ["return function(conf, ctx) ngx.log(ngx.ERR, \"serverless pre function\"); end"] Review comment: I think we should add a new usage example of `conf & ctx` parameter. If we modify the original example, people can easily understand that the `conf & ctx` parameter is necessary. -- 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: [email protected]
