maskit commented on issue #672: Support setting private key content in athenz client plugin URL: https://github.com/apache/incubator-pulsar/pull/672#issuecomment-321729113 OK, that makes sense. Since it's just a plugin I don't object introducing its own unique way, however, it sounds like not athenz specific issue. The same issue will appear if somebody tried to use TLS auth, and they would need to re-implement the logic to read secrets from somewhere. Also, all users who read it from environment variables need to implement the logic even though it's not an unique way. How about adding support for URL schemes (e.g. file:, data:, env:, exec:) to privateKeyPath config param? If we provided the reader and plugins used it, the plugins would get the same benefit when we support new scheme (new data source), plus it can be applicable for any configuration parameters. ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
With regards, Apache Git Services