improvement: respect logout_devices param on password change

Move logout devices comment next to relevant loop

remove unnecessary log
merge-requests/15/head
Devon Hudson 4 years ago
parent 9d4c7c34a0
commit ebb38cd330
No known key found for this signature in database
GPG Key ID: CD06B18E77F6A628

@ -572,16 +572,16 @@ pub async fn change_password_route(
db.users.set_password(&sender_user, &body.new_password)?; db.users.set_password(&sender_user, &body.new_password)?;
// TODO: Read logout_devices field when it's available and respect that, currently not supported in Ruma if body.logout_devices {
// See: https://github.com/ruma/ruma/issues/107 // Logout all devices except the current one
// Logout all devices except the current one for id in db
for id in db .users
.users .all_device_ids(&sender_user)
.all_device_ids(&sender_user) .filter_map(|id| id.ok())
.filter_map(|id| id.ok()) .filter(|id| id != sender_device)
.filter(|id| id != sender_device) {
{ db.users.remove_device(&sender_user, &id)?;
db.users.remove_device(&sender_user, &id)?; }
} }
db.flush().await?; db.flush().await?;

Loading…
Cancel
Save