Add error context on cert validation

Summary:
Add some additional error context when cert check fails instead of overwriting it.

Reason #1 is that our cert check could be wrong, so if it is at least we show the actual error

Reason #2 If #1 isn't true it lets us collect some data to see what errors caused by invalid certs would look like, which could be useful

Reviewed By: JakobDegen

Differential Revision: D59988439

fbshipit-source-id: a2355ea4ac39eaa1f55a431c5fbc951fa63b62f0

app/buck2_client_ctx/src/daemon/client/connect.rs

@@ -639,7 +639,7 @@ impl<'a> BuckdConnectOptions<'a> {
         {
             Ok(client) => Ok(client.with_subscribers(self.subscribers)),
             Err(e) => {
-                handle.await.unwrap()?;
+                handle.await.unwrap().context("Daemon Failed to Connect")?;
                 self.subscribers.handle_daemon_connection_failure(&e);
                 Err(e.into())
             }

app/buck2_file_watcher/src/watchman/core.rs

@@ -152,11 +152,11 @@ async fn with_timeout<R>(
     match tokio::time::timeout(Duration::from_secs(timeout), fut).await {
         Ok(Ok(res)) => Ok(res),
         Ok(Err(e)) => {
-            validate_certs().await?;
+            validate_certs().await.context("Watchman Request Failed")?;
             Err(WatchmanClientError::RequestFailed(e).into())
         }
         Err(_) => {
-            validate_certs().await?;
+            validate_certs().await.context("Watchman Timed Out")?;
             Err(WatchmanClientError::Timeout(timeout).into())
         }
     }