Set the default location set flag at login

If the user has managed to log in we unconditionally set
update_default_location = false. This is done to prevent an edge case
where the geolocation request completes after the user has logged in and
then the default location could change while the user is in the process
of clicking the select location/connect buttons.

1 files changed, 13 insertions, 0 deletions

diff --git a/mullvad-daemon/src/lib.rs b/mullvad-daemon/src/lib.rs
index 62c4e443ea..1825c09d95 100644
--- a/mullvad-daemon/src/lib.rs
+++ b/mullvad-daemon/src/lib.rs
@@ -1562,6 +1562,19 @@ impl Daemon {
                         error.display_chain_with_msg("Failed to update account history")
                     );
                 }
+                if self.settings.update_default_location {
+                    if let Err(e) = self
+                        .settings
+                        .update(move |settings| settings.update_default_location = false)
+                        .await
+                        .map_err(Error::SettingsError)
+                    {
+                        log::error!(
+                            "{}",
+                            e.display_chain_with_msg("Unable to save has_updated_default_country")
+                        );
+                    }
+                }
                 if *self.target_state == TargetState::Secured {
                     log::debug!("Initiating tunnel restart because the account number changed");
                     self.reconnect_tunnel();