From ab540e141e412626eeb53a18a7865e8e9b88013a Mon Sep 17 00:00:00 2001 From: dylan Date: Mon, 30 Jun 2025 13:54:06 -0600 Subject: [PATCH 1/3] adds debug logging for testing --- src/perms/builders.rs | 12 +++++++++++- 1 file changed, 11 insertions(+), 1 deletion(-) diff --git a/src/perms/builders.rs b/src/perms/builders.rs index 6c73fe3..7690245 100644 --- a/src/perms/builders.rs +++ b/src/perms/builders.rs @@ -170,7 +170,17 @@ impl Builders { /// round-robin design, where each builder is allowed to perform an action /// at a specific slot, and what builder is allowed changes with each slot. pub fn is_builder_permissioned(&self, sub: &str) -> Result<(), BuilderPermissionError> { - self.check_query_bounds()?; + let current_slot_time = self.calc().current_timepoint_within_slot(); + let current_builder = self.current_builder().sub; + + if current_slot_time < self.config.block_query_start() { + debug!(current_slot_time, current_builder, "block too early"); + return Err(BuilderPermissionError::ActionAttemptTooEarly); + } + if current_slot_time > self.config.block_query_cutoff() { + debug!(current_slot_time, current_builder, "block too late"); + return Err(BuilderPermissionError::ActionAttemptTooLate); + } if sub != self.current_builder().sub { tracing::debug!( From be9b3e26c0ad2f0ff90bf69fa6eacf27d69ac2bc Mon Sep 17 00:00:00 2001 From: dylan Date: Mon, 30 Jun 2025 13:59:40 -0600 Subject: [PATCH 2/3] add tracing import --- src/perms/builders.rs | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/perms/builders.rs b/src/perms/builders.rs index 7690245..dcdb2bc 100644 --- a/src/perms/builders.rs +++ b/src/perms/builders.rs @@ -174,11 +174,11 @@ impl Builders { let current_builder = self.current_builder().sub; if current_slot_time < self.config.block_query_start() { - debug!(current_slot_time, current_builder, "block too early"); + tracing::debug!(current_slot_time, current_builder, "block too early"); return Err(BuilderPermissionError::ActionAttemptTooEarly); } if current_slot_time > self.config.block_query_cutoff() { - debug!(current_slot_time, current_builder, "block too late"); + tracing::debug!(current_slot_time, current_builder, "block too late"); return Err(BuilderPermissionError::ActionAttemptTooLate); } From 22552d260f40f258159559bd61a2db753b71958b Mon Sep 17 00:00:00 2001 From: dylan Date: Mon, 30 Jun 2025 14:20:35 -0600 Subject: [PATCH 3/3] fix --- src/perms/builders.rs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/perms/builders.rs b/src/perms/builders.rs index dcdb2bc..8e0237a 100644 --- a/src/perms/builders.rs +++ b/src/perms/builders.rs @@ -171,7 +171,7 @@ impl Builders { /// at a specific slot, and what builder is allowed changes with each slot. pub fn is_builder_permissioned(&self, sub: &str) -> Result<(), BuilderPermissionError> { let current_slot_time = self.calc().current_timepoint_within_slot(); - let current_builder = self.current_builder().sub; + let current_builder = self.current_builder().sub.clone(); if current_slot_time < self.config.block_query_start() { tracing::debug!(current_slot_time, current_builder, "block too early");