airship/airshipui
Code Issues Proposed changes

Bump linter from 1.29 to 1.36

Browse Source 
Updating linter to latest version, which also updates
gosec to latest version to keep on top of updated security
checks in source code.

To address security concerns raised by gosec with this update
the MinVersion of TLS is set to 13, as expected in this update to
gosec [0]

[0] 55d368f2e5

Co-Authored-By: Ian Howell <ian.howell0@gmail.com>
Co-Authored-By: Alexander Hughes <Alexander.Hughes@pm.me>
Change-Id: I18cc618d24fb6979944df45f4755f4abc3ebc421
Signed-off-by: Alexander Hughes <Alexander.Hughes@pm.me>
This commit is contained in:
Alexander Hughes 2021-01-29 09:47:34 -05:00
parent 58f978e76a
commit aeaf823ecd
2 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
pkg/webservice/server.go
View File

@ -92,6 +92,7 @@ func WebServer() {
InsecureSkipVerify: false,
ServerName: configs.UIConfig.WebService.Host,
Certificates: getCertificates(),
MinVersion: tls.VersionTLS13,
},
Handler: webServerMux,
ErrorLog: log.Logger(),

2
tools/install_go_linter
View File

@ -3,7 +3,7 @@ set -x
tools_bin_dir="${BASH_SOURCE%/*}"
download_url=https://raw.githubusercontent.com/golangci/golangci-lint/master/install.sh
version=v1.27.0
version=v1.36.0
if ! curl -sfL "$download_url" | sh -s -- -b "$tools_bin_dir/bin" "$version"; then
printf "Something went wrong while installing golangci-lint\n" 1>&2