Avoid caching wheel fetches

crates/puffin-cli/src/commands/compile.rs

@@ -2,9 +2,9 @@ use std::path::Path;
 use std::str::FromStr;
 
 use anyhow::Result;
-use puffin_client::PypiClientBuilder;
 use tracing::debug;
 
+use puffin_client::PypiClientBuilder;
 use puffin_interpreter::PythonExecutable;
 use puffin_platform::tags::Tags;
 use puffin_platform::Platform;

crates/puffin-cli/src/commands/install.rs

@@ -3,10 +3,10 @@ use std::str::FromStr;
 
 use anyhow::Result;
 use async_std::fs::File;
+use install_wheel_rs::{install_wheel, InstallLocation};
 use tracing::debug;
 use url::Url;
 
-use install_wheel_rs::{install_wheel, InstallLocation};
 use puffin_client::PypiClientBuilder;
 use puffin_interpreter::PythonExecutable;
 use puffin_platform::tags::Tags;

crates/puffin-client/src/api.rs

@@ -111,8 +111,7 @@ impl PypiClient {
         url: &Url,
     ) -> Result<Box<dyn AsyncRead + Unpin + Send + Sync>, PypiClientError> {
         Ok(Box::new(
-            // TODO(charlie): Use an uncached client.
+            self.uncached_client
-            self.client
                 .get(url.to_string())
                 .send()
                 .await?

crates/puffin-client/src/client.rs

@@ -66,7 +66,7 @@ impl PypiClientBuilder {
         let retry_strategy = RetryTransientMiddleware::new_with_policy(retry_policy);
 
         let mut client_builder =
-            reqwest_middleware::ClientBuilder::new(client_raw).with(retry_strategy);
+            reqwest_middleware::ClientBuilder::new(client_raw.clone()).with(retry_strategy);
 
         if let Some(path) = self.cache {
             client_builder = client_builder.with(Cache(HttpCache {
@@ -76,10 +76,17 @@ impl PypiClientBuilder {
             }));
         }
 
+        let retry_policy = ExponentialBackoff::builder().build_with_max_retries(self.retries);
+        let retry_strategy = RetryTransientMiddleware::new_with_policy(retry_policy);
+
+        let uncached_client_builder =
+            reqwest_middleware::ClientBuilder::new(client_raw).with(retry_strategy);
+
         PypiClient {
             registry: Arc::new(self.registry),
             proxy: Arc::new(self.proxy),
             client: client_builder.build(),
+            uncached_client: uncached_client_builder.build(),
         }
     }
 }
@@ -89,4 +96,5 @@ pub struct PypiClient {
     pub(crate) registry: Arc<Url>,
     pub(crate) proxy: Arc<Url>,
     pub(crate) client: ClientWithMiddleware,
+    pub(crate) uncached_client: ClientWithMiddleware,
 }

crates/puffin-package/src/wheel.rs

@@ -1,8 +1,9 @@
 use std::str::FromStr;
 
-use puffin_platform::tags::Tags;
 use thiserror::Error;
 
+use puffin_platform::tags::Tags;
+
 #[derive(Debug, Clone, Eq, PartialEq)]
 pub struct WheelFilename {
     pub distribution: String,

crates/puffin-resolve/src/lib.rs

@@ -1,5 +1,4 @@
 use std::collections::{HashMap, HashSet};
-
 use std::str::FromStr;
 
 use anyhow::Result;