Skip to content

Commit c0c44db

Browse files
edit the fields before instantiating the bundle
Signed-off-by: Ramon Petgrave <ramon.petgrave64@gmail.com>
1 parent f47ef16 commit c0c44db

File tree

1 file changed

+7
-5
lines changed

1 file changed

+7
-5
lines changed

test/unit/verify/test_verifier.py

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,7 @@
1414

1515

1616
import hashlib
17+
import json
1718
import logging
1819
from datetime import datetime, timezone
1920

@@ -24,7 +25,7 @@
2425
from sigstore._internal.trust import CertificateAuthority
2526
from sigstore.dsse import StatementBuilder, Subject
2627
from sigstore.errors import VerificationError
27-
from sigstore.models import Bundle, LogEntry
28+
from sigstore.models import Bundle
2829
from sigstore.verify import policy
2930
from sigstore.verify.verifier import Verifier
3031

@@ -240,11 +241,12 @@ def test_vierifier_verify_no_inclusion_promise_and_integrated_time(
240241
There is one exception: When inclusionPromise is present, but integratedTime is not, then we expect a failure
241242
because the integratedTime is required to verify the inclusionPromise.
242243
"""
243-
bundle = Bundle.from_json(asset("tsa/bundle.txt.sigstore").read_bytes())
244-
_dict = bundle.log_entry._to_rekor().to_dict()
244+
bundle_dict = json.loads(asset("tsa/bundle.txt.sigstore").read_bytes())
245+
(entry_dict,) = bundle_dict["verificationMaterial"]["tlogEntries"]
245246
for field in fields_to_delete:
246-
del _dict[field]
247-
bundle._log_entry = LogEntry._from_dict_rekor(_dict)
247+
del entry_dict[field]
248+
# Bundle.from_json() also validates the bundle's layout.
249+
bundle = Bundle.from_json(json.dumps(bundle_dict))
248250
verifier.verify_artifact(
249251
asset("tsa/bundle.txt").read_bytes(),
250252
bundle,

0 commit comments

Comments
 (0)