Merge pull request #291 from accurics/bugfix/xml-output

fix xml output
tenable · Aug 18, 2020 · c44a663 · c44a663
2 parents 85e88e4 + 2719aac
commit c44a663
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/pkg/policy/types.go b/pkg/policy/types.go
@@ -1,6 +1,8 @@
 package policy
 
 import (
+	"encoding/xml"
+
 	"github.com/accurics/terrascan/pkg/iac-providers/output"
 	"github.com/accurics/terrascan/pkg/results"
 )
@@ -12,5 +14,6 @@ type EngineInput struct {
 
 // EngineOutput Contains data output from the engine
 type EngineOutput struct {
-	*results.ViolationStore `json:"results" yaml:"results" xml:"results,attr"`
+	XMLName                 xml.Name `json:"-" yaml:"-" xml:"results"`
+	*results.ViolationStore `json:"results" yaml:"results" xml:"results"`
 }
diff --git a/pkg/results/types.go b/pkg/results/types.go
@@ -42,6 +42,6 @@ type ViolationStats struct {
 
 // ViolationStore Storage area for violation data
 type ViolationStore struct {
-	Violations []*Violation   `json:"violations" yaml:"violations" xml:"violations,attr"`
-	Count      ViolationStats `json:"count" yaml:"count" xml:"count,attr"`
+	Violations []*Violation   `json:"violations" yaml:"violations" xml:"violations>violation"`
+	Count      ViolationStats `json:"count" yaml:"count" xml:"count"`
 }