kevinw66 commented on issue #6786: URL: https://github.com/apache/apisix/issues/6786#issuecomment-1093740197
> > So, should we add such a phase like `enable` or `disable` and receive `conf, ctx` like other phase? > > If we want to mark `enable` or `disable` phase of plugin, we need to refactor APISIX's plugin operation mechanism, so that when APISIX synchronises data from etcd, it can find out which plugin are new by comparing old and new data. and call the `enable` function for those plugins. It is also necessary to know which plugins are removed and to call the `disable` function. > > IMO, I don't like this improvement, it makes things more complicated, and there is no need to implement this now. To achieve similar functionality, I think it could be done with the `init` and `destroy` functions under the existing architecture. Got 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
