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

Token\Key Secret Storage doc update #256

Closed
darylhorton opened this issue Feb 13, 2024 · 3 comments · Fixed by #257
Closed

Token\Key Secret Storage doc update #256

darylhorton opened this issue Feb 13, 2024 · 3 comments · Fixed by #257
Assignees
Labels
documentation Improvements or additions to documentation

Comments

@darylhorton
Copy link

No description provided.

@gdbarron
Copy link
Collaborator

@darylhorton what is this issue for?

@darylhorton
Copy link
Author

Apologies, I thought I saved it properly, but apparently not. Basically, after the initial creation of the secret store in powershell, it is recommended to run the following to get started.
New-VenafiSession -Server my.venafi.com -Credential $myCred -ClientId MyApp -Scope $scope -VaultRefreshTokenName tpp-token -VaultMetadata

The issue is that -VaultMetadata doesn't seem to be an option. I am running the latest version of VenafiPS. Either the issue is with the documentation or the module itself. I was just curious if this is something that should be there or not.

@gdbarron
Copy link
Collaborator

The issue is with the documentation. -VaultMetadata was deprecated a year ago and that was missed. When vaulting secrets the metadata will be stored automatically. I'll get this fixed. Thanks for reporting it.

@gdbarron gdbarron self-assigned this Feb 13, 2024
@gdbarron gdbarron added the documentation Improvements or additions to documentation label Feb 13, 2024
@gdbarron gdbarron changed the title Token\Key Secret Storage Token\Key Secret Storage doc update Feb 13, 2024
@gdbarron gdbarron linked a pull request Feb 13, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants