Skip to content

[Key Server] Reduce gas budget for the evaluation of Seal policies #417

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.

Sign up for GitHub

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

Open
benr-ml wants to merge 2 commits into
base: main
Choose a base branch
from
Open

[Key Server] Reduce gas budget for the evaluation of Seal policies #417

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
276a315
Reduce gas budget to 25M
benr-ml Jan 1, 2026
aca89a8
limit by computation units
benr-ml Jan 1, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 3 additions & 2 deletions crates/key-server/src/server.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -93,7 +93,7 @@ mod seal_package;
pub mod tests;
mod time;

const GAS_BUDGET: u64 = 500_000_000;
const MAX_COMPUTATION_UNITS: u64 = 55_000; // 50K tier + 10% extra buffer
const GIT_VERSION: &str = utils::git_version!();
const DEFAULT_PORT: u16 = 2024;

Expand Down Expand Up @@ -419,11 +419,12 @@ impl Server {
.add_staleness_check_to_ptb(self.options.allowed_staleness, vptb.ptb().clone());

// Evaluate the `seal_approve*` function
let gas_budget = MAX_COMPUTATION_UNITS * gas_price;
let tx_data = TransactionData::new_with_gas_coins(
TransactionKind::ProgrammableTransaction(ptb),
sender,
vec![], // Empty gas payment for dry run
GAS_BUDGET,
gas_budget,
gas_price,
);
let dry_run_res = self
Expand Down