Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BUG] fix openvex report oci id #667

Open
1 task
sozercan opened this issue Jun 17, 2024 · 4 comments · May be fixed by #810
Open
1 task

[BUG] fix openvex report oci id #667

sozercan opened this issue Jun 17, 2024 · 4 comments · May be fixed by #810
Labels
bug Something isn't working good first issue Good for newcomers

Comments

@sozercan
Copy link
Member

sozercan commented Jun 17, 2024

Version of copa

No response

Expected Behavior

it should be:

"@id": "pkg:oci/nginx@sha256....",

https://github.com/package-url/purl-spec/blob/master/PURL-TYPES.rst#oci

However, I am not sure if we can do this since image is not at the registry at this time so sha256 will not be accurate. tag might be more suitable?

Actual Behavior

vex statement output has

"@id": "pkg:oci/docker.io/library/nginx:1.21.6-patched",

https://project-copacetic.github.io/copacetic/website/output

Steps To Reproduce

No response

Are you willing to submit PRs to contribute to this bug fix?

  • Yes, I am willing to implement it.
@sozercan sozercan added the bug Something isn't working label Jun 17, 2024
@sozercan sozercan changed the title [BUG] fix openvex report [BUG] fix openvex report oci id Jun 17, 2024
@sozercan sozercan added the good first issue Good for newcomers label Oct 10, 2024
@shubhamsugara22
Copy link

@sozercan looks a good first issue to work on this repo but will need more info ?
can i try this
Thanks

@prakrit55
Copy link
Contributor

@sozercan, can image be an image digest if yes then we can modify the patchWithContext( ) there.
@shubhamsugara22, I am also on it

imageName, err := reference.ParseNormalizedNamed(image)

@shubhamsugara22
Copy link

@prakrit55 Didn't see any comment , that's fine , will also update if found any helpful info , also looking to collaborate on this one or flying solo?

@prakrit55
Copy link
Contributor

@prakrit55 Didn't see any comment , that's fine , will also update if found any helpful info , also looking to collaborate on this one or flying solo?

@shubhamsugara22, you can work into it, I will go find another one 😄

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working good first issue Good for newcomers
Projects
Status: 🆕 New
Development

Successfully merging a pull request may close this issue.

3 participants