Skip to content

Commit

Permalink
Merge pull request #54 from hashicorp/mw/update-go-version
Browse files Browse the repository at this point in the history
update go-version and address CVE
  • Loading branch information
wilkermichael authored Feb 22, 2023
2 parents 1612af1 + 72405d2 commit 98fdc3e
Show file tree
Hide file tree
Showing 7 changed files with 64 additions and 18 deletions.
28 changes: 23 additions & 5 deletions .github/workflows/bin-ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -3,24 +3,40 @@ on:
push:
branches: '**'
env:
GO_VERSION: '1.19'
CONSUL_LICENSE: ${{ secrets.CONSUL_LICENSE }}
jobs:
get-go-version:
name: "Determine Go toolchain version"
defaults:
run:
working-directory: ./consul-lambda
runs-on: ubuntu-latest
outputs:
go-version: ${{ steps.get-go-version.outputs.go-version }}
steps:
- uses: actions/checkout@v2
- name: Determine Go version
id: get-go-version
run: |
echo "Building with Go $(cat .go-version)"
echo "::set-output name=go-version::$(cat .go-version)"
lint:
defaults:
run:
working-directory: ./consul-lambda/consul-lambda-registrator
runs-on: ubuntu-latest
needs:
- get-go-version
steps:
- name: Checkout
uses: actions/checkout@v3
- uses: actions/setup-go@v3
with:
go-version: ${{ env.GO_VERSION }}
go-version: ${{ needs.get-go-version.outputs.go-version }}
- name: golangci-lint
uses: golangci/golangci-lint-action@v3
uses: golangci/golangci-lint-action@v3.4.0
with:
version: v1.50.1
version: v1.51
args: |
--verbose
only-new-issues: false
Expand All @@ -35,6 +51,8 @@ jobs:
consul-version:
- 1.15.0
- 1.15.0+ent
needs:
- get-go-version
defaults:
run:
working-directory: ./consul-lambda/consul-lambda-registrator
Expand All @@ -47,7 +65,7 @@ jobs:
uses: actions/checkout@v3
- uses: actions/setup-go@v3
with:
go-version: ${{ env.GO_VERSION }}
go-version: ${{ needs.get-go-version.outputs.go-version }}

- name: Install Consul
uses: nickethier/action-setup-hashicorp-tool@main #TODO: update action when migrated to hc org
Expand Down
5 changes: 2 additions & 3 deletions .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -19,8 +19,8 @@ jobs:
- name: Determine Go version
id: get-go-version
run: |
echo "Building with Go $(cat ./go-version)"
echo "::set-output name=go-version::$(cat ./.go-version)"
echo "Building with Go $(cat .go-version)"
echo "::set-output name=go-version::$(cat .go-version)"
get-product-version:
defaults:
Expand Down Expand Up @@ -84,7 +84,6 @@ jobs:
uses: actions/setup-go@v2
with:
go-version: ${{ needs.get-go-version.outputs.go-version }}

- name: Build
run: |
mkdir dist out extensions
Expand Down
34 changes: 27 additions & 7 deletions .github/workflows/terraform-ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,6 @@ on:
- 'main'
- 'release/**'
env:
GO_VERSION: '1.18'
CONSUL_LICENSE: ${{ secrets.CONSUL_LICENSE }}
jobs:
terraform-lint:
Expand All @@ -23,25 +22,43 @@ jobs:
terraform_version: v1.3.1
- name: Validate
run: terraform fmt -check -recursive .
get-go-version:
name: "Determine Go toolchain version"
defaults:
run:
working-directory: ./consul-lambda
runs-on: ubuntu-latest
outputs:
go-version: ${{ steps.get-go-version.outputs.go-version }}
steps:
- uses: actions/checkout@v2
- name: Determine Go version
id: get-go-version
run: |
echo "Building with Go $(cat .go-version)"
echo "::set-output name=go-version::$(cat .go-version)"
go-test-lint:
runs-on: ubuntu-latest
defaults:
run:
working-directory: ./test/acceptance
needs:
- get-go-version
steps:
- name: Checkout
uses: actions/checkout@v3
- uses: actions/setup-go@v3
- name: Setup go
uses: actions/setup-go@v2
with:
go-version: ${{ env.GO_VERSION }}
go-version: ${{ needs.get-go-version.outputs.go-version }}
- name: lint-consul-retry
run: |
go install github.com/hashicorp/lint-consul-retry@latest
lint-consul-retry
- name: golangci-lint
uses: golangci/golangci-lint-action@v3
uses: golangci/golangci-lint-action@v3.4.0
with:
version: v1.45
version: v1.51
args: |
--verbose
only-new-issues: false
Expand All @@ -56,13 +73,16 @@ jobs:
env:
TEST_RESULTS: /tmp/test-results
GOTESTSUM_VERSION: 1.8.0
needs: [go-test-lint, terraform-lint]
needs:
- go-test-lint
- terraform-lint
- get-go-version
steps:
- name: Checkout
uses: actions/checkout@v3
- uses: actions/setup-go@v3
with:
go-version: ${{ env.GO_VERSION }}
go-version: ${{ steps.get-go-version.outputs.go-version }}
- name: Setup gotestsum
shell: bash
run: |
Expand Down
9 changes: 9 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,3 +1,12 @@
## UNRELEASED

FEATURES
* Update minimum go version for project to 1.20 [[GH-1908](https://github.com/hashicorp/terraform-aws-consul-lambda/pull/54)]

BUG FIXES:
* Security:
* Upgrade to use Go 1.20.1 This resolves vulnerabilities [CVE-2022-41724](https://go.dev/issue/58001) in `crypto/tls` and [CVE-2022-41723](https://go.dev/issue/57855) in `net/http`. [[GH-1908](https://github.com/hashicorp/terraform-aws-consul-lambda/pull/54)]

## 0.1.0-beta2 (October 04, 2022)

FEATURES
Expand Down
2 changes: 1 addition & 1 deletion consul-lambda/.go-version
Original file line number Diff line number Diff line change
@@ -1 +1 @@
1.18.2
1.20.1
2 changes: 1 addition & 1 deletion consul-lambda/go.mod
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
module github.com/hashicorp/terraform-aws-consul-lambda/consul-lambda

go 1.19
go 1.20

require (
github.com/aws/aws-lambda-go v1.31.1
Expand Down
2 changes: 1 addition & 1 deletion test/acceptance/go.mod
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
module github.com/hashicorp/terraform-aws-consul-lambda/test/acceptance

go 1.19
go 1.20

require (
github.com/aws/aws-lambda-go v1.13.3
Expand Down

0 comments on commit 98fdc3e

Please sign in to comment.