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

Fix authorization bearer for existing terraform version #481

Merged
merged 3 commits into from
Aug 11, 2022

Conversation

eversC
Copy link
Contributor

@eversC eversC commented Aug 10, 2022

from #479 fork

@eversC
Copy link
Contributor Author

eversC commented Aug 10, 2022

did the /versions endpoint get made private at some point?

I think we may need to bump the orb version here too

Does the same change need to be made to the terraform-v2 orb?

@drewhemm
Copy link
Contributor

Yes, the same change needs to be made to terraform-v2 - apolologies, as the PR should have included both. I have pushed a second commit to my fork

@eversC
Copy link
Contributor Author

eversC commented Aug 10, 2022

Yes, the same change needs to be made to terraform-v2 - apolologies, as the PR should have included both. I have pushed a second commit to my fork

looks good

I think the orb versions (+changelogs) need updating, as that way the changes will get automatically released when we merge.

@drewhemm
Copy link
Contributor

I've updated my fork with the changes to the orb versions and changelogs:

drewhemm@8aa1388

@eversC eversC merged commit 54ff7c9 into master Aug 11, 2022
@eversC eversC deleted the fix-terraform-existing-version branch August 11, 2022 15:18
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.

3 participants