Ashna Mehrotra
Ashna Mehrotra
@codesage01 looks like @amanycodes has opened a PR in #1055
@MiahaCybersec could you try running "go mod tidy"?
LGTM, will wait to merge. @MiahaCybersec @RobertKielty and I discussed adding canary testing before merging large changes.
@MiahaCybersec we won't be prioritizing a specific internal testing pipeline for this right now. I think if we want to rebase this with new changes for now, the integration tests...
@robert-cronin when testing with the nginx:1.27.0 image locally tagged as ashnam/nginx:1.27.0, the patched image does not seem to have oci type https://oci.dag.dev/?image=ashnam%2Fnginx%3A1.27.0-patched
> https://oci.dag.dev/?image=ghcr.io%2Frobert-cronin%2Fnginx%3A1.27.0-patched Looks like its still `Content-Type: application/vnd.docker.distribution.manifest.list.v2+json` rather than `Content-Type: application/vnd.oci.image.index.v1+json"`
@robert-cronin its still showing docker for me, anything I could be doing wrong? https://oci.dag.dev/?image=ashnam%2Fnginx%3A1.27.0-patched. Im testing the pr via "gh pr checkout 949". Also do we need to change the...
> > @robert-cronin its still showing docker for me, anything I could be doing wrong? https://oci.dag.dev/?image=ashnam%2Fnginx%3A1.27.0-patched. Im testing the pr via "gh pr checkout 949". Also do we need to...
@robert-cronin yup it works now, thank you!
@codesage01 we have a PR up in #888 but it has been inactive for some time. @pradhans0906 would you like to continue the issue?