fixup! Remove global API endpoint

mullvad-api/src/bin/relay_list.rs

@@ -2,13 +2,18 @@
 //! Used by the installer artifact packer to bundle the latest available
 //! relay list at the time of creating the installer.
 
-use mullvad_api::{proxy::ApiConnectionMode, rest::Error as RestError, RelayListProxy};
+use mullvad_api::{
+    proxy::ApiConnectionMode, rest::Error as RestError, ApiEndpoint, RelayListProxy,
+};
 use std::process;
 use talpid_types::ErrorExt;
 
 #[tokio::main]
 async fn main() {
-    let runtime = mullvad_api::Runtime::new(tokio::runtime::Handle::current());
+    let runtime = mullvad_api::Runtime::new(
+        tokio::runtime::Handle::current(),
+        &ApiEndpoint::from_env_vars(),
+    );
 
     let relay_list_request =
         RelayListProxy::new(runtime.mullvad_rest_handle(ApiConnectionMode::Direct.into_provider()))

mullvad-api/src/ffi/mod.rs

@@ -88,9 +88,8 @@ impl FfiClient {
 
         // It is imperative that the REST runtime is created within an async context, otherwise
         // ApiAvailability panics.
-        let api_runtime = tokio_runtime.block_on(async {
-            crate::Runtime::with_custom_endpoint(tokio_runtime.handle().clone(), &endpoint)
-        });
+        let api_runtime = tokio_runtime
+            .block_on(async { crate::Runtime::new(tokio_runtime.handle().clone(), &endpoint) });
 
         let context = FfiClient {
             tokio_runtime,

mullvad-api/src/lib.rs

@@ -349,20 +349,8 @@ pub enum Error {
 }
 
 impl Runtime {
-    /// Create a new `Runtime`.
-    /// Will either infer the API endpoint from environment variables, if set, otherwise will use
-    /// the default production endpoint.
-    pub fn new(handle: tokio::runtime::Handle) -> Self {
-        Self::with_custom_endpoint(
-            handle,
-            &ApiEndpoint::from_env_vars(),
-            #[cfg(target_os = "android")]
-            None,
-        )
-    }
-
     /// Will create a new Runtime without a cache with the provided API endpoint.
-    pub fn with_custom_endpoint(
+    pub fn new(
         handle: tokio::runtime::Handle,
         endpoint: &ApiEndpoint,
         #[cfg(target_os = "android")] socket_bypass_tx: Option<mpsc::Sender<SocketBypassRequest>>,
@@ -390,7 +378,7 @@ impl Runtime {
 
         #[cfg(feature = "api-override")]
         if endpoint.should_disable_address_cache() {
-            return Ok(Self::with_custom_endpoint(
+            return Ok(Self::new(
                 handle,
                 endpoint,
                 #[cfg(target_os = "android")]

test/test-manager/src/tests/account.rs

@@ -291,7 +291,7 @@ pub async fn new_device_client() -> anyhow::Result<DevicesProxy> {
     let endpoint = ApiEndpoint::new(api_host, api_address, false);
 
     let api =
-        mullvad_api::Runtime::with_custom_endpoint(tokio::runtime::Handle::current(), &endpoint);
+        mullvad_api::Runtime::new(tokio::runtime::Handle::current(), &endpoint);
 
     let rest_handle = api.mullvad_rest_handle(ApiConnectionMode::Direct.into_provider());
     Ok(DevicesProxy::new(rest_handle))