Skip to content

Commit dcce3c9

Browse files
committed
fix: delete if multiple links
1 parent 005c428 commit dcce3c9

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

api/src/users/links.rs

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -177,7 +177,8 @@ async fn delete_link(
177177
let mut user_model = User::from_db(&user_id.to_string(), &app_state.dynamo)
178178
.await
179179
.unwrap();
180-
let existing_link = user_model.links.pop_if(|l| l.link_address == link_address);
180+
let pos = user_model.links.iter().position(|l| l.link_address == link_address);
181+
let existing_link = pos.map(|i| user_model.links.remove(i));
181182
info!("Deleting link for user {}: {}", user_id, link_address);
182183
if existing_link.is_none() {
183184
return Err(StatusCode::NOT_FOUND);

0 commit comments

Comments
 (0)