Add error panel, panic hook, and updated dependencies (#1)

Cargo.toml

@@ -12,17 +12,17 @@ edition = "2021"
 
 [dependencies]
 clap = { version = "4.3.0", features = ["std", "derive", "help", "usage", "suggestions"], default-features = false }
-ureq = { version = "2.6.2", features = ["json", "tls"], default-features = false }
+ureq = { version = "2.7.1", features = ["json", "tls"], default-features = false }
 serde = { version = "1.0", features = ["derive"], default-features = false }
-serde_with = { version = "2.2.0", default-features = false }
+serde_with = { version = "3.0.0", default-features = false }
 serde_json = { version = "1.0", default-features = false }
 tabled = { version = "0.10", features = ["color"], default-features = false }
 owo-colors = { version = "3.5.0", default-features = false }
 json_to_table = { version = "*", default-features = false }
 base64 = { version = "0.21.0", features = ["std"], default-features = false }
-rustls = { version = "0.20.8", features = ["dangerous_configuration"], default-features = false }
+rustls = { version = "0.21.2", features = ["dangerous_configuration"], default-features = false }
 colored_json = { version = "3", default-features = false }
-chrono = { version = "0.4.23", features = ["std", "default"], default-features = false }
+chrono = { version = "0.4.26", features = ["std", "default"], default-features = false }
 humansize = { version = "2.1.3", default-features = false }
 humantime = { version = "2.1.0" }
 tui = { version = "0.19.0", features = ["crossterm"] }

src/cli/commands/view/app.rs

@@ -280,6 +280,7 @@ pub(crate) struct App<'a> {
     pub should_quit: bool,
     pub connected: bool,
     pub show_help: bool,
+    pub show_error: Option<String>,
     pub tabs: TabsState,
     pub pipelines: StatefulTable<PipelineItem>,
     pub selected_pipeline_vertex: StatefulTable<String>,
@@ -319,6 +320,7 @@ impl<'a> App<'a> {
             show_selected_pipeline_charts: false,
             show_selected_vertex_details: false,
             show_help: false,
+            show_error: None,
             should_quit: false,
             connected: false,
             tabs: TabsState::new(),
@@ -433,24 +435,28 @@ impl<'a> App<'a> {
 
     pub fn on_tick(&mut self) {
         match self.data_fetcher.fetch_info() {
-            Ok(stats) => {
-                self.state.node_info = Some(stats);
+            Ok(info) => {
+                self.state.node_info = Some(info);
                 self.connected = true;
+                self.show_error = None;
             }
-            Err(_) => {
+            Err(err) => {
                 self.state.node_info = None;
                 self.connected = false;
+                self.show_error = Some(err.to_string());
             }
         };
 
         match self.data_fetcher.fetch_stats() {
             Ok(stats) => {
                 self.state.node_stats = Some(stats);
                 self.connected = true;
+                self.show_error = None;
             }
-            Err(_) => {
+            Err(err) => {
                 self.state.node_stats = None;
                 self.connected = false;
+                self.show_error = Some(err.to_string());
             }
         };
 

src/cli/commands/view/ui.rs

@@ -17,7 +17,7 @@ use crate::commands::view::node_charts::render_node_charts;
 use crate::commands::view::pipeline_view;
 
 pub(crate) fn draw<B: Backend>(f: &mut Frame<B>, app: &mut App) {
-    let constraints = if app.show_help {
+    let constraints = if app.show_help || app.show_error.is_some() {
         vec![
             Constraint::Length(3),
             Constraint::Min(0),
@@ -109,7 +109,7 @@ pub(crate) fn draw<B: Backend>(f: &mut Frame<B>, app: &mut App) {
         Span::styled(" @ ", Style::default().fg(Color::Gray)),
         Span::from(app.host),
         Span::styled(
-            format!(" | Sampling every {}ms", app.refresh_interval.as_millis()),
+            format!(" | Sampling every {}s", app.refresh_interval.as_secs()),
             Style::default().fg(Color::Gray),
         ),
     ])];
@@ -128,11 +128,37 @@ pub(crate) fn draw<B: Backend>(f: &mut Frame<B>, app: &mut App) {
         };
     }
 
-    if app.show_help {
+    if app.show_error.is_some() {
+        draw_error_panel(f, app, chunks[2]);
+    } else if app.show_help {
         draw_help_panel(f, chunks[2]);
     }
 }
 
+fn draw_error_panel<B>(f: &mut Frame<B>, app: &mut App, area: Rect)
+where
+    B: Backend,
+{
+    if let Some(error) = &app.show_error {
+        f.render_widget(Block::default().borders(Borders::ALL), area);
+
+        let footer_chunks = Layout::default()
+            .constraints([Constraint::Percentage(100)])
+            .direction(Direction::Horizontal)
+            .margin(1)
+            .split(area);
+
+        let w = Paragraph::new(vec![Spans::from(vec![
+            Span::styled("Error: ", Style::default().fg(Color::Red)),
+            Span::styled(error, Style::default().fg(Color::DarkGray)),
+        ])])
+        .alignment(Alignment::Left)
+        .wrap(Wrap { trim: true });
+
+        f.render_widget(w, footer_chunks[0]);
+    }
+}
+
 fn draw_help_panel<B>(f: &mut Frame<B>, area: Rect)
 where
     B: Backend,

src/cli/main.rs

@@ -1,3 +1,6 @@
+use std::backtrace::Backtrace;
+use std::panic;
+
 use crate::config::Config;
 use crate::errors::AnyError;
 use crate::output::Output;
@@ -27,6 +30,8 @@ fn run() -> Result<ExitCode, AnyError> {
 }
 
 fn main() {
+    setup_panic_hook();
+
     let result = run();
     match result {
         Err(err) => {
@@ -38,3 +43,25 @@ fn main() {
         }
     }
 }
+
+fn setup_panic_hook() {
+    panic::set_hook(Box::new(move |panic_info| {
+        let backtrace = Backtrace::force_capture().to_string();
+        let loc = if let Some(location) = panic_info.location() {
+            format!(" in file {} at line {}", location.file(), location.line())
+        } else {
+            String::new()
+        };
+
+        let message = if let Some(value) = panic_info.payload().downcast_ref::<&str>() {
+            value
+        } else if let Some(value) = panic_info.payload().downcast_ref::<String>() {
+            value
+        } else {
+            "Payload not captured as it is not a string."
+        };
+
+        println!("Panic occurred{} with the message: {}", loc, message);
+        println!("Panic backtrace: \n{}", backtrace);
+    }));
+}