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

Remove the downgrade code #11

Merged
merged 1 commit into from
Apr 23, 2018
Merged

Remove the downgrade code #11

merged 1 commit into from
Apr 23, 2018

Conversation

briankassouf
Copy link
Member

No description provided.

Copy link
Member

@jefferai jefferai left a comment

Choose a reason for hiding this comment

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

🎉

@briankassouf briankassouf merged commit d5a07c3 into master Apr 23, 2018
@briankassouf briankassouf deleted the no-downgrade branch April 23, 2018 19:31
maxb added a commit to maxb/vault-plugin-secrets-kv that referenced this pull request Jul 25, 2023
The `Passthrough` interface ceased to be used after some code was
removed in hashicorp#11.

The `GenerateLeases` function ceased to be used when this code was
made capable of being a separate plugin, rather than a builtin part of
Vault - credit to @remilapeyre for noticing this in hashicorp#55 - I'm just
cherrypicking the removal of unused code from that old unmerged PR,
whilst I had some other unused code to PR the removal of too.
@maxb maxb mentioned this pull request Jul 25, 2023
averche pushed a commit that referenced this pull request Jul 25, 2023
The `Passthrough` interface ceased to be used after some code was
removed in #11.

The `GenerateLeases` function ceased to be used when this code was
made capable of being a separate plugin, rather than a builtin part of
Vault - credit to @remilapeyre for noticing this in #55 - I'm just
cherrypicking the removal of unused code from that old unmerged PR,
whilst I had some other unused code to PR the removal of too.
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

2 participants