Skip to content

add [tencentcloudkms server seal] and [tencentcloudcos physical storage]#9253

Closed
likexian wants to merge 4 commits into
hashicorp:mainfrom
tencentyun:master
Closed

add [tencentcloudkms server seal] and [tencentcloudcos physical storage]#9253
likexian wants to merge 4 commits into
hashicorp:mainfrom
tencentyun:master

Conversation

@likexian
Copy link
Copy Markdown

@likexian likexian commented Jun 18, 2020

Hello,

This PR add TencentCloud KMS as one of the method of server seal and add TencentCloud COS as one of the physical storage.

I have test it and it works.

Please have a review, thank you.

@likexian likexian changed the title add tencentcloudkms server seal add [tencentcloudkms server seal] and [tencentcloudcos physical storage] Jun 30, 2020
@likexian
Copy link
Copy Markdown
Author

likexian commented Jul 2, 2020

Hello @pbernal @jefferai, Would you please have a look at this?

@heatherezell
Copy link
Copy Markdown
Contributor

Hi there! For various reasons, we are not accepting new open-source storage backends or secrets engines from the community. These reasons include supportability, Vault engineering lift, and potential security implications. While we appreciate your effort, I'm going to go and close this PR now. Thanks for understanding.

@jefferai
Copy link
Copy Markdown
Member

jefferai commented Aug 9, 2022

Note that when Vault supports go-kms-wrapping v2 then explicit support in Vault won't be necessary as it can just be built independently as a plug-in.

anwittin pushed a commit that referenced this pull request Sep 18, 2025
Co-authored-by: kelly <69541941+kporter101@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants