fix tuf metadata marshalling (sigstore#447)

asraa · web-flow · commit dd886b869adf · 2021-10-07T16:49:40.000-04:00
Signed-off-by: Asra Ali &lt;asraa@google.com&gt;
diff --git a/pkg/pki/tuf/tuf.go b/pkg/pki/tuf/tuf.go
@@ -71,7 +71,7 @@ func (s Signature) CanonicalValue() ([]byte, error) {
 		return nil, fmt.Errorf("tuf manifest has not been initialized")
 	}
 	// TODO(asraa): Should the Signed payload be canonicalized?
-	canonical, err := cjson.Marshal(s)
+	canonical, err := cjson.Marshal(s.signed)
 	if err != nil {
 		return nil, err
 	}

Original file line number	Diff line number	Diff line change
`@@ -71,7 +71,7 @@ func (s Signature) CanonicalValue() ([]byte, error) {`
`71`	`71`	`return nil, fmt.Errorf("tuf manifest has not been initialized")`
`72`	`72`	`}`
`73`	`73`	`// TODO(asraa): Should the Signed payload be canonicalized?`
`74`		`- canonical, err := cjson.Marshal(s)`
	`74`	`+ canonical, err := cjson.Marshal(s.signed)`
`75`	`75`	`if err != nil {`
`76`	`76`	`return nil, err`
`77`	`77`	`}`