Address review comments

Author: declark1

src/clients.rs

@@ -116,7 +116,7 @@ pub async fn create_http_clients(
                     panic!("error reading cert from {cert_path:?}: {error}")
                 });
                 let identity = reqwest::Identity::from_pem(&cert_pem)
-                    .unwrap_or_else(|error| panic!("error creating identity: {error}"));
+                    .unwrap_or_else(|error| panic!("error parsing cert: {error}"));
                 builder = builder.use_rustls_tls().identity(identity);
             }
             let client = builder

src/orchestrator.rs

@@ -142,13 +142,15 @@ impl Orchestrator {
             }
         });
         match task_handle.await {
+            // Task completed successfully
             Ok(Ok(result)) => Ok(result),
+            // Task failed, return error propagated from child task that failed
             Ok(Err(error)) => {
                 error!(request_id = ?task.request_id, %error, "unary task failed");
                 Err(error)
             }
+            // Task cancelled or panicked
             Err(error) => {
-                // Task failed due to cancellation or panic
                 let error = error.into();
                 error!(request_id = ?task.request_id, %error, "unary task failed");
                 Err(error)