add better error message for non-configured api key

Author: gschoeni

src/cli/src/helpers.rs

@@ -66,10 +66,8 @@ pub async fn check_remote_version_blocking(host: impl AsRef<str>) -> Result<(),
                 ).into()));
             }
         }
-        Err(_) => {
-            return Err(OxenError::basic_str(
-                "Error: unable to verify remote version",
-            ));
+        Err(err) => {
+            return Err(err);
         }
     }
     Ok(())

src/lib/src/api/client.rs

@@ -77,7 +77,7 @@ fn builder_for_host<S: AsRef<str>>(
         Err(err) => {
             log::debug!("remote::client::new_for_host error getting config: {}", err);
 
-            return Ok(builder);
+            return Err(OxenError::must_supply_valid_api_key());
         }
     };
     if let Some(auth_token) = config.auth_token_for_host(host.as_ref()) {

src/lib/src/api/client/repositories.rs

@@ -114,6 +114,7 @@ pub async fn get_by_remote(remote: &Remote) -> Result<Option<RemoteRepository>,
     let client = client::new_for_url(&url)?;
     match client.get(&url).send().await {
         Ok(res) => {
+            log::debug!("get_by_remote status: {}", res.status());
             if 404 == res.status() {
                 return Ok(None);
             }