Skip to content

Commit

Permalink
fix: remove redundant executor attribute on reveal event
Browse files Browse the repository at this point in the history
We only need the public key source, the message sender is not relevant.
  • Loading branch information
Thomasvdam committed Jul 22, 2024
1 parent 53bbcea commit 9192a85
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions contract/src/msgs/data_requests/execute/reveal_result.rs
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ use crate::state::CHAIN_ID;
impl ExecuteHandler for execute::reveal_result::Execute {
/// Posts a data result of a data request with an attached result.
/// This removes the data request from the pool and creates a new entry in the data results.
fn execute(self, deps: DepsMut, env: Env, info: MessageInfo) -> Result<Response, ContractError> {
fn execute(self, deps: DepsMut, env: Env, _info: MessageInfo) -> Result<Response, ContractError> {
// find the data request from the committed pool (if it exists, otherwise error)
let dr_id = Hash::from_hex_str(&self.dr_id)?;
let mut dr = state::load_request(deps.storage, &dr_id)?;
Expand Down Expand Up @@ -52,7 +52,6 @@ impl ExecuteHandler for execute::reveal_result::Execute {
Event::new("seda-reveal").add_attributes([
("dr_id", self.dr_id.clone()),
("reveal", to_json_string(&self.reveal_body)?),
("executor", info.sender.into_string()),
("stdout", to_json_string(&self.stdout)?),
("stderr", to_json_string(&self.stderr)?),
("executor", self.public_key.to_string()),
Expand Down

0 comments on commit 9192a85

Please sign in to comment.