diff options
| author | David Lönnhager <david.l@mullvad.net> | 2022-03-14 12:08:10 +0100 |
|---|---|---|
| committer | David Lönnhager <david.l@mullvad.net> | 2022-03-14 12:08:57 +0100 |
| commit | 3e1271777fd7556a76abc582bd3c44356ecbd15a (patch) | |
| tree | bc03c4027aad5c47f00dfa4c1fb3584dff4d1add | |
| parent | 92b5f952ce754e00b4f9968da5ddfbe745900bee (diff) | |
| download | mullvadvpn-3e1271777fd7556a76abc582bd3c44356ecbd15a.tar.xz mullvadvpn-3e1271777fd7556a76abc582bd3c44356ecbd15a.zip | |
Remove AccountManager::set_inner
| -rw-r--r-- | mullvad-daemon/src/device.rs | 15 |
1 files changed, 5 insertions, 10 deletions
diff --git a/mullvad-daemon/src/device.rs b/mullvad-daemon/src/device.rs index 6009455a63..868dc003d8 100644 --- a/mullvad-daemon/src/device.rs +++ b/mullvad-daemon/src/device.rs @@ -313,7 +313,7 @@ impl AccountManager { let _ = tx.send(self.logout().await); } AccountManagerCommand::SetData(data, tx) => { - let _ = tx.send(self.set(data).await); + let _ = tx.send(self.set(InnerDeviceEvent::Login(data)).await); } AccountManagerCommand::GetData(tx) => { let _ = tx.send(Ok(self.data.clone())); @@ -340,7 +340,7 @@ impl AccountManager { async fn login(&mut self, token: AccountToken) -> Result<(), Error> { let data = self.device_service.generate_for_account(token).await?; - self.set(data).await?; + self.set(InnerDeviceEvent::Login(data)).await?; Ok(()) } @@ -375,12 +375,7 @@ impl AccountManager { }) } - #[inline] - async fn set(&mut self, new_data: DeviceData) -> Result<(), Error> { - self.set_inner(InnerDeviceEvent::Login(new_data)).await - } - - async fn set_inner(&mut self, event: InnerDeviceEvent) -> Result<(), Error> { + async fn set(&mut self, event: InnerDeviceEvent) -> Result<(), Error> { let data = event.data(); if data == self.data.as_ref() { return Ok(()); @@ -421,7 +416,7 @@ impl AccountManager { let mut new_data = data.clone(); new_data.device.pubkey = wg_data.private_key.public_key(); new_data.wg_data = wg_data; - self.set_inner(InnerDeviceEvent::RotatedKey(new_data)).await + self.set(InnerDeviceEvent::RotatedKey(new_data)).await } /// Check if the device is valid for the account, and yank it if it no longer exists. @@ -453,7 +448,7 @@ impl AccountManager { device, ..data.clone() }; - self.set_inner(InnerDeviceEvent::Updated(new_data)).await?; + self.set(InnerDeviceEvent::Updated(new_data)).await?; Ok(ValidationResult::Updated) } } else { |
