-
Notifications
You must be signed in to change notification settings - Fork 0
Harden registryctl governed live evidence validation #462
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from all commits
1ffaf39
5d9b8ba
50dc91a
36459f2
c79e29e
dc862e5
c263b83
5e1a9f9
0422c74
ff46647
121be1b
3a4ff20
50d4197
623e189
0ce574c
ce9e33b
cc304b5
970b168
9b2ef8e
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -1378,6 +1378,7 @@ pub struct EvidenceFormat { | |
| } | ||
|
|
||
| #[derive(Debug, Clone, Serialize, Deserialize)] | ||
| #[serde(deny_unknown_fields)] | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This commit message has no AGENTS.md reference: AGENTS.md:L70-L70 Useful? React with 👍 / 👎. |
||
| pub struct ClaimResultView { | ||
| pub evaluation_id: String, | ||
| pub claim_id: String, | ||
|
|
@@ -1398,6 +1399,7 @@ pub struct ClaimResultView { | |
| } | ||
|
|
||
| #[derive(Debug, Clone, Serialize, Deserialize)] | ||
| #[serde(deny_unknown_fields)] | ||
| pub struct TargetRefView { | ||
| #[serde(rename = "type", default, skip_serializing_if = "String::is_empty")] | ||
| pub entity_type: String, | ||
|
|
@@ -1409,6 +1411,7 @@ pub struct TargetRefView { | |
| } | ||
|
|
||
| #[derive(Debug, Clone, Serialize, Deserialize)] | ||
| #[serde(deny_unknown_fields)] | ||
| pub struct EvidenceEntityRef { | ||
| #[serde(rename = "type")] | ||
| pub entity_type: String, | ||
|
|
@@ -1435,6 +1438,7 @@ pub const PROVENANCE_GENERATED_BY_CLAIM_EVALUATION: &str = "claim_evaluation"; | |
| /// Requester-side identity (client, actor, subject) is deliberately absent; | ||
| /// those live in restricted audit, never on the public wire. | ||
| #[derive(Debug, Clone, Serialize, Deserialize)] | ||
| #[serde(deny_unknown_fields)] | ||
| pub struct ClaimProvenance { | ||
| pub schema_version: String, | ||
| pub generated_by: ProvenanceGeneratedBy, | ||
|
|
@@ -1481,6 +1485,7 @@ impl ClaimProvenance { | |
| /// `policy_id` here names the *evaluation* policy under which the result was | ||
| /// produced. | ||
| #[derive(Debug, Clone, Serialize, Deserialize)] | ||
| #[serde(deny_unknown_fields)] | ||
| pub struct ProvenanceGeneratedBy { | ||
| #[serde(rename = "type")] | ||
| pub entry_type: String, | ||
|
|
@@ -1512,6 +1517,7 @@ pub struct ProvenanceGeneratedBy { | |
| /// The consumed side of a claim provenance record: how many Relay consultations | ||
| /// contributed to the claim. | ||
| #[derive(Debug, Clone, Serialize, Deserialize)] | ||
| #[serde(deny_unknown_fields)] | ||
| pub struct ProvenanceUsed { | ||
| pub relay_consultation_count: usize, | ||
| } | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This commit changes crate runtime/client validation, a docs-site tutorial, and release conformance materials together, which crosses three owning areas in one review unit. That makes it harder to apply the different verification gates and security-review notes for adopter evidence versus Notary model behavior; split the crate, docs, and release changes into scoped changes before merging.
AGENTS.md reference: AGENTS.md:L77-L78
Useful? React with 👍 / 👎.