bisakhmondal commented on a change in pull request #5594:
URL: https://github.com/apache/apisix/pull/5594#discussion_r758123078



##########
File path: apisix/plugins/azure-functions.lua
##########
@@ -14,30 +14,15 @@
 -- See the License for the specific language governing permissions and
 -- limitations under the License.
 
-local core = require("apisix.core")
-local http = require("resty.http")
 local plugin = require("apisix.plugin")
-local ngx  = ngx
-local plugin_name = "azure-functions"
+local plugin_name, plugin_version, priority = "azure-functions", 0.1, -1900
 
-local schema = {
+local azure_authz_schema = {

Review comment:
       I agree. This plugin depends on the refactoring proposed in #5616. For 
sake of fast development, I developed the feature on top of the branch 
([ref](https://github.com/apache/apisix/pull/5594#issuecomment-979496919)). As 
the changes in 5616 has been merged into master, I have rebased it. Thanks : ) 




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

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


Reply via email to