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

Update vault-install.sh #8

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Conversation

trv-dwunder2
Copy link

What does this PR do?

Fix issue with new hashi structure with new extension type

What image or gif best represents this PR?

Why is this needed?

Hashi added an extension to their releases

How does it work?

Please explain how this was tested

  • If applicable, have you wrote unit tests?

Screenshots (optional)

Documentation

  • Have you updated the appropriate documentation?

Anything Else?

Fix issue with new hashi structure with new extension type
@aaronrea
Copy link

Do you have a link to any indication of this in the Hashi Docs that we could reference?

@trv-dwunder2
Copy link
Author

The only way that you can see this by going to the down load site and looking at the various files; naming that they have. https://releases.hashicorp.com/vault/ code looks for this first instance of +ent. It may have one of the extensions part in it the line or not. Didnt think of this before. Instead of removing the extension part. could do a cut on "+" and take the first part.

Better way to get the current version
@trv-dwunder2
Copy link
Author

Updated to look for +ent and use cut to isolate the version number. cleaner.

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