smtan-gl

Results 10 comments of 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...