Skip to content

fix(tray): use /live instead of /api/v1/health for server readiness check #2523

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
blackdeathdrow wants to merge 2 commits into
base: main
Choose a base branch
from
+10 −1
Open

fix(tray): use /live instead of /api/v1/health for server readiness check #2523

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
846ee46
fix(tray): pass API key in wait_for_server health check instead of sw…
blackdeathdrow Jul 2, 2026
3822b38
add missing include cstdlib for std::getenv
blackdeathdrow Jul 2, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
11 changes: 10 additions & 1 deletion src/cpp/tray/main.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,7 @@

#include <atomic>
#include <chrono>
#include <cstdlib>
#include <iostream>
#include <string>
#include <thread>
Expand Down Expand Up @@ -69,14 +70,22 @@ static bool wait_for_server(const std::string& host, int port, int timeout_secon
std::string connect_host = (host.empty() || host == "0.0.0.0" || host == "localhost")
? "127.0.0.1" : host;

// Pass API key if set - prefer admin key over regular API key
const char* admin_api_key = std::getenv("LEMONADE_ADMIN_API_KEY");
const char* api_key = admin_api_key ? admin_api_key : std::getenv("LEMONADE_API_KEY");
httplib::Headers headers;
if (api_key && api_key[0]) {
headers.emplace("Authorization", std::string("Bearer ") + api_key);
}

for (int i = 0; i < timeout_seconds * 2; ++i) {
try {
httplib::Client cli(connect_host, port);
cli.set_connection_timeout(1);
cli.set_read_timeout(5);
// Use /api/v1/health instead of /live — /live responds before the model
// cache is built, which causes 500s on /models if clients connect too early.
auto res = cli.Get("/api/v1/health");
auto res = cli.Get("/api/v1/health", headers);
if (res && res->status == 200) {
return true;
}
Expand Down
Loading