bzp2010 commented on a change in pull request #5874:
URL: https://github.com/apache/apisix/pull/5874#discussion_r775929570



##########
File path: apisix/plugins/opa/helper.lua
##########
@@ -45,16 +48,68 @@ local function build_http_request(conf, ctx)
 end
 
 
-function _M.build_opa_input(conf, ctx, subsystem)
-    local request = build_http_request(conf, ctx)
+local function build_http_route(conf, ctx, remove_upstream)
+    local route = core.table.clone(ctx.matched_route).value
+
+    if remove_upstream and route and route.upstream then

Review comment:
       @tzssangglass Yes, it has been used to generate upstream data before, on 
that time, it did not need to remove. It is possible that it will be used again 
in the future, should I remove it now?




-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to