Skip to content

Commit 7d12067

Browse files
authored
Fix banner if fetching latest release fails (#244)
1 parent 100b848 commit 7d12067

File tree

1 file changed

+13
-17
lines changed

1 file changed

+13
-17
lines changed

server/src/banner.rs

Lines changed: 13 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -101,24 +101,10 @@ pub mod about {
101101
current_version,
102102
);
103103

104-
// check for latest release, if it cannot be fetched then print error as warn and return
105-
let latest_release = match update::get_latest() {
106-
Ok(latest_release) => latest_release,
107-
Err(_) => {
108-
return;
104+
if let Ok(latest_release) = update::get_latest() {
105+
if latest_release.version > current_version {
106+
print_latest_release(latest_release);
109107
}
110-
};
111-
112-
if latest_release.version > current_version {
113-
let time_since_latest_release = chrono::Utc::now() - latest_release.date;
114-
let time_since_latest_release = humanize_time(time_since_latest_release);
115-
116-
let fmt_latest_version = format!(
117-
" ( v{} released {} ago. Download new release from https://github.com/parseablehq/parseable/releases/latest )",
118-
latest_release.version, time_since_latest_release
119-
);
120-
121-
eprint!("{}", fmt_latest_version.red());
122108
}
123109

124110
eprintln!(
@@ -129,6 +115,16 @@ pub mod about {
129115
);
130116
}
131117

118+
fn print_latest_release(latest_release: update::LatestRelease) {
119+
let time_since_latest_release = chrono::Utc::now() - latest_release.date;
120+
let time_since_latest_release = humanize_time(time_since_latest_release);
121+
let fmt_latest_version = format!(
122+
" ( v{} released {} ago. Download new release from https://github.com/parseablehq/parseable/releases/latest )",
123+
latest_release.version, time_since_latest_release
124+
);
125+
eprint!("{}", fmt_latest_version.red());
126+
}
127+
132128
pub fn print() {
133129
// print current version
134130
let current = current();

0 commit comments

Comments
 (0)