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

Add support for consul secret engine #73

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Haennetz
Copy link
Collaborator

Added #7.

@Haennetz Haennetz force-pushed the feature/add-consul-secret-engine branch from 27fa8f4 to 7cb8626 Compare February 10, 2024 15:00
@Haennetz Haennetz force-pushed the feature/add-consul-secret-engine branch from 7cb8626 to aed566a Compare February 23, 2024 23:22
@Haennetz Haennetz force-pushed the feature/add-consul-secret-engine branch from aed566a to abc3b5d Compare February 23, 2024 23:29
@stormshield-gt stormshield-gt self-requested a review October 31, 2024 11:00
Copy link
Collaborator

@stormshield-gt stormshield-gt left a comment

Choose a reason for hiding this comment

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

That's a very nice addition, thanks for your work

/// * Path: {self.mount}/roles/{self.name}
/// * Method: POST
/// * Response: N/A
/// * Reference: https://www.vaultproject.io/api-docs/secret/consul#configure-access
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
/// * Reference: https://www.vaultproject.io/api-docs/secret/consul#configure-access
/// * Reference: https://www.vaultproject.io/api-docs/secret/consul#create-update-role

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.

2 participants