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

(bug-fix) Update modules to accept new API header #99

Closed
wants to merge 1 commit into from
Closed

Conversation

lucywyman
Copy link
Contributor

Hashicorp recently added a Release
API

that includes a new Content-Type header
application/vnd+hashicorp.releases-api.v0+json. This gets used
somewhere in making requests, but the hc-install Go library was
previously at version 0.3.1 which raised an error when it encountered
this header. V0.3.2 accepts the header, so this updates the Go modules
in this repo using go get -u && go mod tidy.

Hashicorp [recently added a Release
API](https://www.hashicorp.com/blog/announcing-the-hashicorp-releases-api)
that includes a new `Content-Type` header
`application/vnd+hashicorp.releases-api.v0+json`. This gets used
somewhere in making requests, but the `hc-install` Go library was
previously at version 0.3.1 which raised an error when it encountered
this header. V0.3.2 accepts the header, so this updates the Go modules
in this repo using `go get -u && go mod tidy`.
@lucywyman
Copy link
Contributor Author

Honestly kind of unclear to me if this will just get magically fixed in CI and I don't need to do this?

@ctawiah
Copy link
Contributor

ctawiah commented May 12, 2022

Honestly kind of unclear to me if this will just get magically fixed in CI and I don't need to do this?

Yeah this terraform SDK version update should fix it based on what I can tell from their release notes for 2.16.0. The latest hc-install now includes the application/vnd+harshipcorp.releases-api.v0+json header https://github.com/hashicorp/hc-install/blob/main/internal/releasesjson/releases.go#L82

Copy link
Contributor

@ctawiah ctawiah left a comment

Choose a reason for hiding this comment

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

LGTM

@lucywyman
Copy link
Contributor Author

Opened a PR against the private repo hashicorp/terraform-plugin-sdk@f74b5a6

@lucywyman lucywyman closed this May 12, 2022
@lucywyman lucywyman deleted the fix-ci branch August 11, 2022 21:54
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