maskit commented on PR #13807: URL: https://github.com/apache/trafficserver/pull/13807#issuecomment-6048405803
I like the idea of configuring the plugin with a config file. And I'm fine with enabling the plugin to handle multiple fingerprints with one instance. I'm not happy with the removal of the command line style configuration. Although keeping it does not make sense for plugin.yaml, master branch still supports plugin.config as well. It'd be unfortunate to be unable to use plugin.config because of the plugin incompatibility. Keeping the support for the command line style would make migration from 10.x to 11.0 easier. -- 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]
