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

umask final version against master (again) #256

Merged
merged 2 commits into from
Sep 16, 2020
Merged

Conversation

shoenig
Copy link
Member

@shoenig shoenig commented May 28, 2020

This is #198 but rebased with master as of May 2020.

Nomad needs a recent fix for hashicorp/nomad#7854

cc @langmartin

shoenig added a commit to hashicorp/nomad that referenced this pull request May 28, 2020
Fixes #7854

Nomad requires a version of go-getter that is currently in PR (hashicorp/go-getter#256)
We also require some recent bug fix to go-getter around the handling of URL redirects.

Update our vendor'd copy of go-getter to the newly rebased umask changes so that we can incorporate
the latest changes for go-getter.
shoenig added a commit to hashicorp/nomad that referenced this pull request May 28, 2020
Fixes #7854

Nomad requires a version of go-getter that is currently in PR (hashicorp/go-getter#256)
We also require some recent bug fix to go-getter around the handling of URL redirects.

Update our vendor'd copy of go-getter to the newly rebased umask changes so that we can incorporate
the latest changes for go-getter.
Copy link
Contributor

@langmartin langmartin left a comment

Choose a reason for hiding this comment

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

This looks like everything...

Copy link
Contributor

@notnoop notnoop left a comment

Choose a reason for hiding this comment

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

LGTM and this has been running as-in in nomad for many versions now.

@notnoop notnoop merged commit 7ac233b into master Sep 16, 2020
@azr azr deleted the f-umask-master-may-2020 branch September 16, 2020 15:09
@azr
Copy link
Contributor

azr commented Sep 16, 2020

Nice one !

notnoop pushed a commit to hashicorp/nomad that referenced this pull request Oct 15, 2020
Update go-getter to the latest release now that hashicorp/go-getter#256 is merged.
Also, pick up a xz CVE fix that go-getter merged in hashicorp/go-getter#279 .
fredrikhgrelland pushed a commit to fredrikhgrelland/nomad that referenced this pull request Oct 22, 2020
Update go-getter to the latest release now that hashicorp/go-getter#256 is merged.
Also, pick up a xz CVE fix that go-getter merged in hashicorp/go-getter#279 .
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

4 participants