Skip to content

Commit e61dd1e

Browse files
committed
fix subject for oci artifact of attestation manifest
Signed-off-by: CrazyMax <[email protected]>
1 parent 2960a0e commit e61dd1e

File tree

2 files changed

+8
-2
lines changed

2 files changed

+8
-2
lines changed

client/client_test.go

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9613,7 +9613,9 @@ func testExportAttestations(t *testing.T, sb integration.Sandbox, ociArtifact bo
96139613
if ociArtifact {
96149614
subject := att.Manifest.Subject
96159615
require.NotNil(t, subject)
9616-
require.Equal(t, bases[i].Desc, *subject)
9616+
require.Equal(t, bases[i].Desc.MediaType, subject.MediaType)
9617+
require.Equal(t, bases[i].Desc.Digest, subject.Digest)
9618+
require.Equal(t, bases[i].Desc.Size, subject.Size)
96179619
require.Equal(t, "application/vnd.docker.attestation.manifest.v1+json", att.Manifest.ArtifactType)
96189620
require.Equal(t, ocispecs.DescriptorEmptyJSON, att.Manifest.Config)
96199621
} else {

exporter/containerimage/writer.go

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -617,7 +617,11 @@ func (ic *ImageWriter) commitAttestationsManifest(ctx context.Context, opts *Ima
617617

618618
if ociArtifact {
619619
mfst.ArtifactType = attestationManifestArtifactType
620-
mfst.Subject = &target
620+
mfst.Subject = &ocispecs.Descriptor{
621+
Digest: target.Digest,
622+
Size: target.Size,
623+
MediaType: target.MediaType,
624+
}
621625
}
622626

623627
labels := map[string]string{

0 commit comments

Comments
 (0)