diff --git a/crates/omnigraph-policy/src/lib.rs b/crates/omnigraph-policy/src/lib.rs index a98830a..84c7c05 100644 --- a/crates/omnigraph-policy/src/lib.rs +++ b/crates/omnigraph-policy/src/lib.rs @@ -428,8 +428,6 @@ impl PolicyEngine { pub fn authorize(&self, actor_id: &str, request: &PolicyRequest) -> Result { if !self.known_actors.contains(actor_id) { return Ok(self.deny( - actor_id, - request, None, format!( "policy denied action '{}' for unknown actor '{}'", @@ -503,16 +501,11 @@ impl PolicyEngine { .unwrap_or_default(), actor_id ); - self.deny(actor_id, request, matched_rule_id, message) + self.deny(matched_rule_id, message) } }) } - pub fn validate_request(&self, actor_id: &str, request: &PolicyRequest) -> Result<()> { - let _ = self.authorize(actor_id, request)?; - Ok(()) - } - pub fn run_tests(&self, tests: &PolicyTestConfig) -> Result<()> { if tests.version != 1 { bail!("policy test version must be 1"); @@ -548,13 +541,7 @@ impl PolicyEngine { self.known_actors.len() } - fn deny( - &self, - _actor_id: &str, - _request: &PolicyRequest, - matched_rule_id: Option, - message: String, - ) -> PolicyDecision { + fn deny(&self, matched_rule_id: Option, message: String) -> PolicyDecision { PolicyDecision { allowed: false, matched_rule_id,