fix(report): change a receiver of MarshalJSON

go.mod

@@ -26,7 +26,7 @@ require (
  github.com/aquasecurity/testdocker v0.0.0-20240730042311-4642e94c7fc8
  github.com/aquasecurity/tml v0.6.1
  github.com/aquasecurity/trivy-checks v0.13.1-0.20240830230553-53ddbbade784
- github.com/aquasecurity/trivy-db v0.0.0-20240718084044-d23a6ca8ba04
+ github.com/aquasecurity/trivy-db v0.0.0-20240910133327-7e0f4d2ed4c1
  github.com/aquasecurity/trivy-java-db v0.0.0-20240109071736-184bd7481d48
  github.com/aquasecurity/trivy-kubernetes v0.6.7-0.20240707095038-0300bc49b68b
  github.com/aws/aws-sdk-go-v2 v1.30.5
@@ -384,7 +384,7 @@ require (
  golang.org/x/sys v0.23.0 // indirect
  golang.org/x/telemetry v0.0.0-20240522233618-39ace7a40ae7 // indirect
  golang.org/x/time v0.6.0 // indirect
- golang.org/x/tools v0.23.0 // indirect
+ golang.org/x/tools v0.24.0 // indirect
  google.golang.org/api v0.172.0 // indirect
  google.golang.org/genproto v0.0.0-20240311173647-c811ad7063a7 // indirect
  google.golang.org/genproto/googleapis/api v0.0.0-20240701130421-f6361c86f094 // indirect

go.sum

@@ -349,8 +349,8 @@ github.com/aquasecurity/tml v0.6.1 h1:y2ZlGSfrhnn7t4ZJ/0rotuH+v5Jgv6BDDO5jB6A9gw
 github.com/aquasecurity/tml v0.6.1/go.mod h1:OnYMWY5lvI9ejU7yH9LCberWaaTBW7hBFsITiIMY2yY=
 github.com/aquasecurity/trivy-checks v0.13.1-0.20240830230553-53ddbbade784 h1:1rvPiCK8uQd3sarOuZ60nwksHpxsNdrvptz4eDW/V14=
 github.com/aquasecurity/trivy-checks v0.13.1-0.20240830230553-53ddbbade784/go.mod h1:Ralz7PWmR3LirHlXxVtUXc+7CFmWE82jbLk7+TPvV/0=
-github.com/aquasecurity/trivy-db v0.0.0-20240718084044-d23a6ca8ba04 h1:6/T8sFdNVG/AwOGoK6X55h7hF7LYqK8bsuPz8iEz8jM=
-github.com/aquasecurity/trivy-db v0.0.0-20240718084044-d23a6ca8ba04/go.mod h1:0T6oy2t1Iedt+yi3Ml5cpOYp5FZT4MI1/mx+3p+PIs8=
+github.com/aquasecurity/trivy-db v0.0.0-20240910133327-7e0f4d2ed4c1 h1:G0gnacAORRUqz2Tm5MqivSpldY2GZ74ijhJcMsae+sA=
+github.com/aquasecurity/trivy-db v0.0.0-20240910133327-7e0f4d2ed4c1/go.mod h1:PYkSRx4dlgFATEt+okGwibvbxVEtqsOdH+vX/saACYE=
 github.com/aquasecurity/trivy-java-db v0.0.0-20240109071736-184bd7481d48 h1:JVgBIuIYbwG+ekC5lUHUpGJboPYiCcxiz06RCtz8neI=
 github.com/aquasecurity/trivy-java-db v0.0.0-20240109071736-184bd7481d48/go.mod h1:Ldya37FLi0e/5Cjq2T5Bty7cFkzUDwTcPeQua+2M8i8=
 github.com/aquasecurity/trivy-kubernetes v0.6.7-0.20240707095038-0300bc49b68b h1:h7gsIzHyrxpQnayOuQI0kX7+8rVcqhV6G5bM3KVFyJU=
@@ -1806,8 +1806,8 @@ golang.org/x/tools v0.1.12/go.mod h1:hNGJHUnrk76NpqgfD5Aqm5Crs+Hm0VOH/i9J2+nxYbc
 golang.org/x/tools v0.6.0/go.mod h1:Xwgl3UAJ/d3gWutnCtw505GrjyAbvKui8lOU390QaIU=
 golang.org/x/tools v0.13.0/go.mod h1:HvlwmtVNQAhOuCjW7xxvovg8wbNq7LwfXh/k7wXUl58=
 golang.org/x/tools v0.17.0/go.mod h1:xsh6VxdV005rRVaS6SSAf9oiAqljS7UZUacMZ8Bnsps=
-golang.org/x/tools v0.23.0 h1:SGsXPZ+2l4JsgaCKkx+FQ9YZ5XEtA1GZYuoDjenLjvg=
-golang.org/x/tools v0.23.0/go.mod h1:pnu6ufv6vQkll6szChhK3C3L/ruaIv5eBeztNG8wtsI=
+golang.org/x/tools v0.24.0 h1:J1shsA93PJUEVaUSaay7UXAyE8aimq3GW0pjlolpa24=
+golang.org/x/tools v0.24.0/go.mod h1:YhNqVBIfWHdzvTLs0d8LCuMhkKUgSUKldakyV7W/WDQ=
 golang.org/x/vuln v1.1.3 h1:NPGnvPOTgnjBc9HTaUx+nj+EaUYxl5SJOWqaDYGaFYw=
 golang.org/x/vuln v1.1.3/go.mod h1:7Le6Fadm5FOqE9C926BCD0g12NWyhg7cxV4BwcPFuNY=
 golang.org/x/xerrors v0.0.0-20190717185122-a985d3407aa7/go.mod h1:I/5z698sn9Ka8TeJc9MKroUUfqBBauWjQqLJ2OPfmY0=

pkg/fanal/types/package.go

@@ -78,7 +78,7 @@ type PkgIdentifier struct {
 }
 
 // MarshalJSON customizes the JSON encoding of PkgIdentifier.
-func (id *PkgIdentifier) MarshalJSON() ([]byte, error) {
+func (id PkgIdentifier) MarshalJSON() ([]byte, error) {
  var p string
  if id.PURL != nil {
  p = id.PURL.String()
@@ -90,7 +90,7 @@ func (id *PkgIdentifier) MarshalJSON() ([]byte, error) {
  *Alias
  }{
  PURL: p,
- Alias: (*Alias)(id),
+ Alias: (*Alias)(&id),
  })
 }
 

pkg/report/template_test.go

@@ -6,11 +6,13 @@ import (
  "testing"
  "time"
 
+ "github.com/package-url/packageurl-go"
  "github.com/stretchr/testify/assert"
  "github.com/stretchr/testify/require"
 
  dbTypes "github.com/aquasecurity/trivy-db/pkg/types"
  "github.com/aquasecurity/trivy/pkg/clock"
+ ftypes "github.com/aquasecurity/trivy/pkg/fanal/types"
  "github.com/aquasecurity/trivy/pkg/report"
  "github.com/aquasecurity/trivy/pkg/types"
 )
@@ -156,6 +158,33 @@ func TestReportWriter_Template(t *testing.T) {
  template: `{{ $high := 0 }}{{ $critical := 0 }}{{ range . }}{{ range .Vulnerabilities}}{{ if eq .Severity "HIGH" }}{{ $high = add $high 1 }}{{ end }}{{ if eq .Severity "CRITICAL" }}{{ $critical = add $critical 1 }}{{ end }}{{ end }}Critical: {{ $critical }}, High: {{ $high }}{{ end }}`,
  expected: `Critical: 2, High: 1`,
  },
+ {
+ name: "custom JSON marshaler",
+ detectedVulns: []types.DetectedVulnerability{
+ {
+ VulnerabilityID: "CVE-2019-0000",
+ PkgName: "foo",
+ Status: dbTypes.StatusAffected,
+ PkgIdentifier: ftypes.PkgIdentifier{
+ PURL: &packageurl.PackageURL{
+ Type: packageurl.TypeNPM,
+ Name: "foobar",
+ Version: "1.2.3",
+ },
+ },
+ },
+ },
+ template: `{{ range . }}{{ range .Vulnerabilities}}{{ toPrettyJson . }}{{ end }}{{ end }}`,
+ expected: `{
+ "VulnerabilityID": "CVE-2019-0000",
+ "PkgName": "foo",
+ "PkgIdentifier": {
+ "PURL": "pkg:npm/foobar@1.2.3"
+ },
+ "Status": "affected",
+ "Layer": {}
+}`,
+ },
  {
  name: "happy path: env var parsing",
  detectedVulns: []types.DetectedVulnerability{},