Skip to content

Commit

Permalink
Add comment that integration time is only trustworthy with an SET (#442)
Browse files Browse the repository at this point in the history
  • Loading branch information
haydentherapper authored Dec 10, 2024
1 parent 0bb3807 commit ad0a758
Show file tree
Hide file tree
Showing 10 changed files with 20 additions and 6 deletions.
2 changes: 1 addition & 1 deletion gen/jsonschema/schemas/Bundle.schema.json
Original file line number Diff line number Diff line change
Expand Up @@ -314,7 +314,7 @@
},
"integratedTime": {
"type": "string",
"description": "The UNIX timestamp from the log when the entry was persisted."
"description": "The UNIX timestamp from the log when the entry was persisted. The integration time MUST NOT be trusted if inclusion_promise is omitted."
},
"inclusionPromise": {
"$ref": "#/definitions/dev.sigstore.rekor.v1.InclusionPromise",
Expand Down
2 changes: 1 addition & 1 deletion gen/jsonschema/schemas/Input.schema.json
Original file line number Diff line number Diff line change
Expand Up @@ -507,7 +507,7 @@
},
"integratedTime": {
"type": "string",
"description": "The UNIX timestamp from the log when the entry was persisted."
"description": "The UNIX timestamp from the log when the entry was persisted. The integration time MUST NOT be trusted if inclusion_promise is omitted."
},
"inclusionPromise": {
"$ref": "#/definitions/dev.sigstore.rekor.v1.InclusionPromise",
Expand Down
2 changes: 1 addition & 1 deletion gen/jsonschema/schemas/TransparencyLogEntry.schema.json
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@
},
"integratedTime": {
"type": "string",
"description": "The UNIX timestamp from the log when the entry was persisted."
"description": "The UNIX timestamp from the log when the entry was persisted. The integration time MUST NOT be trusted if inclusion_promise is omitted."
},
"inclusionPromise": {
"$ref": "#/definitions/dev.sigstore.rekor.v1.InclusionPromise",
Expand Down
2 changes: 1 addition & 1 deletion gen/jsonschema/schemas/VerificationMaterial.schema.json
Original file line number Diff line number Diff line change
Expand Up @@ -231,7 +231,7 @@
},
"integratedTime": {
"type": "string",
"description": "The UNIX timestamp from the log when the entry was persisted."
"description": "The UNIX timestamp from the log when the entry was persisted. The integration time MUST NOT be trusted if inclusion_promise is omitted."
},
"inclusionPromise": {
"$ref": "#/definitions/dev.sigstore.rekor.v1.InclusionPromise",
Expand Down
2 changes: 2 additions & 0 deletions gen/pb-go/rekor/v1/sigstore_rekor.pb.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Original file line number Diff line number Diff line change
Expand Up @@ -127,6 +127,8 @@ pub struct TransparencyLogEntry {
#[prost(message, optional, tag = "3")]
pub kind_version: ::core::option::Option<KindVersion>,
/// The UNIX timestamp from the log when the entry was persisted.
/// The integration time MUST NOT be trusted if inclusion_promise
/// is omitted.
#[prost(int64, tag = "4")]
pub integrated_time: i64,
/// The inclusion promise/signed entry timestamp from the log.
Expand Down
Binary file not shown.
6 changes: 5 additions & 1 deletion gen/pb-typescript/src/__generated__/sigstore_rekor.ts

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 2 additions & 0 deletions protos/sigstore_rekor.proto
Original file line number Diff line number Diff line change
Expand Up @@ -101,6 +101,8 @@ message TransparencyLogEntry {
// verification.
KindVersion kind_version = 3 [(google.api.field_behavior) = REQUIRED];
// The UNIX timestamp from the log when the entry was persisted.
// The integration time MUST NOT be trusted if inclusion_promise
// is omitted.
int64 integrated_time = 4 [(google.api.field_behavior) = REQUIRED];
// The inclusion promise/signed entry timestamp from the log.
// Required for v0.1 bundles, and MUST be verified.
Expand Down

0 comments on commit ad0a758

Please sign in to comment.