jfrazee commented on pull request #5034: URL: https://github.com/apache/nifi/pull/5034#issuecomment-832076294
> @jfrazee Good questions. I'm using the term "Vault" as implementation-specific shorthand for HashiCorp Vault (similar to Spring, which calls their HashiCorp Vault service simply "Spring Vault"). I understand there are other similar technologies with similar names, like Azure Key Vault, so that could be a point of confusion. My thought here was to provide a HashiCorp Vault-specific integration point, which can be used in multiple places (Sensitive Properties Providers, Controller Services/Processors, and even ParameterContext providers in the future). That makes sense; it's util code to be used in the future in implementations of existing interfaces. > What do you think about renaming all of these to HashiCorpVault* to help reduce the confusion? Unfortunately, I do think that'd be good, but I also realize it's a super annoying ask. -- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
