libsql: Improve sync protocol probe tracing

Author: penberg

libsql/src/database/builder.rs

@@ -6,6 +6,8 @@ use crate::{Database, Result};
 
 use super::DbType;
 
+use tracing::trace;
+
 /// A builder for [`Database`]. This struct can be used to build
 /// all variants of [`Database`]. These variants include:
 ///
@@ -363,6 +365,7 @@ cfg_replication! {
                 use super::SyncProtocol;
                 match sync_protocol {
                     p @ (SyncProtocol::Auto | SyncProtocol::V2) => {
+                        trace!("Probing for sync protocol version for {}", url);
                         let client = hyper::client::Client::builder()
                             .build::<_, hyper::Body>(connector.clone());
 
@@ -381,6 +384,8 @@ cfg_replication! {
                             .await
                             .map_err(|err| crate::Error::Sync(err.into()))?;
 
+                        trace!("Probe for sync protocol version for {} returned status {}", url, res.status());
+
                         if res.status() == http::StatusCode::UNAUTHORIZED {
                             return Err(crate::Error::Sync("Unauthorized".into()));
                         }
@@ -397,15 +402,18 @@ cfg_replication! {
                         }
 
                         if res.status().is_success() {
+                            trace!("Using sync protocol v2 for {}", url);
                             return Builder::new_synced_database(path, url, auth_token)
                                 .remote_writes(true)
                                 .read_your_writes(read_your_writes)
                                 .build()
                                 .await;
                         }
-
+                        trace!("Using sync protocol v1 for {} based on probe results", url);
+                    }
+                    SyncProtocol::V1 => {
+                        trace!("Using sync protocol v1 for {}", url);
                     }
-                    SyncProtocol::V1 => {}
                 }
             }