Commitb397d1e

blink-so[bot]

and

kylecarbs

committed

fix: use more restrictive file permissions in test (0o600)

Fixes gosec G306 linting error that expects WriteFile permissionsto be 0600 or less for security reasons.Co-authored-by: kylecarbs <7122116+kylecarbs@users.noreply.github.com>

1 parenta45984d commitb397d1eCopy full SHA for b397d1e

File tree

1 file changed

-2

lines changed

provisioner/terraform
- executor_internal_test.go

1 file changed

-2

lines changed

`‎provisioner/terraform/executor_internal_test.go`

Lines changed: 2 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -197,7 +197,7 @@ func TestCalculateFileChecksum(t *testing.T) {`
`197`	`197`	`testFile:=filepath.Join(tmpDir,"test.txt")`
`198`	`198`	`testContent:="test content for checksum"`
`199`	`199`
`200`		`-err:=os.WriteFile(testFile, []byte(testContent),0o644)`
	`200`	`+err:=os.WriteFile(testFile, []byte(testContent),0o600)`
`201`	`201`	`require.NoError(t,err)`
`202`	`202`
`203`	`203`	`checksum1:=calculateFileChecksum(testFile)`
`@@ -209,7 +209,7 @@ func TestCalculateFileChecksum(t *testing.T) {`
`209`	`209`	`require.Equal(t,checksum1,checksum2)`
`210`	`210`
`211`	`211`	`// Different content should produce different checksum`
`212`		`-err=os.WriteFile(testFile, []byte("different content"),0o644)`
	`212`	`+err=os.WriteFile(testFile, []byte("different content"),0o600)`
`213`	`213`	`require.NoError(t,err)`
`214`	`214`
`215`	`215`	`checksum3:=calculateFileChecksum(testFile)`

0 commit comments

Comments

(0)

Movatterモバイル変換

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commitb397d1e

File tree

1 file changed

1 file changed

`‎provisioner/terraform/executor_internal_test.go`

0 commit comments