Manually print error chain

reqwest#2199[0] removed the error source from its Display impl, causing
us to lose context on why a request failed.

Manually iterate over error sources and append them to output message to
restore the original behavior. The first source is the intial error that
we have wrapped, so that must be skipped in order to prevent duplicating
the message.

[0] seanmonstar/reqwest#2199

Author: wfchandler

cli/src/cmd_auth.rs

@@ -4,6 +4,7 @@
 
 // Copyright 2024 Oxide Computer Company
 
+use std::error::Error;
 use std::fs::{File, OpenOptions};
 use std::io::{self, Write};
 use std::path::Path;
@@ -512,7 +513,20 @@ impl CmdAuthStatus {
     fn error_msg(e: &oxide::Error<oxide::types::Error>) -> String {
         match e {
             oxide::Error::ErrorResponse(ee) => format!("Error Response: {}", ee.message),
-            ee => ee.to_string(),
+            ee => {
+                let mut err = ee.to_string();
+
+                // The immediate source of the error is already shown in the
+                // Display impl. We need to skip the first source to avoid
+                // duplicating it.
+                let mut s = ee.source().and_then(|e| e.source());
+                while let Some(source) = s {
+                    err += &format!(": {source}");
+                    s = source.source();
+                }
+
+                err
+            }
         }
     }
 }

cli/tests/data/test_auth_status.stdout

@@ -1,4 +1,4 @@
 Profile "jennifer" (<TEST-SERVER>) status: Authenticated
 Profile "lightman" (<TEST-SERVER>) status: Authenticated
 Profile "malvin" (<TEST-SERVER>) status: Error Response: ** IMPROPER REQUEST **
-Profile "sting" (https://unresolvabledomainnameihope) status: Communication Error: error sending request for url (https://unresolvabledomainnameihope/v1/me)
+Profile "sting" (https://unresolvabledomainnameihope) status: Communication Error: error sending request for url (https://unresolvabledomainnameihope/v1/me): client error (Connect):