[GitHub] [apisix] bzp2010 commented on a change in pull request #5734: feat: support OPA plugin

2021-12-11 Thread GitBox
bzp2010 commented on a change in pull request #5734: URL: https://github.com/apache/apisix/pull/5734#discussion_r767153384 ## File path: t/plugin/opa.t ## @@ -0,0 +1,139 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one or more +# contributor license agreement

[GitHub] [apisix] bzp2010 commented on a change in pull request #5734: feat: support OPA plugin

2021-12-11 Thread GitBox
bzp2010 commented on a change in pull request #5734: URL: https://github.com/apache/apisix/pull/5734#discussion_r767153384 ## File path: t/plugin/opa.t ## @@ -0,0 +1,139 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one or more +# contributor license agreement

[GitHub] [apisix] bzp2010 commented on a change in pull request #5734: feat: support OPA plugin

2021-12-10 Thread GitBox
bzp2010 commented on a change in pull request #5734: URL: https://github.com/apache/apisix/pull/5734#discussion_r766456808 ## File path: t/plugin/opa.t ## @@ -0,0 +1,139 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one or more +# contributor license agreement

[GitHub] [apisix] bzp2010 commented on a change in pull request #5734: feat: support OPA plugin

2021-12-10 Thread GitBox
bzp2010 commented on a change in pull request #5734: URL: https://github.com/apache/apisix/pull/5734#discussion_r766455352 ## File path: apisix/plugins/opa.lua ## @@ -0,0 +1,103 @@ +-- +-- Licensed to the Apache Software Foundation (ASF) under one or more +-- contributor licens

[GitHub] [apisix] bzp2010 commented on a change in pull request #5734: feat: support OPA plugin

2021-12-10 Thread GitBox
bzp2010 commented on a change in pull request #5734: URL: https://github.com/apache/apisix/pull/5734#discussion_r766454197 ## File path: t/plugin/opa.t ## @@ -0,0 +1,140 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one or more +# contributor license agreement

[GitHub] [apisix] bzp2010 commented on a change in pull request #5734: feat: support OPA plugin

2021-12-09 Thread GitBox
bzp2010 commented on a change in pull request #5734: URL: https://github.com/apache/apisix/pull/5734#discussion_r766384350 ## File path: apisix/plugins/opa.lua ## @@ -0,0 +1,104 @@ +-- +-- Licensed to the Apache Software Foundation (ASF) under one or more +-- contributor licens

[GitHub] [apisix] bzp2010 commented on a change in pull request #5734: feat: support OPA plugin

2021-12-09 Thread GitBox
bzp2010 commented on a change in pull request #5734: URL: https://github.com/apache/apisix/pull/5734#discussion_r765865748 ## File path: apisix/core/request.lua ## @@ -269,6 +269,15 @@ function _M.get_port(ctx) end +function _M.get_path(ctx, original) Review comment:

[GitHub] [apisix] bzp2010 commented on a change in pull request #5734: feat: support OPA plugin

2021-12-09 Thread GitBox
bzp2010 commented on a change in pull request #5734: URL: https://github.com/apache/apisix/pull/5734#discussion_r765865189 ## File path: apisix/plugins/opa/helper.lua ## @@ -0,0 +1,62 @@ +-- +-- Licensed to the Apache Software Foundation (ASF) under one or more +-- contributor

[GitHub] [apisix] bzp2010 commented on a change in pull request #5734: feat: support OPA plugin

2021-12-09 Thread GitBox
bzp2010 commented on a change in pull request #5734: URL: https://github.com/apache/apisix/pull/5734#discussion_r765864564 ## File path: apisix/plugins/opa.lua ## @@ -0,0 +1,104 @@ +-- +-- Licensed to the Apache Software Foundation (ASF) under one or more +-- contributor licens

[GitHub] [apisix] bzp2010 commented on a change in pull request #5734: feat: support OPA plugin

2021-12-09 Thread GitBox
bzp2010 commented on a change in pull request #5734: URL: https://github.com/apache/apisix/pull/5734#discussion_r765855281 ## File path: t/plugin/opa.t ## @@ -0,0 +1,140 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one or more +# contributor license agreement

[GitHub] [apisix] bzp2010 commented on a change in pull request #5734: feat: support OPA plugin

2021-12-09 Thread GitBox
bzp2010 commented on a change in pull request #5734: URL: https://github.com/apache/apisix/pull/5734#discussion_r765855281 ## File path: t/plugin/opa.t ## @@ -0,0 +1,140 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one or more +# contributor license agreement

[GitHub] [apisix] bzp2010 commented on a change in pull request #5734: feat: support OPA plugin

2021-12-09 Thread GitBox
bzp2010 commented on a change in pull request #5734: URL: https://github.com/apache/apisix/pull/5734#discussion_r765852586 ## File path: apisix/plugins/opa/helper.lua ## @@ -0,0 +1,62 @@ +-- +-- Licensed to the Apache Software Foundation (ASF) under one or more +-- contributor