Skip to content

Commit

Permalink
wip: demo hacking
Browse files Browse the repository at this point in the history
  • Loading branch information
mikhailswift committed Nov 3, 2023
1 parent 57ecdaf commit f617e8b
Show file tree
Hide file tree
Showing 3 changed files with 94 additions and 8 deletions.
86 changes: 86 additions & 0 deletions attestation/policyverify/policyverify.go
Original file line number Diff line number Diff line change
Expand Up @@ -19,11 +19,13 @@ import (
"crypto/x509"
"encoding/json"
"fmt"
"strings"
"time"

"github.com/testifysec/go-witness/attestation"
"github.com/testifysec/go-witness/cryptoutil"
"github.com/testifysec/go-witness/dsse"
"github.com/testifysec/go-witness/intoto"
"github.com/testifysec/go-witness/log"
"github.com/testifysec/go-witness/policy"
"github.com/testifysec/go-witness/slsa"
Expand All @@ -48,6 +50,7 @@ type Attestor struct {
policyVerifiers []cryptoutil.Verifier
collectionSource source.Sourcer
subjectDigests []string
addtlSubjects map[string]cryptoutil.DigestSet
}

type Option func(*Attestor)
Expand Down Expand Up @@ -110,6 +113,10 @@ func (a *Attestor) Subjects() map[string]cryptoutil.DigestSet {
}

subjects[fmt.Sprintf("policy:%v", a.VerificationSummary.Policy.URI)] = a.VerificationSummary.Policy.Digest
for name, ds := range a.addtlSubjects {
subjects[name] = ds
}

return subjects
}

Expand Down Expand Up @@ -178,9 +185,88 @@ func (a *Attestor) Attest(ctx *attestation.AttestationContext) error {
return fmt.Errorf("failed to generate verification summary: %w", err)
}

a.findRelevantSubjects(policyResult.EvidenceByStep)
return nil
}

// findRelevantSubjects will find any image tags in attestations that passed our policy check.
// we do this by searching the subject digests the user tested the policy against, and if we find
// a collection with OCI subjects for an imageid that matches one of the provided subject digests,
// we grab the imagetag subjects off of that attestation.
// todo: we need a better solution for this
func (a *Attestor) findRelevantSubjects(evidenceByStep map[string][]source.VerifiedCollection) {
const imageTagSubjectPrefix = "https://witness.dev/attestations/oci/v0.1/imagetag:"
const imageIdSubjectPrefix = "https://witness.dev/attestations/oci/v0.1/imageid:"
const githubProjectPrefix = "https://witness.dev/attestations/github/v0.1/projecturl:"
const gitlabProjectPrefix = "https://witness.dev/attestations/gitlab/v0.1/projecturl:"

for _, collections := range evidenceByStep {
for _, collection := range collections {
candidates := make([]intoto.Subject, 0)
matchedSubject := false

// search through every subject on the in-toto statment. if we find imagetags, we set them aside as possible candidates.
// if we find an imageid subject that matches, we consider all the candidates to be matching subjects and return them
for _, subject := range collection.Statement.Subject {
// if we find an image tag subject, add it to the list of candidates
if strings.HasPrefix(subject.Name, imageTagSubjectPrefix) {
candidates = append(candidates, intoto.Subject{
Name: fmt.Sprintf("imagetag:%v", strings.TrimPrefix(subject.Name, imageTagSubjectPrefix)),
Digest: subject.Digest,
})
}

if strings.HasPrefix(subject.Name, githubProjectPrefix) {
candidates = append(candidates, intoto.Subject{
Name: fmt.Sprintf("projecturl:%v", strings.TrimPrefix(subject.Name, githubProjectPrefix)),
Digest: subject.Digest,
})
}

if strings.HasPrefix(subject.Name, gitlabProjectPrefix) {
candidates = append(candidates, intoto.Subject{
Name: fmt.Sprintf("projecturl:%v", strings.TrimPrefix(subject.Name, gitlabProjectPrefix)),
Digest: subject.Digest,
})
}

// if we find an imageid subject, check to see if any the digests we verified match the imageid
if strings.HasPrefix(subject.Name, imageIdSubjectPrefix) {
for _, imageIdDigest := range subject.Digest {
for _, testImageIdDigest := range a.subjectDigests {
if imageIdDigest == testImageIdDigest {
matchedSubject = true
}
}

// if we found a matching imageid subject with one of our test subject digests, stop looking
if matchedSubject {
break
}
}
}
}

// after we've checked all the subjects, if we found a match, add our candidates to our additional subjects
if matchedSubject {
for _, candidate := range candidates {
ds := cryptoutil.DigestSet{}
for hash, value := range candidate.Digest {
digestValue, err := cryptoutil.DigestValueFromString(hash)
if err != nil {
continue
}

ds[digestValue] = value
}

a.addtlSubjects[candidate.Name] = ds
}
}
}
}
}

func verificationSummaryFromResults(ctx *attestation.AttestationContext, policyEnvelope dsse.Envelope, policyResult policy.PolicyResult, accepted bool) (slsa.VerificationSummary, error) {
inputAttestations := make([]slsa.ResourceDescriptor, 0, len(policyResult.EvidenceByStep))
for _, input := range policyResult.EvidenceByStep {
Expand Down
14 changes: 7 additions & 7 deletions cryptoutil/digestset.go
Original file line number Diff line number Diff line change
Expand Up @@ -85,20 +85,20 @@ func (dv DigestValue) New() hash.Hash {

type DigestSet map[DigestValue]string

func HashToString(h crypto.Hash) (string, error) {
if name, ok := hashNames[DigestValue{Hash: h}]; ok {
func DigestValueToString(dv DigestValue) (string, error) {
if name, ok := hashNames[dv]; ok {
return name, nil
}

return "", ErrUnsupportedHash(h.String())
return "", ErrUnsupportedHash(dv.Hash.String())
}

func HashFromString(name string) (crypto.Hash, error) {
if hash, ok := hashesByName[name]; ok {
return hash.Hash, nil
func DigestValueFromString(name string) (DigestValue, error) {
if dv, ok := hashesByName[name]; ok {
return dv, nil
}

return crypto.Hash(0), ErrUnsupportedHash(name)
return DigestValue{}, ErrUnsupportedHash(name)
}

// Equal returns true if every digest for hash functions both artifacts have in common are equal.
Expand Down
2 changes: 1 addition & 1 deletion cryptoutil/gitoid.go
Original file line number Diff line number Diff line change
Expand Up @@ -61,7 +61,7 @@ func (gh *gitoidHasher) Reset() {

// Size returns the number of bytes Sum will return.
func (gh *gitoidHasher) Size() int {
hashName, err := HashToString(gh.hash)
hashName, err := DigestValueToString(DigestValue{Hash: gh.hash})
if err != nil {
return 0
}
Expand Down

0 comments on commit f617e8b

Please sign in to comment.