Fix GoSec Test (#258)

* (feat) fix gosec test * change approach
bridgecrewio · Dec 21, 2021 · 68a8a2b · 68a8a2b
1 parent a9cbb71
commit 68a8a2b
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/src/common/yaml/yaml_writer.go b/src/common/yaml/yaml_writer.go
@@ -20,8 +20,8 @@ import (
 const SingleIndent = "  "
 
 func WriteYAMLFile(readFilePath string, blocks []structure.IBlock, writeFilePath string, tagsAttributeName string, resourcesStartToken string) error {
-	// read file bytes
 	// #nosec G304
+	// read file bytes
 	originFileSrc, err := ioutil.ReadFile(readFilePath)
 	if err != nil {
 		return fmt.Errorf("failed to read file %s because %s", readFilePath, err)

diff --git a/src/terraform/structure/terraform_parser.go b/src/terraform/structure/terraform_parser.go
@@ -117,8 +117,8 @@ func (p *TerrraformParser) ValidFile(_ string) bool {
 }
 
 func (p *TerrraformParser) ParseFile(filePath string) ([]structure.IBlock, error) {
-	// read file bytes
 	// #nosec G304
+	// read file bytes
 	src, err := ioutil.ReadFile(filePath)
 	if err != nil {
 		return nil, fmt.Errorf("failed to read file %s because %s", filePath, err)
@@ -171,8 +171,8 @@ func (p *TerrraformParser) ParseFile(filePath string) ([]structure.IBlock, error
 }
 
 func (p *TerrraformParser) WriteFile(readFilePath string, blocks []structure.IBlock, writeFilePath string) error {
-	// read file bytes
 	// #nosec G304
+	// read file bytes
 	src, err := ioutil.ReadFile(readFilePath)
 	if err != nil {
 		return fmt.Errorf("failed to read file %s because %s", readFilePath, err)