incorporate review changes

tenable · May 4, 2021 · 5f8c83d · 5f8c83d
1 parent 8bb60a2
commit 5f8c83d
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/build/Dockerfile b/build/Dockerfile
@@ -27,9 +27,9 @@ RUN addgroup --gid 101 terrascan && \
     adduser -S --uid 101 --ingroup terrascan terrascan && \
     apk add --no-cache git openssh
 
-# create .ssh folder and change owner to terrascan
-RUN mkdir -p /home/terrascan/.ssh && \
-    chown -R terrascan:terrascan /home/terrascan/.ssh
+# create ~/.ssh & ~/bin folder and change owner to terrascan
+RUN mkdir -p /home/terrascan/.ssh /home/terrascan/bin && \
+    chown -R terrascan:terrascan /home/terrascan
 
 # run as non root user
 USER terrascan

diff --git a/pkg/iac-providers/kubernetes/v1/normalize.go b/pkg/iac-providers/kubernetes/v1/normalize.go
@@ -163,7 +163,7 @@ func readSkipRulesFromAnnotations(annotations map[string]interface{}, resourceID
 		skipRules := make([]output.SkipRule, 0)
 		err := json.Unmarshal([]byte(rules), &skipRules)
 		if err != nil {
-			zap.S().Debugf("terrascanSkip rules '%s' cannot be unmarshalled to json", rules)
+			zap.S().Debugf("json string %s cannot be unmarshalled to []output.SkipRules struct schema", rules)
 			return nil
 		}
 		return skipRules