smtan-gl
smtan-gl
@chen-keinan I've opened a [PR](https://github.com/aquasecurity/trivy/pull/7444) to propose a potential fix. Appreciate your feedback on whether this approach works 🙂
@chen-keinan I'm curious about the `RepoTags` field in `Metadata` being an array. Could you share in what scenario there might be multiple `RepoTags`? ``` { "ClusterName": "", "Findings": [ {...
> Hi @smtan-gl thanks for the contribution! it's really nice and important. > > I left some comments about this PR. > > also I took a look at #5889...
@afdesk I've updated the PR, could you have another look please 😄
> @smtan-gl could you sign CLA for a while? thanks! @afdesk, I've just signed it. Thank you.
@afdesk, the bug https://github.com/aquasecurity/trivy/issues/5889 occurs only with the `--report summary` configuration. In your example, you used `--report all` which [does not consolidate the report](https://github.com/aquasecurity/trivy/blob/35c4262d0b5ac18376a2ca53213b476103b5d51a/pkg/k8s/report/json.go#L23). As such, there are 2 reports...
Thanks for reviewing the PR @afdesk 😄 May I know what's the next steps needed to have this PR merged?
> > Thanks for reviewing the PR @afdesk 😄 May I know what's the next steps needed to have this PR merged? > > @smtan-gl thanks for your contribution again....
> Hi @smtan-gl > > there is a concern about this solution: how people associate the result with the metadata if it's N x N? > > wdyt about it?...
> Next ideas came from the point summary report basically is for reference and for table output. Thanks for the response @afdesk. I agree with your points. However, I assume...