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.

Sign up for GitHub

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

Jump to bottom

Should keyvault be in sdk/security or sdk/keyvault ? #6412

Open
antkmsft opened this issue Feb 12, 2025 · 1 comment
Open

Should keyvault be in sdk/security or sdk/keyvault ? #6412

antkmsft opened this issue Feb 12, 2025 · 1 comment
Assignees
@gearama
Labels
Client This issue points to a problem in the data-plane of the library. KeyVault needs-team-attention Workflow: This issue needs attention from Azure service team or SDK team

Comments

@antkmsft
Copy link
Member

antkmsft commented Feb 12, 2025 •
edited
Loading

In Go, keyvault is located in sdk/security.

In Rust and C++, it is located in sdk/keyvault.

I am going to open an issue for all 3 repos, if we want some consistency.

See also #6411 (same question about messaging/eventhubs)
@antkmsft antkmsft mentioned this issue Feb 12, 2025
Closed
@github-actions github-actions bot added Client This issue points to a problem in the data-plane of the library. KeyVault needs-team-attention Workflow: This issue needs attention from Azure service team or SDK team labels Feb 12, 2025
@github-actions GitHub Actions
Copy link

Thank you for your feedback. Tagging and routing to the team member best able to assist.

This was referenced Feb 12, 2025
Closed
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@gearama gearama

Labels
Client This issue points to a problem in the data-plane of the library. KeyVault needs-team-attention Workflow: This issue needs attention from Azure service team or SDK team
Projects
Azure SDK for Key Vault
Status: Untriaged
Milestone
No milestone
Development

No branches or pull requests
2 participants
@antkmsft @gearama