fix(report): change a receiver of MarshalJSON

[knqyf263]

Description

A pointer receiver is used for MarshalJSON of some structs by mistake, causing some marshaling problems. If DetectedVulnerability is not a slice, custom MarshalJSON is not called.

PoC

package main

import (
	"encoding/json"
	"fmt"
)

type Status int

func (s *Status) MarshalJSON() ([]byte, error) {
	return json.Marshal("hello")
}

type Foo struct {
	Status Status
}

func main() {
	f1 := []Foo{{Status: 1}}
	jsonData, err := json.Marshal(f1)
	if err != nil {
		panic(err)
	}
	fmt.Println(string(jsonData))

	f2 := Foo{Status: 1}
	jsonData, err = json.Marshal(f2)
	if err != nil {
		panic(err)
	}
	fmt.Println(string(jsonData))
}

Related issues

• Close bug(template): incorrect JSON marshaling for some fields  #7464

Related PRs

• fix: do not use a pointer receiver for Status.MarshalJSON trivy-db#436

Checklist

• I've read the guidelines for contributing to this repository.
• I've followed the conventions in the PR title.
• I've added tests that prove my fix is effective or that my feature works.
• I've updated the documentation with the relevant information (if needed).
• I've added usage information (if the PR introduces new options)
• I've included a "before" and "after" example to the description (if the PR is a user interface change).

[DmitriyLewen]

LGTM

[DmitriyLewen]

@aqua-bot backport release/v0.55

[aqua-bot]

Backport PR created: #7490