Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Transit: Remove enterprise no-ops references from API and SDK #196

Conversation

Gabrielopesantos
Copy link
Contributor

@Gabrielopesantos Gabrielopesantos commented Mar 10, 2024

Description
This PR is a follow-up to #142, it removes the managed_key_util.go file from the Transit engine, with no-ops enterprise functions and all of these functions.
Besides that, it also removes all references and usages of managed keys in the engine and SDK.

@Gabrielopesantos Gabrielopesantos force-pushed the gabrielopesantos/remove-transit-enterprise-no-ops branch from 0552789 to d4d1052 Compare March 10, 2024 22:48
@Gabrielopesantos Gabrielopesantos marked this pull request as ready for review March 11, 2024 23:27
@Gabrielopesantos Gabrielopesantos force-pushed the gabrielopesantos/remove-transit-enterprise-no-ops branch from d4d1052 to fa535cf Compare March 11, 2024 23:28
@Gabrielopesantos Gabrielopesantos force-pushed the gabrielopesantos/remove-transit-enterprise-no-ops branch from 307e469 to 985f5b7 Compare March 12, 2024 22:13
Copy link
Member

@cipherboy cipherboy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changes look good to me! Many thanks @Gabrielopesantos!

@naphelps naphelps self-requested a review March 13, 2024 14:08
@naphelps naphelps added this pull request to the merge queue Mar 13, 2024
@naphelps naphelps removed this pull request from the merge queue due to a manual request Mar 13, 2024
Signed-off-by: Gabriel Santos <gabrielopesantos97@gmail.com>
Signed-off-by: Gabriel Santos <gabrielopesantos97@gmail.com>
Signed-off-by: Gabriel Santos <gabrielopesantos97@gmail.com>
Signed-off-by: Gabriel Santos <gabrielopesantos97@gmail.com>
@Gabrielopesantos Gabrielopesantos force-pushed the gabrielopesantos/remove-transit-enterprise-no-ops branch from 985f5b7 to c2f7bd6 Compare March 13, 2024 23:29
@naphelps naphelps merged commit 8f33837 into openbao:main Mar 14, 2024
70 of 79 checks passed
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.

None yet

3 participants