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.

Sign up for GitHub

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

Jump to bottom

Add support for Go 1.17 #2007

Merged
merged 1 commit into from
Sep 7, 2021
Merged

Add support for Go 1.17 #2007

merged 1 commit into from
Sep 7, 2021

Conversation

bai
Copy link
Contributor

@bai bai commented Sep 7, 2021

Add Go 1.17 to CI.

@caution-tape-bot
Copy link

We noticed that this PR either modifies or introduces code that pipes the output of curl or wget into a shell. This is dangerous because it grants full control over the machine that downloads and executes the script, with no way to control or monitor what is executed.
Code should be fetched in a way where its authenticity can be verified, for instance through a package manager, or by verifying a cryptographic hash or signature. If you are unsure how to do this, please ask in #help-appsec and we'll be happy to take a look. cc: @Shopify/appsec-breakers

@bai bai requested a review from dnwe September 7, 2021 09:02
@dnwe dnwe merged commit 7f53062 into master Sep 7, 2021
@bai bai deleted the go1.17 branch September 13, 2021 07:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dnwe dnwe dnwe approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@bai @dnwe