fix: retry and attempt with errors (#514)

* fix: always delay before next request

* change approach

* bump

Author: leanmendoza

godot/export_presets.cfg

@@ -394,7 +394,7 @@ architectures/armeabi-v7a=false
 architectures/arm64-v8a=true
 architectures/x86=false
 architectures/x86_64=true
-version/code=16
+version/code=17
 version/name="1.0"
 package/unique_name="org.decentraland.godotexplorer"
 package/name="Decentraland"

lib/src/auth/decentraland_auth_server.rs

@@ -56,13 +56,22 @@ async fn fetch_polling_server(
 ) -> Result<(String, serde_json::Value), anyhow::Error> {
     let url = format!("{AUTH_SERVER_ENDPOINT_URL}/{req_id}");
     let mut attempt = 0;
+    let mut requested_time = std::time::Instant::now();
+
     loop {
         tracing::debug!("trying req_id {:?} attempt ${attempt}", req_id);
         if attempt >= AUTH_SERVER_RETRIES {
             return Err(anyhow::Error::msg("too many atempts"));
         }
         attempt += 1;
 
+        let diff = (std::time::Instant::now() - requested_time).as_millis() as i64;
+        let remaining_delay = (AUTH_SERVER_RETRY_INTERVAL.as_millis() as i64) - diff;
+        if remaining_delay > 0 {
+            tokio::time::sleep(Duration::from_millis(remaining_delay as u64)).await;
+        }
+
+        requested_time = std::time::Instant::now();
         let response = reqwest::Client::builder()
             .timeout(AUTH_SERVER_REQUEST_TIMEOUT)
             .build()
@@ -74,7 +83,6 @@ async fn fetch_polling_server(
         let response = match response {
             Ok(response) => {
                 if response.status().as_u16() == 204 {
-                    tokio::time::sleep(AUTH_SERVER_RETRY_INTERVAL).await;
                     continue;
                 } else if response.status().is_success() {
                     match response.json::<RequestResult>().await {

lib/src/dcl/js/runtime.rs

@@ -46,7 +46,7 @@ fn op_get_file_url(
         });
     }
 
-    Err(anyhow!("not found"))
+    Err(anyhow!("not found {filename}"))
 }
 
 #[op]