4nk/sdk_client
5
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Add simple duplicate state check on update_process

Browse Source
This commit is contained in:
Sosthene 2025-09-07 18:32:29 +02:00
parent cf31bad168
commit 12f0fb23eb
1 changed files with 4 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
src/api.rs
View File

@ -1303,19 +1303,10 @@ pub fn update_process(
roles.clone() roles.clone()
)?; )?;
// TODO duplicate check is broken // We check that the new state doesn't already exist
// // We compare the new state with the previous one if process.get_state_for_id(&new_state.state_id).is_ok() {
// let last_state_merkle_root = &prev_state.state_id; return Err(ApiError::new("New state already known".to_owned()));
}
// if *last_state_merkle_root == new_state.state_id {
// return Err(ApiError::new("new proposed state is identical to the previous commited state".to_owned()));
// }
// // We check that we don't have already a similar concurrent state
// let concurrent_processes = process.get_latest_concurrent_states()?;
// if concurrent_processes.iter().any(|p| p.state_id == new_state.state_id) {
// return Err(ApiError::new("New state already known".to_owned()));
// }
let diffs = create_diffs(&lock_local_device()?, &process, &new_state, &members_list)?; let diffs = create_diffs(&lock_local_device()?, &process, &new_state, &members_list)?;