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

Is HVAC unable to verify a re-key operation with the recovery_key=True flag? #1097

Open
briantist opened this issue Oct 18, 2023 Discussed in #1094 · 0 comments · May be fixed by #1095
Open

Is HVAC unable to verify a re-key operation with the recovery_key=True flag? #1097

briantist opened this issue Oct 18, 2023 Discussed in #1094 · 0 comments · May be fixed by #1095
Labels
enhancement a new feature or addition system backend generally related to the Vault system backend routes

Comments

@briantist
Copy link
Contributor

Discussed in #1094

Originally posted by Ar-personal October 17, 2023
The HVAC code usually swaps out the api path for re-key operations from api_path = "/v1/sys/rekey/" to api_path = "/v1/sys/rekey-recovery-key/ However the rekey_verify function only has the former option, am I missing something here or is this incomplete?

def rekey_verify(self, key, nonce):
...
api_path = "/v1/sys/rekey/verify"
@briantist briantist added enhancement a new feature or addition system backend generally related to the Vault system backend routes labels Oct 18, 2023
@briantist briantist linked a pull request Oct 18, 2023 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement a new feature or addition system backend generally related to the Vault system backend routes
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Allow verifying recovery keys wilsonge/hvac
1 participant
@briantist