cli: fix crash on wrong server base url (#25497)
* llama-cli: fix crash on wrong server base url by catching exceptions and graceful exit * review: leaner catch group: json error and standard exception
This commit is contained in:
@@ -153,9 +153,19 @@ bool cli_context::init() {
|
|||||||
|
|
||||||
if (use_external_server) {
|
if (use_external_server) {
|
||||||
spinner.reset();
|
spinner.reset();
|
||||||
if (!list_and_ask_models()) {
|
try {
|
||||||
|
if (!list_and_ask_models()) {
|
||||||
|
return false;
|
||||||
|
}
|
||||||
|
} catch (const json::parse_error & e) {
|
||||||
|
ui::show_error(e.what());
|
||||||
|
ui::show_message("This might be caused by an incorrect server-base endpoint URL");
|
||||||
|
return false;
|
||||||
|
} catch (const std::exception & e) {
|
||||||
|
ui::show_error(e.what());
|
||||||
return false;
|
return false;
|
||||||
}
|
}
|
||||||
|
|
||||||
// restore the spinner for the next step
|
// restore the spinner for the next step
|
||||||
spinner.emplace("Waiting for server...");
|
spinner.emplace("Waiting for server...");
|
||||||
}
|
}
|
||||||
|
|||||||
Reference in New Issue
Block a user